Condense namespaces in tint/utils.

This PR condenses the namespaces in the tint/utils folder.

Change-Id: I6b541a6383386346a0b11ba46074a5981fe75f66
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/86028
Auto-Submit: Dan Sinclair <dsinclair@chromium.org>
Reviewed-by: Ben Clayton <bclayton@google.com>
Commit-Queue: Dan Sinclair <dsinclair@chromium.org>

src/tint/utils/defer.h

@@ -19,8 +19,7 @@
 
 #include "src/tint/utils/concat.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 /// Defer executes a function or function like object when it is destructed.
 template <typename F>
@@ -51,8 +50,7 @@ inline Defer<F> MakeDefer(F&& f) {
   return Defer<F>(std::forward<F>(f));
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 /// TINT_DEFER(S) executes the statement(s) `S` when exiting the current lexical
 /// scope.

src/tint/utils/defer_test.cc

@@ -16,8 +16,7 @@
 
 #include "gtest/gtest.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 TEST(DeferTest, Basic) {
@@ -40,5 +39,4 @@ TEST(DeferTest, DeferOrder) {
 }
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/enum_set.h

@@ -21,8 +21,7 @@
 #include <type_traits>
 #include <utility>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 /// EnumSet is a set of enum values.
 /// @note As the EnumSet is backed by a single uint64_t value, it can only hold
@@ -237,8 +236,7 @@ inline std::ostream& operator<<(std::ostream& out, EnumSet<ENUM> set) {
   return out << "}";
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 namespace std {
 

src/tint/utils/enum_set_test.cc

@@ -19,8 +19,7 @@
 
 #include "gmock/gmock.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 using ::testing::ElementsAre;
@@ -240,5 +239,4 @@ TEST(EnumSetTest, Ostream) {
 }
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/hash.h

@@ -20,8 +20,7 @@
 #include <functional>
 #include <vector>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace detail {
 
 /// Helper for obtaining a seed bias value for HashCombine with a bit-width
@@ -77,7 +76,6 @@ size_t Hash(const ARGS&... args) {
   return hash;
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 #endif  // SRC_TINT_UTILS_HASH_H_

src/tint/utils/hash_test.cc

@@ -18,8 +18,7 @@
 
 #include "gtest/gtest.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 TEST(HashTests, Basic) {
@@ -45,5 +44,4 @@ TEST(HashTests, Vector) {
 }
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/io/command.h

@@ -18,8 +18,7 @@
 #include <string>
 #include <utility>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 /// Command is a helper used by tests for executing a process with a number of
 /// arguments and an optional stdin string, and then collecting and returning
@@ -78,7 +77,6 @@ class Command {
   std::string input_;
 };
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 #endif  // SRC_TINT_UTILS_IO_COMMAND_H_

src/tint/utils/io/command_other.cc

@@ -14,8 +14,7 @@
 
 #include "src/tint/utils/io/command.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 Command::Command(const std::string&) {}
 
@@ -33,5 +32,4 @@ Command::Output Command::Exec(std::initializer_list<std::string>) const {
   return out;
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/io/command_posix.cc

@@ -21,8 +21,7 @@
 #include <sstream>
 #include <vector>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 namespace {
 
@@ -261,5 +260,4 @@ Command::Output Command::Exec(
   }
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/io/command_test.cc

@@ -16,8 +16,7 @@
 
 #include "gtest/gtest.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 #ifdef _WIN32
@@ -88,5 +87,4 @@ TEST(CommandTest, False) {
 #endif
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/io/command_windows.cc

@@ -19,8 +19,7 @@
 #include <sstream>
 #include <string>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 namespace {
 
@@ -245,5 +244,4 @@ Command::Output Command::Exec(
   return output;
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/io/tmpfile.h

@@ -18,8 +18,7 @@
 #include <sstream>
 #include <string>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 /// TmpFile constructs a temporary file that can be written to, and is
 /// automatically deleted on destruction.
@@ -70,7 +69,6 @@ class TmpFile {
   std::string path_;
 };
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 #endif  // SRC_TINT_UTILS_IO_TMPFILE_H_

src/tint/utils/io/tmpfile_other.cc

@@ -14,8 +14,7 @@
 
 #include "src/tint/utils/io/tmpfile.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 TmpFile::TmpFile(std::string) {}
 
@@ -25,5 +24,4 @@ bool TmpFile::Append(const void*, size_t) const {
   return false;
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/io/tmpfile_posix.cc

@@ -19,8 +19,7 @@
 
 #include "src/tint/debug.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 namespace {
 
@@ -66,5 +65,4 @@ bool TmpFile::Append(const void* data, size_t size) const {
   return false;
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/io/tmpfile_test.cc

@@ -18,8 +18,7 @@
 
 #include "gtest/gtest.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 TEST(TmpFileTest, WriteReadAppendDelete) {
@@ -86,5 +85,4 @@ TEST(TmpFileTest, FileExtension) {
 }
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/io/tmpfile_windows.cc

@@ -17,8 +17,7 @@
 #include <stdio.h>
 #include <cstdio>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 namespace {
 
@@ -59,5 +58,4 @@ bool TmpFile::Append(const void* data, size_t size) const {
   return true;
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/map.h

@@ -17,8 +17,7 @@
 
 #include <unordered_map>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 /// Lookup is a utility function for fetching a value from an unordered map if
 /// it exists, otherwise returning the `if_missing` argument.
@@ -56,7 +55,6 @@ V GetOrCreate(std::unordered_map<K, V, H, C>& map,
   return value;
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 #endif  // SRC_TINT_UTILS_MAP_H_

src/tint/utils/map_test.cc

@@ -18,8 +18,7 @@
 
 #include "gtest/gtest.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 TEST(Lookup, Test) {
@@ -54,5 +53,4 @@ TEST(GetOrCreateTest, ExistingKey) {
 }
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/math.h

@@ -19,8 +19,7 @@
 #include <string>
 #include <type_traits>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 /// @param alignment the next multiple to round `value` to
 /// @param value the value to round to the next multiple of `alignment`
@@ -51,7 +50,6 @@ inline std::enable_if_t<std::is_unsigned<T>::value, T> MaxAlignOf(T value) {
   return pot;
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 #endif  // SRC_TINT_UTILS_MATH_H_

src/tint/utils/math_test.cc

@@ -16,8 +16,7 @@
 
 #include "gtest/gtest.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 TEST(MathTests, RoundUp) {
@@ -79,5 +78,4 @@ TEST(MathTests, MaxAlignOf) {
 }
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/reverse.h

@@ -17,8 +17,7 @@
 
 #include <iterator>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 namespace detail {
 /// Used by utils::Reverse to hold the underlying iterable.
@@ -58,7 +57,6 @@ detail::ReverseIterable<T> Reverse(T&& iterable) {
   return {iterable};
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 #endif  // SRC_TINT_UTILS_REVERSE_H_

src/tint/utils/reverse_test.cc

@@ -18,8 +18,7 @@
 
 #include "gmock/gmock.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 TEST(ReverseTest, Vector) {
@@ -32,5 +31,4 @@ TEST(ReverseTest, Vector) {
 }
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/scoped_assignment.h

@@ -20,8 +20,7 @@
 
 #include "src/tint/utils/concat.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 /// Helper class that temporarily assigns a value to a variable for the lifetime
 /// of the ScopedAssignment object. Once the ScopedAssignment is destructed, the
@@ -50,8 +49,7 @@ class ScopedAssignment {
   T old_value_;
 };
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 /// TINT_SCOPED_ASSIGNMENT(var, val) assigns `val` to `var`, and automatically
 /// restores the original value of `var` when exiting the current lexical scope.

src/tint/utils/scoped_assignment_test.cc

@@ -16,8 +16,7 @@
 
 #include "gtest/gtest.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 TEST(ScopedAssignmentTest, Scopes) {
@@ -43,5 +42,4 @@ TEST(ScopedAssignmentTest, Scopes) {
 }
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/string.h

@@ -17,8 +17,7 @@
 
 #include <string>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 /// @param str the string to apply replacements to
 /// @param substr the string to search for
@@ -35,7 +34,6 @@ inline std::string ReplaceAll(std::string str,
   return str;
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 #endif  // SRC_TINT_UTILS_STRING_H_

src/tint/utils/string_test.cc

@@ -16,8 +16,7 @@
 
 #include "gtest/gtest.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 TEST(StringTest, ReplaceAll) {
@@ -34,5 +33,4 @@ TEST(StringTest, ReplaceAll) {
 }
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/to_const_ptr_vec.h

@@ -18,8 +18,7 @@
 
 #include <vector>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 /// @param in a vector of `T*`
 /// @returns a vector of `const T*` with the content of `in`.
@@ -33,7 +32,6 @@ std::vector<const T*> ToConstPtrVec(const std::vector<T*>& in) {
   return out;
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 #endif  // SRC_TINT_UTILS_TO_CONST_PTR_VEC_H_

src/tint/utils/transform.h

@@ -22,8 +22,7 @@
 
 #include "src/tint/traits.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 /// Transform performs an element-wise transformation of a vector.
 /// @param in the input vector.
@@ -56,7 +55,6 @@ auto Transform(const std::vector<IN>& in, TRANSFORMER&& transform)
   return result;
 }
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 #endif  // SRC_TINT_UTILS_TRANSFORM_H_

src/tint/utils/transform_test.cc

@@ -23,8 +23,7 @@
   static_assert(std::is_same<decltype(vector)::value_type, expected>::value, \
                 "unexpected result vector element type")
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 TEST(TransformTest, Empty) {
@@ -90,5 +89,4 @@ TEST(TransformTest, TransformDifferentType) {
 }
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint

src/tint/utils/unique_vector.h

@@ -21,8 +21,7 @@
 #include <utility>
 #include <vector>
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 
 /// UniqueVector is an ordered container that only contains unique items.
 /// Attempting to add a duplicate is a no-op.
@@ -107,7 +106,6 @@ struct UniqueVector {
   std::unordered_set<T, HASH, EQUAL> set;
 };
 
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint
 
 #endif  // SRC_TINT_UTILS_UNIQUE_VECTOR_H_

src/tint/utils/unique_vector_test.cc

@@ -17,8 +17,7 @@
 
 #include "gtest/gtest.h"
 
-namespace tint {
+namespace tint::utils {
-namespace utils {
 namespace {
 
 TEST(UniqueVectorTest, Empty) {
@@ -141,5 +140,4 @@ TEST(UniqueVectorTest, PopBack) {
 }
 
 }  // namespace
-}  // namespace utils
+}  // namespace tint::utils
-}  // namespace tint