Clean formatting after http://crrev.com/2544803002/.
As recommended, reformat the array definition (locally disable
clang-format) and use range-base loop (as it works with plain
arrays).
BUG=None
Review-Url: https://codereview.chromium.org/2556743002
Cr-Original-Commit-Position: refs/heads/master@{#436902}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: cdeaacd4acb0467526ea9a4ce27859f731417b6c
diff --git a/tools/gn/xcode_writer.cc b/tools/gn/xcode_writer.cc
index 6e50428..b6c0bc6 100644
--- a/tools/gn/xcode_writer.cc
+++ b/tools/gn/xcode_writer.cc
@@ -36,10 +36,15 @@
bool capture_at_generation;
};
+// clang-format off
SafeEnvironmentVariableInfo kSafeEnvironmentVariables[] = {
- {"HOME", true}, {"LANG", true}, {"PATH", true},
- {"USER", true}, {"TMPDIR", false},
+ {"HOME", true},
+ {"LANG", true},
+ {"PATH", true},
+ {"USER", true},
+ {"TMPDIR", false},
};
+// clang-format on
XcodeWriter::TargetOsType GetTargetOs(const Args& args) {
const Value* target_os_value = args.GetArgOverride(variables::kTargetOs);
@@ -63,8 +68,7 @@
// variable overridding settings, including the SDK, thus breaking hermetic
// build).
script << "env -i ";
- for (size_t i = 0; i < arraysize(kSafeEnvironmentVariables); ++i) {
- const auto& variable = kSafeEnvironmentVariables[i];
+ for (const auto& variable : kSafeEnvironmentVariables) {
script << variable.name << "=\"";
std::string value;
