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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: rlz/win/lib/process_info.cc

Issue 1513043002: clang/win: Let remaining chromium_code targets build with -Wextra. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.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
« device/hid/hid_service_win.cc ('K') | « rlz/win/lib/machine_deal.cc ('k') | rlz/win/lib/rlz_value_store_registry.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: rlz/win/lib/process_info.cc
diff --git a/rlz/win/lib/process_info.cc b/rlz/win/lib/process_info.cc
index cbab44a00e5ea62c56978712d680d8ff923de227..8eceb352bbeb740648452c61531dcc7c775f1372 100644
--- a/rlz/win/lib/process_info.cc
+++ b/rlz/win/lib/process_info.cc
@@ -59,7 +59,7 @@ bool GetUserGroup(long* group) {
DOMAIN_ALIAS_RID_POWER_USERS};
SID_IDENTIFIER_AUTHORITY nt_authority = {SECURITY_NT_AUTHORITY};
- for (int i = 0; i < arraysize(user_groups) && *group == 0; ++i) {
+ for (size_t i = 0; i < arraysize(user_groups) && *group == 0; ++i) {
PSID current_group;
if (AllocateAndInitializeSid(&nt_authority, 2,
SECURITY_BUILTIN_DOMAIN_RID,
« device/hid/hid_service_win.cc ('K') | « rlz/win/lib/machine_deal.cc ('k') | rlz/win/lib/rlz_value_store_registry.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698