Re: Review Request 58041: Don't mark a value parameter `const`.

2017-03-30 Thread Neil Conway
> On March 30, 2017, 8:36 a.m., Benjamin Bannier wrote: > > src/tests/fetcher_cache_tests.cpp > > Line 410 (original), 410 (patched) > > > > > > You do not need to remove the `const` here, e.g., the following is ok,

Re: Review Request 58041: Don't mark a value parameter `const`.

2017-03-30 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58041/#review170544 --- Fix it, then Ship it! src/tests/fetcher_cache_tests.cpp Line 4

Re: Review Request 58041: Don't mark a value parameter `const`.

2017-03-29 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58041/ --- (Updated March 30, 2017, 1:15 a.m.) Review request for mesos and Benjamin Banni

Review Request 58041: Don't mark a value parameter `const`.

2017-03-29 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58041/ --- Review request for mesos and Benjamin Bannier. Repository: mesos Description