rg_system: Replaced RG_BUILD_TYPE with RG_BUILD_RELEASE

Release or not is all we care about. Any more nuance is controlled by other flags.

Author: ducalex

components/retro-go/CMakeLists.txt

@@ -46,8 +46,8 @@ if(RG_BUILD_VERSION)
     component_compile_options(-DRG_BUILD_VERSION="${RG_BUILD_VERSION}")
 endif()
 
-if(RG_BUILD_TYPE)
-    component_compile_options(-DRG_BUILD_TYPE=${RG_BUILD_TYPE})
+if(RG_BUILD_RELEASE)
+    component_compile_options(-DRG_BUILD_RELEASE=1)
 endif()
 
 set(RG_TARGET "RG_TARGET_${RG_BUILD_TARGET}")

components/retro-go/rg_system.c

@@ -328,7 +328,7 @@ rg_app_t *rg_system_init(int sampleRate, const rg_handlers_t *handlers, const rg
         .tickTimeout = 3000000,
         .watchdog = true,
         .lowMemoryMode = false,
-    #if RG_BUILD_TYPE == 1
+    #if RG_BUILD_RELEASE
         .isRelease = true,
         .logLevel = RG_LOG_INFO,
     #else

components/retro-go/rg_system.h

@@ -288,11 +288,12 @@ float rg_emu_get_speed(void);
 #define RG_LOGE(x, ...) rg_system_log(RG_LOG_ERROR, RG_LOG_TAG, x, ## __VA_ARGS__)
 #define RG_LOGW(x, ...) rg_system_log(RG_LOG_WARN, RG_LOG_TAG, x, ## __VA_ARGS__)
 #define RG_LOGI(x, ...) rg_system_log(RG_LOG_INFO, RG_LOG_TAG, x, ## __VA_ARGS__)
+#if RG_BUILD_RELEASE
+#define RG_LOGD(x, ...)
+#define RG_LOGV(x, ...)
+#else
 #define RG_LOGD(x, ...) rg_system_log(RG_LOG_DEBUG, RG_LOG_TAG, x, ## __VA_ARGS__)
-#if RG_BUILD_TYPE != 1
 #define RG_LOGV(x, ...) rg_system_log(RG_LOG_VERBOSE, RG_LOG_TAG, x, ## __VA_ARGS__)
-#else
-#define RG_LOGV(x, ...)
 #endif
 
 #ifdef RG_ENABLE_PROFILING

rg_tool.py

@@ -205,7 +205,7 @@ def build_app(app, device_type, with_profiling=False, no_networking=False, is_re
     args = ["idf.py", "app"]
     args.append(f"-DRG_BUILD_VERSION={PROJECT_VER}")
     args.append(f"-DRG_BUILD_TARGET={re.sub(r'[^A-Z0-9]', '_', device_type.upper())}")
-    args.append(f"-DRG_BUILD_TYPE={1 if is_release else 0}")
+    args.append(f"-DRG_BUILD_RELEASE={1 if is_release else 0}")
     args.append(f"-DRG_ENABLE_PROFILING={1 if with_profiling else 0}")
     args.append(f"-DRG_ENABLE_NETWORKING={0 if no_networking else 1}")
     run(args, cwd=os.path.join(os.getcwd(), app))