----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57160/#review167164 -----------------------------------------------------------
Fix it, then Ship it! 3rdparty/libprocess/src/libevent_ssl_socket.hpp (lines 47 - 48) <https://reviews.apache.org/r/57160/#comment239335> With the removal of `virtual`, this fits on one line again :) 3rdparty/libprocess/src/libevent_ssl_socket.hpp (lines 51 - 54) <https://reviews.apache.org/r/57160/#comment239336> Same whitespace with this line. - Joseph Wu On Feb. 28, 2017, 12:51 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57160/ > ----------------------------------------------------------- > > (Updated Feb. 28, 2017, 12:51 p.m.) > > > Review request for mesos, Benjamin Bannier and Joseph Wu. > > > Repository: mesos > > > Description > ------- > > Inconsistent use of the `override` keyword in > `LibeventSSLSocketImpl` was causing warnings during > clang builds. This patch makes use of the keyword > across all relevant declarations in the class. > > > Diffs > ----- > > 3rdparty/libprocess/src/libevent_ssl_socket.hpp > e589a04d14378f265a8fca871c9f5b0c577f5713 > > Diff: https://reviews.apache.org/r/57160/diff/ > > > Testing > ------- > > `make check` > > > Thanks, > > Greg Mann > >
