fix: conflicting enum id on iOS

Author: JakubGonera

common/rnexecutorch/Log.cpp

@@ -15,12 +15,12 @@ namespace rnexecutorch {
 #ifdef __ANDROID__
 android_LogPriority androidLogLevel(LOG_LEVEL logLevel) {
   switch (logLevel) {
-  case LOG_LEVEL::INFO:
+  case LOG_LEVEL::Info:
   default:
     return ANDROID_LOG_INFO;
-  case LOG_LEVEL::ERROR:
+  case LOG_LEVEL::Error:
     return ANDROID_LOG_ERROR;
-  case LOG_LEVEL::DEBUG:
+  case LOG_LEVEL::Debug:
     return ANDROID_LOG_DEBUG;
   }
 }
@@ -49,14 +49,14 @@ void log(LOG_LEVEL logLevel, const char *fmt, ...) {
 #ifdef __APPLE__
 
   switch (logLevel) {
-  case LOG_LEVEL::INFO:
+  case LOG_LEVEL::Info:
   default:
     os_log_info(OS_LOG_DEFAULT, "%s", buf);
     break;
-  case LOG_LEVEL::ERROR:
+  case LOG_LEVEL::Error:
     os_log_error(OS_LOG_DEFAULT, "%s", buf);
     break;
-  case LOG_LEVEL::DEBUG:
+  case LOG_LEVEL::Debug:
     os_log_debug(OS_LOG_DEFAULT, "%s", buf);
     break;
   }

common/rnexecutorch/Log.h

@@ -2,7 +2,7 @@
 
 namespace rnexecutorch {
 
-enum class LOG_LEVEL { INFO, ERROR, DEBUG };
+enum class LOG_LEVEL { Info, Error, Debug };
 
 // const char* instead of const std::string& as va_start doesn't take references
 void log(LOG_LEVEL logLevel, const char *fmt, ...);

common/rnexecutorch/RnExecutorchInstaller.cpp

@@ -40,15 +40,10 @@ jsi::Function RnExecutorchInstaller::loadStyleTransfer(
           // version:
           // https://github.com/facebook/react-native/commit/3132cc88dd46f95898a756456bebeeb6c248f20e
           throw jsi::JSError(runtime, e.what());
-          return jsi::Value();
         } catch (const std::exception &e) {
           throw jsi::JSError(runtime, e.what());
-          return jsi::Value();
-          ;
         } catch (...) {
           throw jsi::JSError(runtime, "Unknown error");
-          return jsi::Value();
-          ;
         }
       });
 }