[kudu-CR] stopwatch: avoid clang use-after-move warnings in LOG TIMING macros

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8502 ) Change subject: stopwatch: avoid clang use-after-move warnings in LOG_TIMING macros .. stopwatch: avoid clang use-after-move

[kudu-CR] stopwatch: avoid clang use-after-move warnings in LOG TIMING macros

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8502 ) Change subject: stopwatch: avoid clang use-after-move warnings in LOG_TIMING macros .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] stopwatch: avoid clang use-after-move warnings in LOG TIMING macros

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has removed a vote on this change. Change subject: stopwatch: avoid clang use-after-move warnings in LOG_TIMING macros .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit

[kudu-CR] stopwatch: avoid clang use-after-move warnings in LOG TIMING macros

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8502 ) Change subject: stopwatch: avoid clang use-after-move warnings in LOG_TIMING macros .. Patch Set 3: Verified+1 unrelated flake -- To

[kudu-CR] stopwatch: avoid clang use-after-move warnings in LOG TIMING macros

2017-11-08 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8502 to look at the new patch set (#3). Change subject: stopwatch: avoid clang use-after-move warnings in LOG_TIMING macros

[kudu-CR] stopwatch: avoid clang use-after-move warnings in LOG TIMING macros

2017-11-08 Thread Todd Lipcon (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8502 to look at the new patch set (#2). Change subject: stopwatch: avoid clang use-after-move warnings in LOG_TIMING macros

[kudu-CR] stopwatch: avoid clang use-after-move warnings in LOG TIMING macros

2017-11-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8502 ) Change subject: stopwatch: avoid clang use-after-move warnings in LOG_TIMING macros .. Patch Set 1: apparently my constexpr trick is