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

Unified Diff: include/utils/win/SkHRESULT.h

Issue 1503463004: clang/win: Let SK_TRACEHR not produce -Wunused-value warnings in release builds. (Closed) Base URL: https://chromium.googlesource.com/skia.git@master
Patch Set: . Created 5 years 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: include/utils/win/SkHRESULT.h
diff --git a/include/utils/win/SkHRESULT.h b/include/utils/win/SkHRESULT.h
index a6cc1d21b9db5e7e463453af96a37c7587632826..a0ffd5935cd0aafc9cbf06b8c0abe3cfdee8922f 100644
--- a/include/utils/win/SkHRESULT.h
+++ b/include/utils/win/SkHRESULT.h
@@ -17,7 +17,7 @@ void SkTraceHR(const char* file, unsigned long line,
#ifdef SK_DEBUG
#define SK_TRACEHR(_hr, _msg) SkTraceHR(__FILE__, __LINE__, _hr, _msg)
#else
-#define SK_TRACEHR(_hr, _msg) _hr
+#define SK_TRACEHR(_hr, _msg) sk_ignore_unused_variable(_hr)
#endif
#define HR_GENERAL(_ex, _msg, _ret) {\
« 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