Re: Review Request 45995: Fixed memory leak of `Route` in `finalize()` in libprocess.

2016-07-13 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45995/#review142097 --- Note: I replaced `NULL` with `nullptr` prior to committing. - Jos

Re: Review Request 45995: Fixed memory leak of `Route` in `finalize()` in libprocess.

2016-06-02 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45995/ --- (Updated June 2, 2016, 8:31 p.m.) Review request for mesos and Joris Van Remoor

Re: Review Request 45995: Fixed memory leak of `Route` in `finalize()` in libprocess.

2016-05-07 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45995/#review132130 --- Ship it! Ship It! - haosdent huang On May 3, 2016, 9:40 a.m.

Re: Review Request 45995: Fixed memory leak of `Route` in `finalize()` in libprocess.

2016-05-04 Thread Zhiwei Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45995/#review131788 --- Ship it! Ship It! - Zhiwei Chen On May 3, 2016, 5:40 p.m., N

Re: Review Request 45995: Fixed memory leak of `Route` in `finalize()` in libprocess.

2016-05-03 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45995/ --- (Updated May 3, 2016, 9:40 a.m.) Review request for mesos and Joris Van Remoort

Re: Review Request 45995: Fixed memory leak of `Route` in `finalize()` in libprocess.

2016-05-03 Thread Neil Conway
> On May 2, 2016, 5:53 p.m., haosdent huang wrote: > > 3rdparty/libprocess/src/process.cpp, line 1007 > > > > > > I am no sure if use `processes_route` style to keep consistent with > > `process_manager`. Good cat

Re: Review Request 45995: Fixed memory leak of `Route` in `finalize()` in libprocess.

2016-05-02 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45995/#review131340 --- 3rdparty/libprocess/src/process.cpp (line 1007)

Re: Review Request 45995: Fixed memory leak of `Route` in `finalize()` in libprocess.

2016-04-15 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45995/#review129106 --- Ship it! Ship It! - Alexander Rojas On April 11, 2016, 3:39

Re: Review Request 45995: Fixed memory leak of `Route` in `finalize()` in libprocess.

2016-04-14 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45995/#review128955 --- Ship it! Ship It! - Joseph Wu On April 10, 2016, 6:39 p.m.,