----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11331/#review21334 -----------------------------------------------------------
third_party/libprocess/third_party/stout/include/stout/duration.hpp <https://reviews.apache.org/r/11331/#comment44240> You can tack the string onto the output operator here instead of storing and assigning to the sign string, right? third_party/libprocess/third_party/stout/include/stout/duration.hpp <https://reviews.apache.org/r/11331/#comment44242> Hm.. what was the motivation behind this technique, it's quite hard to follow. Could this be done more simply without boost; instead using the modulo operator to detect whole numbers? - Ben Mahler On May 22, 2013, 7:08 p.m., Jiang Yan Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11331/ > ----------------------------------------------------------- > > (Updated May 22, 2013, 7:08 p.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > - Currently when the result isn't a whole number, we go one-level down in > terms of the unit of time and see if we can get a whole number there. If so, > use the lower unit; if not, use the higher unit. > > > This addresses bug MESOS-477. > https://issues.apache.org/jira/browse/MESOS-477 > > > Diffs > ----- > > third_party/libprocess/third_party/stout/include/stout/duration.hpp > c943ab196871cf4c55f2a2dce5ef85ffe3a22577 > third_party/libprocess/third_party/stout/tests/duration_tests.cpp > 421615aa510f540ccfc0413d3252bbdedd70f7da > > Diff: https://reviews.apache.org/r/11331/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jiang Yan Xu > >
