From d331d3a0b5c497a46e2636f308234be66566c04c Mon Sep 17 00:00:00 2001 From: Vincent Ambo Date: Wed, 20 May 2020 04:33:07 +0100 Subject: refactor(3p/nix): Apply clang-tidy's modernize-* fixes This applies the modernization fixes listed here: https://clang.llvm.org/extra/clang-tidy/checks/list.html The 'modernize-use-trailing-return-type' fix was excluded due to my personal preference (more specifically, I think the 'auto' keyword is misleading in that position). --- third_party/nix/src/libutil/xml-writer.cc | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'third_party/nix/src/libutil/xml-writer.cc') diff --git a/third_party/nix/src/libutil/xml-writer.cc b/third_party/nix/src/libutil/xml-writer.cc index ed41286cc1a3..11cd632399ad 100644 --- a/third_party/nix/src/libutil/xml-writer.cc +++ b/third_party/nix/src/libutil/xml-writer.cc @@ -1,6 +1,6 @@ #include "xml-writer.hh" -#include +#include namespace nix { @@ -68,8 +68,7 @@ void XMLWriter::writeEmptyElement(const string& name, const XMLAttrs& attrs) { void XMLWriter::writeAttrs(const XMLAttrs& attrs) { for (auto& i : attrs) { output << " " << i.first << "=\""; - for (size_t j = 0; j < i.second.size(); ++j) { - char c = i.second[j]; + for (char c : i.second) { if (c == '"') { output << """; } else if (c == '<') { -- cgit 1.4.1