Re: Review Request 58038: Removed redundant call to `std::string::c_str()` in Mesos.

2017-03-30 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58038/#review170536 --- Ship it! - Benjamin Bannier On March 30, 2017, 3:15 a.m.,

Re: Review Request 58038: Removed redundant call to `std::string::c_str()` in Mesos.

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

Re: Review Request 58038: Removed redundant call to `std::string::c_str()` in Mesos.

2017-03-29 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58038/#review170479 --- Could you add `readability-redundant-string-cstr` to the default

Review Request 58038: Removed redundant call to `std::string::c_str()` in Mesos.

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