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

PR-URL: #35719
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Richard Lau <[email protected]>

Author: Trott

Diff for: 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 for: 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 for: 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 for: 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 for: 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 for: 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 for: 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 for: 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 for: 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 {