Re: Review Request 71986: Removed redundant calls to `c_str` flagged by mesos-tidy.

2020-01-13 Thread Till Toenshoff via Review Board

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/71986/#review219234
---


Ship it!




Ship It!

- Till Toenshoff


On Jan. 13, 2020, 9:49 a.m., Benjamin Bannier wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/71986/
> ---
> 
> (Updated Jan. 13, 2020, 9:49 a.m.)
> 
> 
> Review request for mesos and Benno Evers.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> These calls were flagged by the upstream
> `readability-redundant-string-cstr` check.
> 
> 
> Diffs
> -
> 
>   src/linux/systemd.cpp d9cb5662cbf8e4d3531518af8bb633d9bb12d038 
> 
> 
> Diff: https://reviews.apache.org/r/71986/diff/1/
> 
> 
> Testing
> ---
> 
> confirmed that `clang-tidy` does not report anything from these lines anymore
> 
> 
> Thanks,
> 
> Benjamin Bannier
> 
>



Review Request 71986: Removed redundant calls to `c_str` flagged by mesos-tidy.

2020-01-13 Thread Benjamin Bannier

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/71986/
---

Review request for mesos and Benno Evers.


Repository: mesos


Description
---

These calls were flagged by the upstream
`readability-redundant-string-cstr` check.


Diffs
-

  src/linux/systemd.cpp d9cb5662cbf8e4d3531518af8bb633d9bb12d038 


Diff: https://reviews.apache.org/r/71986/diff/1/


Testing
---

confirmed that `clang-tidy` does not report anything from these lines anymore


Thanks,

Benjamin Bannier