Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-14 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42030/#review114514 --- Ship it! Ship It! - Timothy Chen On Jan. 12, 2016, 11:34 p.m.,

Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42030/#review114151 --- Patch looks great! Reviews applied: [42030] Passed command: expor

Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-12 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42030/ --- (Updated Jan. 12, 2016, 11:33 p.m.) Review request for mesos, Ben Mahler, Jojy

Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-12 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42030/ --- (Updated Jan. 12, 2016, 11:31 p.m.) Review request for mesos, Ben Mahler, Jojy

Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-08 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42030/#review113534 --- 3rdparty/libprocess/README.md (line 257)

Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-08 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42030/#review113529 --- Patch looks great! Reviews applied: [42030] Passed command: expor

Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-08 Thread Neil Conway
> On Jan. 7, 2016, 11:21 p.m., Neil Conway wrote: > > 3rdparty/libprocess/README.md, line 323 > > > > > > Do we really have to look at all the code paths within `doHttpGet`? By > > the time `getBlob()` returns to th

Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-08 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42030/ --- (Updated Jan. 8, 2016, 5:01 p.m.) Review request for mesos, Ben Mahler, Jojy Va

Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-08 Thread Greg Mann
> On Jan. 7, 2016, 11:21 p.m., Neil Conway wrote: > > 3rdparty/libprocess/README.md, line 323 > > > > > > Do we really have to look at all the code paths within `doHttpGet`? By > > the time `getBlob()` returns to th

Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-08 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42030/ --- (Updated Jan. 8, 2016, 4:55 p.m.) Review request for mesos, Ben Mahler, Jojy Va

Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-07 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42030/#review113353 --- 3rdparty/libprocess/README.md (line 247)

Re: Review Request 42030: Added example of a `defer` bug to libprocess README.

2016-01-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42030/#review113319 --- Patch looks great! Reviews applied: [42030] Passed command: expor