src: fixup includes after x509 changes

nodejs · Aug 7, 2024 · dd0f080 · dd0f080
1 parent 907c7a8
commit dd0f080
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 12 deletions.
diff --git a/deps/ncrypto/engine.cc b/deps/ncrypto/engine.cc
@@ -58,7 +58,7 @@ EnginePointer EnginePointer::getEngineByName(const std::string_view name,
       }
     }
   }
-  return std::move(engine);
+  return engine;
 }
 
 bool EnginePointer::setAsDefault(uint32_t flags, CryptoErrorList* errors) {

diff --git a/deps/ncrypto/ncrypto.cc b/deps/ncrypto/ncrypto.cc
@@ -1,13 +1,12 @@
 #include "ncrypto.h"
 #include <algorithm>
 #include <cstring>
-#include "openssl/bn.h"
-#include "openssl/evp.h"
-#include "openssl/pkcs12.h"
-#include "openssl/types.h"
-#include "openssl/x509v3.h"
+#include <openssl/bn.h>
+#include <openssl/evp.h>
+#include <openssl/pkcs12.h>
+#include <openssl/x509v3.h>
 #if OPENSSL_VERSION_MAJOR >= 3
-#include "openssl/provider.h"
+#include <openssl/provider.h>
 #endif
 
 namespace ncrypto {

diff --git a/deps/ncrypto/ncrypto.h b/deps/ncrypto/ncrypto.h
@@ -6,8 +6,7 @@
 #include <optional>
 #include <string>
 #include <string_view>
-#include "openssl/bn.h"
-#include "openssl/types.h"
+#include <openssl/bn.h>
 #include <openssl/x509.h>
 #include <openssl/dh.h>
 #include <openssl/dsa.h>

diff --git a/src/crypto/crypto_x509.cc b/src/crypto/crypto_x509.cc
@@ -9,9 +9,6 @@
 #include "ncrypto.h"
 #include "node_errors.h"
 #include "util-inl.h"
-#include "v8-function-callback.h"
-#include "v8-function.h"
-#include "v8-primitive.h"
 #include "v8.h"
 
 #include <string>
-Original file line number
+Diff line change
@@ Expand Up @@
           }
         }
       }
-      return std::move(engine);
+      return engine;
     }
     bool EnginePointer::setAsDefault(uint32_t flags, CryptoErrorList* errors) {
@@ Expand Down @@