-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34018/
-----------------------------------------------------------

(Updated May 9, 2015, 7:50 p.m.)


Review request for mesos, Benjamin Hindman and Joris Van Remoortere.


Bugs: MESOS-2670
    https://issues.apache.org/jira/browse/MESOS-2670


Repository: mesos


Description
-------

According Joris advice, only replace the 'lambda::bind' expressions match these 
rules:

1. Binds to a static function without any side-effects.
2. Is self contained (i.e. does not rely on contextual parameters)
3. Does not bind in any arguments. (i.e. only uses lambda::_N for arguments)
4. Is only called in 1 place OR is so small that it is ok to repeat the code.


Diffs (updated)
-----

  src/linux/cgroups.cpp df3211a0c25d7a16f36814886d14f81caaef2b9c 
  src/log/network.hpp 7c74a55cc2a71fa2acd207605f972e7fbd203be4 
  src/slave/containerizer/isolators/cgroups/cpushare.cpp 
6a5b2b5c6e2844fe1a10815956569194b6f56681 
  src/slave/containerizer/isolators/cgroups/mem.cpp 
2c218b2b83cf42f54dbc7ec4c2ba8960b6e194de 
  src/slave/containerizer/isolators/network/port_mapping.cpp 
a4abaff30bb4646b1b1edfdbbc243c9e3f6851df 
  src/slave/containerizer/mesos/containerizer.cpp 
f2587280dc0e1d566d2b856a80358c7b3896c603 

Diff: https://reviews.apache.org/r/34018/diff/


Testing
-------

make check


Thanks,

haosdent huang

Reply via email to