From dc589b541f9e34353c4c2cc5944a291590504f45 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Tue, 20 Oct 2020 06:54:39 -0700
Subject: [PATCH] doc,src,test: revise C++ code for linter update

PR-URL: https://github.com/nodejs/node/pull/35719
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
---
 doc/api/addons.md                                      | 2 +-
 src/node_report.cc                                     | 2 +-
 test/addons/async-cleanup-hook/binding.cc              | 2 +-
 test/addons/openssl-binding/binding.cc                 | 4 ++--
 test/addons/openssl-client-cert-engine/testengine.cc   | 6 +++---
 test/addons/openssl-key-engine/testkeyengine.cc        | 6 +++---
 test/addons/stringbytes-external-exceed-max/binding.cc | 2 +-
 test/addons/worker-addon/binding.cc                    | 6 +++---
 test/addons/zlib-binding/binding.cc                    | 2 +-
 9 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/doc/api/addons.md b/doc/api/addons.md
index b3402a7bf427ab..62573f55d06d48 100644
--- a/doc/api/addons.md
+++ b/doc/api/addons.md
@@ -269,9 +269,9 @@ The following `addon.cc` uses `AddEnvironmentCleanupHook`:
 
 ```cpp
 // addon.cc
+#include <node.h>
 #include <assert.h>
 #include <stdlib.h>
-#include <node.h>
 
 using node::AddEnvironmentCleanupHook;
 using v8::HandleScope;
diff --git a/src/node_report.cc b/src/node_report.cc
index e7bfe7fef09d14..3b97bb705b6985 100644
--- a/src/node_report.cc
+++ b/src/node_report.cc
@@ -12,8 +12,8 @@
 #ifdef _WIN32
 #include <Windows.h>
 #else  // !_WIN32
-#include <sys/resource.h>
 #include <cxxabi.h>
+#include <sys/resource.h>
 #include <dlfcn.h>
 #endif
 
diff --git a/test/addons/async-cleanup-hook/binding.cc b/test/addons/async-cleanup-hook/binding.cc
index d18da7a094f71a..0a11220850fca2 100644
--- a/test/addons/async-cleanup-hook/binding.cc
+++ b/test/addons/async-cleanup-hook/binding.cc
@@ -1,6 +1,6 @@
-#include <assert.h>
 #include <node.h>
 #include <uv.h>
+#include <assert.h>
 
 void MustNotCall(void* arg, void(*cb)(void*), void* cbarg) {
   assert(0);
diff --git a/test/addons/openssl-binding/binding.cc b/test/addons/openssl-binding/binding.cc
index 05849b6c14fe31..79472dd7f25272 100644
--- a/test/addons/openssl-binding/binding.cc
+++ b/test/addons/openssl-binding/binding.cc
@@ -1,7 +1,7 @@
-#include <node.h>
-#include <assert.h>
 #include <openssl/rand.h>
 #include <openssl/ssl.h>
+#include <node.h>
+#include <assert.h>
 
 namespace {
 
diff --git a/test/addons/openssl-client-cert-engine/testengine.cc b/test/addons/openssl-client-cert-engine/testengine.cc
index 7ccb56e08015a7..95712901e69c8b 100644
--- a/test/addons/openssl-client-cert-engine/testengine.cc
+++ b/test/addons/openssl-client-cert-engine/testengine.cc
@@ -1,10 +1,10 @@
+#include <openssl/engine.h>
+#include <openssl/pem.h>
+
 #include <assert.h>
 #include <string.h>
 #include <stdlib.h>
 
-#include <openssl/engine.h>
-#include <openssl/pem.h>
-
 #include <fstream>
 #include <iterator>
 #include <string>
diff --git a/test/addons/openssl-key-engine/testkeyengine.cc b/test/addons/openssl-key-engine/testkeyengine.cc
index e1dc22164ac4c1..704027ba5a43d2 100644
--- a/test/addons/openssl-key-engine/testkeyengine.cc
+++ b/test/addons/openssl-key-engine/testkeyengine.cc
@@ -1,10 +1,10 @@
+#include <openssl/engine.h>
+#include <openssl/pem.h>
+
 #include <assert.h>
 #include <string.h>
 #include <stdlib.h>
 
-#include <openssl/engine.h>
-#include <openssl/pem.h>
-
 #include <fstream>
 #include <iterator>
 #include <string>
diff --git a/test/addons/stringbytes-external-exceed-max/binding.cc b/test/addons/stringbytes-external-exceed-max/binding.cc
index 0e3cd3f4d0a1a1..c452a093849213 100644
--- a/test/addons/stringbytes-external-exceed-max/binding.cc
+++ b/test/addons/stringbytes-external-exceed-max/binding.cc
@@ -1,6 +1,6 @@
-#include <stdlib.h>
 #include <node.h>
 #include <v8.h>
+#include <stdlib.h>
 
 #ifdef _AIX
 // AIX allows over-allocation, and will SIGKILL when the allocated pages are
diff --git a/test/addons/worker-addon/binding.cc b/test/addons/worker-addon/binding.cc
index d5a07ff20959e8..5c443e9af1caae 100644
--- a/test/addons/worker-addon/binding.cc
+++ b/test/addons/worker-addon/binding.cc
@@ -1,9 +1,9 @@
-#include <assert.h>
 #include <node.h>
-#include <stdio.h>
-#include <stdlib.h>
 #include <v8.h>
 #include <uv.h>
+#include <assert.h>
+#include <stdio.h>
+#include <stdlib.h>
 
 using v8::Context;
 using v8::Local;
diff --git a/test/addons/zlib-binding/binding.cc b/test/addons/zlib-binding/binding.cc
index 53d25642f0d563..3a958f26af1ac6 100644
--- a/test/addons/zlib-binding/binding.cc
+++ b/test/addons/zlib-binding/binding.cc
@@ -1,7 +1,7 @@
 #include <node.h>
 #include <node_buffer.h>
-#include <assert.h>
 #include <zlib.h>
+#include <assert.h>
 
 namespace {