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

Unified Diff: minidump/minidump_string_writer.h

Issue 699313007: minidump: Use the crashpad::internal namespace more appropriately (Closed) Base URL: https://chromium.googlesource.com/crashpad/crashpad@master
Patch Set: Created 6 years, 1 month 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
« no previous file with comments | « minidump/minidump_rva_list_writer_test.cc ('k') | minidump/minidump_string_writer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: minidump/minidump_string_writer.h
diff --git a/minidump/minidump_string_writer.h b/minidump/minidump_string_writer.h
index 319f507bba13c5c45f656fce8386e5e7ee10cd8a..a12885cd17bcf4d8e21322b6c1b84073e351f180 100644
--- a/minidump/minidump_string_writer.h
+++ b/minidump/minidump_string_writer.h
@@ -126,25 +126,11 @@ class MinidumpUTF8StringWriter final
DISALLOW_COPY_AND_ASSIGN(MinidumpUTF8StringWriter);
};
-//! \cond
-
-struct MinidumpStringListWriterUTF16Traits {
- using MinidumpStringWriterType = MinidumpUTF16StringWriter;
-};
-
-struct MinidumpStringListWriterUTF8Traits {
- using MinidumpStringWriterType = MinidumpUTF8StringWriter;
-};
-
-//! \endcond
-
//! \brief The writer for a MinidumpRVAList object in a minidump file,
-//! containing a list of \a Traits::MinidumpStringWriterType objects.
-template <typename Traits>
-class MinidumpStringListWriter : public MinidumpRVAListWriter {
+//! containing a list of \a MinidumpStringWriterType objects.
+template <typename MinidumpStringWriterType>
+class MinidumpStringListWriter final : public MinidumpRVAListWriter {
public:
- using MinidumpStringWriterType = typename Traits::MinidumpStringWriterType;
-
MinidumpStringListWriter();
~MinidumpStringListWriter() override;
@@ -165,7 +151,7 @@ class MinidumpStringListWriter : public MinidumpRVAListWriter {
//!
//! This object creates a new string writer with string value \a string_utf8,
//! takes ownership of it, and becomes its parent in the overall tree of
- //! internal::MinidumpWritable objects.
+ //! MinidumpWritable objects.
//!
//! \note Valid in #kStateMutable.
void AddStringUTF8(const std::string& string_utf8);
@@ -183,12 +169,13 @@ class MinidumpStringListWriter : public MinidumpRVAListWriter {
DISALLOW_COPY_AND_ASSIGN(MinidumpStringListWriter);
};
-using MinidumpUTF16StringListWriter =
- MinidumpStringListWriter<MinidumpStringListWriterUTF16Traits>;
-using MinidumpUTF8StringListWriter =
- MinidumpStringListWriter<MinidumpStringListWriterUTF8Traits>;
-
} // namespace internal
+
+using MinidumpUTF16StringListWriter = internal::MinidumpStringListWriter<
+ internal::MinidumpUTF16StringWriter>;
+using MinidumpUTF8StringListWriter = internal::MinidumpStringListWriter<
+ internal::MinidumpUTF8StringWriter>;
+
} // namespace crashpad
#endif // CRASHPAD_MINIDUMP_MINIDUMP_STRING_WRITER_H_
« no previous file with comments | « minidump/minidump_rva_list_writer_test.cc ('k') | minidump/minidump_string_writer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698