Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(764)

Unified Diff: src/platform/elapsed-timer.h

Issue 23523034: Replace uses of MUST_USE_RESULT by V8_WARN_UNUSED_RESULT in ElapsedTimer. (Closed) Base URL: git://github.com/v8/v8.git@master
Patch Set: Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/platform/elapsed-timer.h
diff --git a/src/platform/elapsed-timer.h b/src/platform/elapsed-timer.h
index e5bcf232f384348258eec5c7a239ce3ec148b007..773abbc8aacc8b1096fd00a06f1a5743c5fb88fd 100644
--- a/src/platform/elapsed-timer.h
+++ b/src/platform/elapsed-timer.h
@@ -87,7 +87,7 @@ class ElapsedTimer V8_FINAL BASE_EMBEDDED {
// Returns the time elapsed since the previous start. This method may only
// be called on a previously started timer.
- MUST_USE_RESULT TimeDelta Elapsed() const {
+ TimeDelta Elapsed() const V8_WARN_UNUSED_RESULT {
ASSERT(IsStarted());
TimeDelta elapsed = Now() - start_ticks_;
ASSERT(elapsed.InMicroseconds() >= 0);
@@ -97,13 +97,13 @@ class ElapsedTimer V8_FINAL BASE_EMBEDDED {
// Returns |true| if the specified |time_delta| has elapsed since the
// previous start, or |false| if not. This method may only be called on
// a previously started timer.
- MUST_USE_RESULT bool HasExpired(TimeDelta time_delta) const {
+ bool HasExpired(TimeDelta time_delta) const V8_WARN_UNUSED_RESULT {
ASSERT(IsStarted());
return Elapsed() >= time_delta;
}
private:
- MUST_USE_RESULT V8_INLINE(static TimeTicks Now()) {
+ V8_INLINE(static TimeTicks Now()) V8_WARN_UNUSED_RESULT {
TimeTicks now = TimeTicks::HighResNow();
ASSERT(!now.IsNull());
return now;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698