Resolve some simple clang-tidy warnings.

These include:

- use of nullptr
- use of `= default` for trivial methods
- use of nested namespaces
- use of emplace_back()
- matching /*variable=*/ comments with actual name.

Change-Id: I8a0639f442fd0fed39d3061b83d855302d7b7d7e
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/111170
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
diff --git a/constants/font_encodings.cpp b/constants/font_encodings.cpp
index 4359701..9b2d09f 100644
--- a/constants/font_encodings.cpp
+++ b/constants/font_encodings.cpp
@@ -4,8 +4,7 @@
 
 #include "constants/font_encodings.h"
 
-namespace pdfium {
-namespace font_encodings {
+namespace pdfium::font_encodings {
 
 // ISO 32000-1:2008 spec, table D1.
 const char kMacRomanEncoding[] = "MacRomanEncoding";
@@ -13,5 +12,4 @@
 const char kPDFDocEncoding[] = "PDFDocEncoding";
 const char kMacExpertEncoding[] = "MacExpertEncoding";
 
-}  // namespace font_encodings
-}  // namespace pdfium
+}  // namespace pdfium::font_encodings