mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <>
Subject Review Request 58289: Avoided excessive copying in the default implementation of 'stringify'.
Date Mon, 10 Apr 2017 18:12:26 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos and Neil Conway.

Bugs: MESOS-6560

Repository: mesos


The default implementation of 'stringify' was always making needless
copies of its argument which can be expensive for larger types. This
patch changes the default overload of 'stringify' to always takes its
argument by const reference.

This change requires changing 'stringify' for 'bool' arguments from a
'stringify' template specialization to an overload.


  3rdparty/stout/include/stout/stringify.hpp 698431583d2288d3c635211e651914316bfd3ee9 



`make check` (Fedora 25)


Benjamin Bannier

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message