Re: Review Request 60831: Removed extra/unnecessary allocations of Message.

2017-07-18 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60831/ --- (Updated July 18, 2017, 11:44 p.m.) Review request for mesos and Benjamin

Re: Review Request 60831: Removed extra/unnecessary allocations of Message.

2017-07-18 Thread Benjamin Hindman
> On July 15, 2017, 1:25 a.m., Benjamin Mahler wrote: > > 3rdparty/libprocess/src/process.cpp > > Line 455 (original), 455 (patched) > > > > > > Why did you decide to take `Message` here instead of `Message&&`? It

Re: Review Request 60831: Removed extra/unnecessary allocations of Message.

2017-07-14 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60831/#review180623 --- Fix it, then Ship it! Looks good, it would be nice to tease

Review Request 60831: Removed extra/unnecessary allocations of Message.

2017-07-14 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60831/ --- Review request for mesos and Benjamin Mahler. Repository: mesos Description