doc,src,test: revise C++ code for linter update

PR-URL: #35719 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com>
nodejs · Trott · Oct 23, 2020 · Oct 20, 2020 · Oct 20, 2020 · Oct 9, 2020
commit 4fb780334ded9ab805ac1064d5105b2a37734d4d
diff --git a/doc/api/addons.md b/doc/api/addons.md
@@ -271,9 +271,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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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::Function;

diff --git 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 {