Chromium Code Reviews

Unified Diff: sandbox/win/src/process_mitigations_test.cc

Issue 1220133003: Fixed all unused-variable Clang warnings on Windows. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@iaccessible2-fix-gn
Patch Set: Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Index: sandbox/win/src/process_mitigations_test.cc
diff --git a/sandbox/win/src/process_mitigations_test.cc b/sandbox/win/src/process_mitigations_test.cc
index 4d2e9c6e9562a6abdd3488afaa2a4636cc893fac..4b3d3956973d08b9888f5a030f77aa49ec5610f8 100644
--- a/sandbox/win/src/process_mitigations_test.cc
+++ b/sandbox/win/src/process_mitigations_test.cc
@@ -165,9 +165,9 @@ SBOX_TESTS_COMMAND int CheckDep(int argc, wchar_t **argv) {
return SBOX_TEST_THIRD_ERROR;
}
- static const int MEM_EXECUTE_OPTION_ENABLE = 1;
+ // static const int MEM_EXECUTE_OPTION_ENABLE = 1;
Nico 2015/07/02 17:34:26 ?
Matt Giuca 2015/07/03 03:46:29 Well I thought that since this seems to represent
Nico 2015/07/03 04:11:15 As-is, it looks like code that was left in acciden
Matt Giuca 2015/07/03 04:33:45 jschuh: Can I get your quick opinion on this. You
static const int MEM_EXECUTE_OPTION_DISABLE = 2;
- static const int MEM_EXECUTE_OPTION_ATL7_THUNK_EMULATION = 4;
+ // static const int MEM_EXECUTE_OPTION_ATL7_THUNK_EMULATION = 4;
Nico 2015/07/02 17:34:26 ?
jschuh 2015/07/06 22:16:18 Yeah, looks like dead constants. lgtm on removing
Matt Giuca 2015/07/07 04:52:17 Well they were dead the moment they were added, so
static const int MEM_EXECUTE_OPTION_PERMANENT = 8;
dep_flags &= 0xff;

Powered by Google App Engine