Re: Review Request 37018: [3/3] Removed whitespace padding in overloaded operators (stout).

2015-08-09 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37018/#review94681 --- Ship it! I'll get this committed for you since it's been agreed

Re: Review Request 37018: [3/3] Removed whitespace padding in overloaded operators (stout).

2015-08-09 Thread Artem Harutyunyan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37018/ --- (Updated Aug. 9, 2015, 3:16 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37018: [3/3] Removed whitespace padding in overloaded operators (stout).

2015-08-09 Thread Artem Harutyunyan
On Aug. 8, 2015, 3:07 p.m., Mesos ReviewBot wrote: Bad patch! Reviews applied: [37013, 37017, 37018] Failed command: ./support/apply-review.sh -n -r 37018 Error: 2015-08-08 22:07:03 URL:https://reviews.apache.org/r/37018/diff/raw/ [49035/49035] - 37018.patch [1] error:

Re: Review Request 37018: [3/3] Removed whitespace padding in overloaded operators (stout).

2015-08-09 Thread Michael Park
On Aug. 9, 2015, 11:10 p.m., Michael Park wrote: 3rdparty/libprocess/3rdparty/stout/include/stout/cache.hpp, line 98 https://reviews.apache.org/r/37018/diff/3/?file=1035715#file1035715line98 Removed unnecessary specialization: ``. Turns out I was wrong about this. Committed a fix.

Re: Review Request 37018: [3/3] Removed whitespace padding in overloaded operators (stout).

2015-08-08 Thread Artem Harutyunyan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37018/ --- (Updated Aug. 8, 2015, 2:54 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37018: [3/3] Removed whitespace padding in overloaded operators (stout).

2015-08-08 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37018/#review94653 --- Bad patch! Reviews applied: [37013, 37017, 37018] Failed command:

Re: Review Request 37018: [3/3] Removed whitespace padding in overloaded operators (stout).

2015-08-08 Thread Michael Park
On Aug. 8, 2015, 10:07 p.m., Mesos ReviewBot wrote: Bad patch! Reviews applied: [37013, 37017, 37018] Failed command: ./support/apply-review.sh -n -r 37018 Error: 2015-08-08 22:07:03 URL:https://reviews.apache.org/r/37018/diff/raw/ [49035/49035] - 37018.patch [1] error: