Re: Review Request 64807: Removed redundant `struct` prefix in "linux/ns.cpp".

2017-12-22 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64807/#review194435 --- FAIL: Failed to apply the dependent review: 64686. Failed

Re: Review Request 64807: Removed redundant `struct` prefix in "linux/ns.cpp".

2017-12-22 Thread Alexander Rukletsov
> On Dec. 22, 2017, 2:29 p.m., Benjamin Bannier wrote: > > src/linux/ns.cpp > > Line 184 (original), 184 (patched) > > > > > > Kill this `struct` as well? Not sure we can do this. There is also `stat()` POSIX

Re: Review Request 64807: Removed redundant `struct` prefix in "linux/ns.cpp".

2017-12-22 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64807/#review194428 --- Fix it, then Ship it! src/linux/ns.cpp Line 184 (original),

Review Request 64807: Removed redundant `struct` prefix in "linux/ns.cpp".

2017-12-22 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64807/ --- Review request for mesos, Benjamin Bannier and Michael Park. Repository: mesos