owenpan added inline comments.
================ Comment at: clang/include/clang/Format/Format.h:2124-2126 + /// namespaces and fixes invalid existing ones. This is omitted for short + /// namespaces, what a short namespace is, is controlled by + /// "ShortNamespaceLines". ---------------- Or something like that? ================ Comment at: clang/include/clang/Format/Format.h:2133 /// } // namespace a } + /// namespace short { namespace short { + /// void baz(); void baz(); ---------------- `short` is a C++ keyword. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137883/new/ https://reviews.llvm.org/D137883 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits