----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25623/#review53291 -----------------------------------------------------------
Bad patch! Reviews applied: [25623] Failed command: ./support/mesos-style.py Error: Checking 504 files using filter --filter=-,+build/class,+build/deprecated,+build/endif_comment,+readability/todo,+readability/namespace,+runtime/vlog,+whitespace/blank_line,+whitespace/comma,+whitespace/end_of_line,+whitespace/ending_newline,+whitespace/forcolon,+whitespace/indent,+whitespace/line_length,+whitespace/tab,+whitespace/todo 3rdparty/libprocess/3rdparty/stout/include/stout/net.hpp:102: Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2] Total errors found: 1 - Mesos ReviewBot On Sept. 14, 2014, 5:47 p.m., Kamil Domanski wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25623/ > ----------------------------------------------------------- > > (Updated Sept. 14, 2014, 5:47 p.m.) > > > Review request for mesos, Adam B, Benjamin Hindman, Bernd Mathiske, and Vinod > Kone. > > > Bugs: MESOS-1794 > https://issues.apache.org/jira/browse/MESOS-1794 > > > Repository: mesos-git > > > Description > ------- > > This divides {{Try<int> net::download(const std::string& url, const > std::string& path)}} into three overloads: > > 1. Try<int> net::download(const std::string& url, FILE* file) downloads a > file and sends it to FILE stream > 2. Try<int> net::download(const std::string& url, int fdout) opens a file > stream based on a file descriptor and calls 1) > 3. Try<int> net::download(const std::string& url, const std::string& path) > opens a file at _path_ and passes the descriptor to 2), effectively working > as before this change > > This will allow to download into any file descriptor, such as a pipe, e.g. > directly to extraction, as proposed by Bernd Mathiske in MESOS-1667. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/net.hpp 7138bc2 > > Diff: https://reviews.apache.org/r/25623/diff/ > > > Testing > ------- > > `cd 3rdparty/libprocess/ && make check` for compilation test only > > https://gist.github.com/kdomanski/f1d39266f6abc0f14f58 for operation test > > > Thanks, > > Kamil Domanski > >
