----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46094/#review128490 -----------------------------------------------------------
Patch looks great! Reviews applied: [45991, 45995, 45996, 45999, 46094] Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On April 12, 2016, 2:02 p.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46094/ > ----------------------------------------------------------- > > (Updated April 12, 2016, 2:02 p.m.) > > > Review request for mesos, Joris Van Remoortere and Joseph Wu. > > > Repository: mesos > > > Description > ------- > > Fixed memory leaks in Encoder/Decoder tests in libprocess. > > > Diffs > ----- > > 3rdparty/libprocess/src/tests/decoder_tests.cpp > bd990c5eb77e47d7f617199bcc89e9432af7cc51 > 3rdparty/libprocess/src/tests/encoder_tests.cpp > 61ec8d8722245179a929e954e6338606973b819b > > Diff: https://reviews.apache.org/r/46094/diff/ > > > Testing > ------- > > make check > > Verified that the number of leaked allocations decreases (from ~129 to ~92) > with this change. Obviously there are more leaks to investigate but at first > glance they seem a bit more subtle. > > > Thanks, > > Neil Conway > >