Bug 1971269 - Part 1: Handle *.hpp files in check_spidermonkey_style. r=spidermonkey-reviewers,jandem

Differential Revision: https://phabricator.services.mozilla.com/D255327

Author: anba

config/check_spidermonkey_style.py

@@ -267,7 +267,7 @@ def get(filename):
         if filename.endswith(("inlines.h", "-inl.h")):
             return FileKind.INL_H
 
-        if filename.endswith(".h"):
+        if filename.endswith((".h", ".hpp")):
             return FileKind.H
 
         if filename.endswith(".tbl"):
@@ -462,7 +462,7 @@ def section(self, enclosing_inclname):
         if self.is_system:
             return 2
 
-        if not self.inclname.endswith(".h"):
+        if not self.inclname.endswith((".h", ".hpp")):
             return 7
 
         # A couple of modules have the .h file in js/ and the .cpp file elsewhere and so need

js/src/builtin/intl/Segmenter.cpp

@@ -20,6 +20,9 @@
 #include "builtin/intl/CommonFunctions.h"
 #include "gc/AllocKind.h"
 #include "gc/GCContext.h"
+#include "icu4x/GraphemeClusterSegmenter.hpp"
+#include "icu4x/SentenceSegmenter.hpp"
+#include "icu4x/WordSegmenter.hpp"
 #include "js/CallArgs.h"
 #include "js/PropertyDescriptor.h"
 #include "js/PropertySpec.h"
@@ -37,10 +40,6 @@
 #include "vm/JSObject-inl.h"
 #include "vm/NativeObject-inl.h"
 
-#include "icu4x/GraphemeClusterSegmenter.hpp"
-#include "icu4x/SentenceSegmenter.hpp"
-#include "icu4x/WordSegmenter.hpp"
-
 using namespace js;
 
 const JSClassOps SegmenterObject::classOps_ = {

js/src/builtin/temporal/Calendar.cpp

@@ -30,6 +30,8 @@
 #include <stdint.h>
 #include <utility>
 
+
+#include "diplomat_runtime.hpp"
 #include "jsnum.h"
 #include "jstypes.h"
 #include "NamespaceImports.h"
@@ -52,6 +54,9 @@
 #include "builtin/temporal/ZonedDateTime.h"
 #include "gc/Barrier.h"
 #include "gc/GCEnum.h"
+#include "icu4x/Calendar.hpp"
+#include "icu4x/Date.hpp"
+#include "icu4x/IsoDate.hpp"
 #include "js/AllocPolicy.h"
 #include "js/ErrorReport.h"
 #include "js/friend/ErrorMessages.h"
@@ -72,11 +77,6 @@
 #include "vm/JSObject-inl.h"
 #include "vm/ObjectOperations-inl.h"
 
-#include "diplomat_runtime.hpp"
-#include "icu4x/Calendar.hpp"
-#include "icu4x/Date.hpp"
-#include "icu4x/IsoDate.hpp"
-
 // diplomat_simple_write isn't defined in C++ headers, but we have to use it to
 // avoid memory allocation.
 // (https://github.com/rust-diplomat/diplomat/issues/866)