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

(Updated May 15, 2014, 4:34 p.m.)


Review request for mesos, Benjamin Hindman and Dominic Hamon.


Changes
-------

Rebased.


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


Repository: mesos-git


Description
-------

Enabled whitespace/blank_line rule for cpplint, which covers the following:
'Redundant blank line at the start of a code block '
'should be deleted.'
'Redundant blank line at the end of a code block '
'should be deleted.'
'Do not leave a blank line after "public|protected|private:"'
'"class|struct:" should be preceded by a blank line'


Diffs (updated)
-----

  src/authorizer/authorizer.hpp f37091a 
  src/common/resources.cpp 512dabf 
  src/slave/containerizer/linux_launcher.hpp 84249c3 
  src/slave/slave.cpp 4a8adf0 
  src/tests/cluster.hpp 5dce50d 
  src/tests/isolator.hpp b1a3ac2 
  src/tests/mesos.hpp 15afbbb 
  support/mesos-style.py 9948817 

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


Testing (updated)
-------

Ran mesos-style.py over entire source base with whitespace/blank_line enabled.
$ support/mesos-style.py
Checking 453 files using filter 
--filter=-,+build/class,+build/deprecated,+build/endif_comment,+runtime/vlog,+whitespace/blank_line,+whitespace/comma,+whitespace/ending_newline,+whitespace/forcolon,+whitespace/indent,+whitespace/line_length,+whitespace/tab,+whitespace/todo
Total errors found: 0


Thanks,

Adam B

Reply via email to