No bug - Add comments to explain why they are ignored by clang-format r=me DONTBUILD
authorSylvestre Ledru <sledru@mozilla.com>
Wed, 30 Aug 2017 18:53:35 +0200
changeset 429653 2749a844a9f3e091d1944edf3fd93245cb5e7d5e
parent 429652 a8369cf48aebbe56520e308e30d4ef82abaec4ce
child 429654 d9b405d82cffb07343a5f2fd941e029298c7f6c4
push id1567
push userjlorenzo@mozilla.com
push dateThu, 02 Nov 2017 12:36:05 +0000
treeherdermozilla-release@e512c14a0406 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
milestone57.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
No bug - Add comments to explain why they are ignored by clang-format r=me DONTBUILD MozReview-Commit-ID: B8fsjYiWzkm
.clang-format-ignore
--- a/.clang-format-ignore
+++ b/.clang-format-ignore
@@ -1,27 +1,32 @@
+# Uses the LLVM coding style
 ^build/clang-plugin/.*
+# The two templates cannot be formatted
 ^config/gcc-stl-wrapper.template.h
 ^config/msvc-stl-wrapper.template.h
 ^dom/base/test/.*
 ^dom/bindings/test/.*
 ^dom/media/gtest/.*
 ^gfx/testsd/.*
 ^image/test/.*
 ^ipc/ipdl/test/.*
 ^ipc/testshell/.*
 ^js/src/jsapi-tests/.*
+# Ignored because of bug 1342657
 ^layout/style/nsCSSPropAliasList.h
+# Ignored because of bug 1342657
 ^layout/style/nsCSSPropList.h
 ^media/mtransport/test/.*
 ^mfbt/tests/.*
 ^storage/test/.*
 ^testing/gtest/.*
 ^tools/profiler/tests/.*
 ^uriloader/exthandler/tests/.*
+# JNI code is generated
 ^widget/android/GeneratedJNINatives.h
 ^widget/android/GeneratedJNIWrappers.cpp
 ^widget/android/GeneratedJNIWrappers.h
 ^widget/android/fennec/FennecJNINatives.h
 ^widget/android/fennec/FennecJNIWrappers.cpp
 ^widget/android/fennec/FennecJNIWrappers.h
 ^widget/tests/.*
 ^xpcom/glue/tests/.*