clang-format the JS directory

.github/actions/clang-format/action.yml

@@ -21,4 +21,4 @@ runs:
       run: |
         shopt -s globstar
         shopt -s nullglob
-        clang-format-${{ inputs.version }} --dry-run --Werror **/*.cpp **/*.h **/*.m **/*.mm
+        clang-format-${{ inputs.version }} --dry-run --Werror **/*.cpp **/*.cc **/*.h **/*.hh **/*.m **/*.mm

.github/workflows/validate-cpp.yml

@@ -19,3 +19,8 @@ jobs:
         uses: ./.github/actions/clang-format
         with:
           directory: ./yoga
+
+      - name: clang-format (JavaScript Bindings)
+        uses: ./.github/actions/clang-format
+        with:
+          directory: ./javascript

javascript/sources/Node.cc

@@ -369,7 +369,7 @@ Node* Node::getChild(unsigned index) {
   return Node::fromYGNode(nodePtr);
 }
 
-void Node::setMeasureFunc(MeasureCallback *measureFunc) {
+void Node::setMeasureFunc(MeasureCallback* measureFunc) {
   m_measureFunc.reset(measureFunc);
 
   YGNodeSetMeasureFunc(m_node, &globalMeasureFunc);
@@ -389,7 +389,7 @@ Size Node::callMeasureFunc(
   return m_measureFunc->measure(width, widthMode, height, heightMode);
 }
 
-void Node::setDirtiedFunc(DirtiedCallback *dirtiedFunc) {
+void Node::setDirtiedFunc(DirtiedCallback* dirtiedFunc) {
   m_dirtiedFunc.reset(dirtiedFunc);
 
   YGNodeSetDirtiedFunc(m_node, &globalDirtiedFunc);

javascript/sources/Node.hh

@@ -20,7 +20,8 @@
 class MeasureCallback {
 public:
   virtual ~MeasureCallback() {}
-  virtual Size measure(float width,
+  virtual Size measure(
+      float width,
       int widthMode,
       float height,
       int heightMode) = 0;
@@ -29,8 +30,7 @@ public:
 class MeasureCallbackWrapper : public emscripten::wrapper<MeasureCallback> {
 public:
   EMSCRIPTEN_WRAPPER(MeasureCallbackWrapper);
-  Size measure(float width, int widthMode, float height, int heightMode)
+  Size measure(float width, int widthMode, float height, int heightMode) {
-  {
     return call<Size>("measure", width, widthMode, height, heightMode);
   }
 };
@@ -44,10 +44,7 @@ public:
 class DirtiedCallbackWrapper : public emscripten::wrapper<DirtiedCallback> {
 public:
   EMSCRIPTEN_WRAPPER(DirtiedCallbackWrapper);
-  void dirtied()
+  void dirtied() { return call<void>("dirtied"); }
-  {
-    return call<void>("dirtied");
-  }
 };
 
 class Node {
@@ -180,7 +177,7 @@ public: // Tree hierarchy inspectors
   Node* getChild(unsigned index);
 
 public: // Measure func mutators
-  void setMeasureFunc(MeasureCallback *measureFunc);
+  void setMeasureFunc(MeasureCallback* measureFunc);
   void unsetMeasureFunc(void);
 
 public: // Measure func inspectors
@@ -191,7 +188,7 @@ public: // Measure func inspectors
       int heightMode) const;
 
 public: // Dirtied func mutators
-  void setDirtiedFunc(DirtiedCallback *dirtiedFunc);
+  void setDirtiedFunc(DirtiedCallback* dirtiedFunc);
   void unsetDirtiedFunc(void);
 
 public: // Dirtied func inspectors

javascript/sources/embind.cc

@@ -19,21 +19,22 @@ using namespace emscripten;
 EMSCRIPTEN_BINDINGS(YOGA_LAYOUT) {
   class_<MeasureCallback>("MeasureCallback")
       .function("measure", &MeasureCallback::measure, pure_virtual())
-    .allow_subclass<MeasureCallbackWrapper>("MeasureCallbackWrapper")
+      .allow_subclass<MeasureCallbackWrapper>("MeasureCallbackWrapper");
-  ;
   class_<DirtiedCallback>("DirtiedCallback")
       .function("dirtied", &DirtiedCallback::dirtied, pure_virtual())
-    .allow_subclass<DirtiedCallbackWrapper>("DirtiedCallbackWrapper")
+      .allow_subclass<DirtiedCallbackWrapper>("DirtiedCallbackWrapper");
-  ;
 
   class_<Config>("Config")
       .constructor<>(&Config::create, allow_raw_pointers())
       .class_function<>("create", &Config::create, allow_raw_pointers())
       .class_function<>("destroy", &Config::destroy, allow_raw_pointers())
-    .function("setExperimentalFeatureEnabled", &Config::setExperimentalFeatureEnabled)
+      .function(
+          "setExperimentalFeatureEnabled",
+          &Config::setExperimentalFeatureEnabled)
       .function("setPointScaleFactor", &Config::setPointScaleFactor)
-    .function("isExperimentalFeatureEnabled", &Config::isExperimentalFeatureEnabled)
+      .function(
-  ;
+          "isExperimentalFeatureEnabled",
+          &Config::isExperimentalFeatureEnabled);
 
   value_object<Layout>("Layout")
       .field("left", &Layout::left)
@@ -41,24 +42,23 @@ EMSCRIPTEN_BINDINGS(YOGA_LAYOUT) {
       .field("top", &Layout::top)
       .field("bottom", &Layout::bottom)
       .field("width", &Layout::width)
-    .field("height", &Layout::height)
+      .field("height", &Layout::height);
-  ;
 
   value_object<Size>("Size")
       .field("width", &Size::width)
-    .field("height", &Size::height)
+      .field("height", &Size::height);
-  ;
 
   value_object<Value>("Value")
       .field("value", &Value::value)
-    .field("unit", &Value::unit)
+      .field("unit", &Value::unit);
-  ;
 
   class_<Node>("Node")
       .constructor<>(&Node::createDefault, allow_raw_pointers())
 
-    .class_function<>("createDefault", &Node::createDefault, allow_raw_pointers())
+      .class_function<>(
-    .class_function<>("createWithConfig", &Node::createWithConfig, allow_raw_pointers())
+          "createDefault", &Node::createDefault, allow_raw_pointers())
+      .class_function<>(
+          "createWithConfig", &Node::createWithConfig, allow_raw_pointers())
       .class_function<>("destroy", &Node::destroy, allow_raw_pointers())
       .function("reset", &Node::reset)
 
@@ -183,6 +183,5 @@ EMSCRIPTEN_BINDINGS(YOGA_LAYOUT) {
 
       .function("getComputedMargin", &Node::getComputedMargin)
       .function("getComputedBorder", &Node::getComputedBorder)
-    .function("getComputedPadding", &Node::getComputedPadding)
+      .function("getComputedPadding", &Node::getComputedPadding);
-  ;
 }