Use __EMSCRIPTEN__ instead of EMSCRIPTEN to wrap wasm-specific code (#200)

Author: ianthomas23

src/utils/config_overrides.cpp

@@ -1,6 +1,6 @@
 #include "config_overrides.hpp"
 
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
 #    include <iterator>
 
 #    include <git2/sys/config.h>
@@ -10,7 +10,7 @@
 
 void apply_config_overrides([[maybe_unused]] git_repository* repo)
 {
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
     // Config values to override, each in "name=value" form. core.filemode is disabled because
     // the emscripten filesystem does support executable file permissions but some JupyterLite
     // drive implementations do not.

src/wasm/libgit2_internals.cpp

@@ -1,4 +1,4 @@
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
 
 #    include "libgit2_internals.hpp"
 
@@ -63,4 +63,4 @@ std::optional<http_service> select_service(git_smart_service_t action)
     return std::nullopt;
 }
 
-#endif  // EMSCRIPTEN
+#endif  // __EMSCRIPTEN__

src/wasm/libgit2_internals.hpp

@@ -1,6 +1,6 @@
 #pragma once
 
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
 
 #    include <optional>
 #    include <string>
@@ -63,4 +63,4 @@ std::string name_for_method(git_http_method method);
 
 std::optional<http_service> select_service(git_smart_service_t action);
 
-#endif  // EMSCRIPTEN
+#endif  // __EMSCRIPTEN__

src/wasm/read_buffer.cpp

@@ -1,4 +1,4 @@
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
 
 #    include "read_buffer.hpp"
 
@@ -9,4 +9,4 @@ read_buffer_t::read_buffer_t(char* buffer, size_t buffer_size, size_t* bytes_rea
 {
 }
 
-#endif  // EMSCRIPTEN
+#endif  // __EMSCRIPTEN__

src/wasm/read_buffer.hpp

@@ -1,6 +1,6 @@
 #pragma once
 
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
 
 #    include <stddef.h>
 
@@ -15,4 +15,4 @@ struct read_buffer_t
     size_t* m_bytes_read;
 };
 
-#endif  // EMSCRIPTEN
+#endif  // __EMSCRIPTEN__

src/wasm/response.cpp

@@ -1,4 +1,4 @@
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
 
 #    include "response.hpp"
 
@@ -82,4 +82,4 @@ void wasm_http_response::set_git_error(std::string_view url) const
     );
 }
 
-#endif  // EMSCRIPTEN
+#endif  // __EMSCRIPTEN__

src/wasm/response.hpp

@@ -1,6 +1,6 @@
 #pragma once
 
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
 
 #    include <map>
 #    include <optional>
@@ -40,4 +40,4 @@ class wasm_http_response
     std::multimap<const std::string, const std::string> m_response_headers;
 };
 
-#endif  // EMSCRIPTEN
+#endif  // __EMSCRIPTEN__

src/wasm/scope.cpp

@@ -1,20 +1,20 @@
 #include "scope.hpp"
 
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
 #    include "transport.hpp"
 #endif
 
 wasm_http_transport_scope::wasm_http_transport_scope()
 {
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
     git_transport_register("http", create_wasm_http_transport, nullptr);
     git_transport_register("https", create_wasm_http_transport, nullptr);
 #endif
 }
 
 wasm_http_transport_scope::~wasm_http_transport_scope()
 {
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
     git_transport_unregister("http");
     git_transport_unregister("https");
 #endif

src/wasm/scope.hpp

@@ -3,7 +3,7 @@
 #include "../utils/common.hpp"
 
 // Scope object to enable/disable browser-based wasm http transport scheme for libgit2.
-// This is a no-op unless EMSCRIPTEN is defined.
+// This is a no-op unless __EMSCRIPTEN__ is defined.
 class wasm_http_transport_scope : noncopyable_nonmovable
 {
 public:

src/wasm/stream.cpp

@@ -1,4 +1,4 @@
-#ifdef EMSCRIPTEN
+#ifdef __EMSCRIPTEN__
 
 #    include "stream.hpp"
 
@@ -654,4 +654,4 @@ int wasm_http_stream_write(git_smart_subtransport_stream* s, const char* buffer,
     return write(stream, buffer, buffer_size);
 }
 
-#endif  // EMSCRIPTEN
+#endif  // __EMSCRIPTEN__