more

Author: pavelsavara

src/coreclr/CMakeLists.txt

@@ -69,9 +69,9 @@ if(NOT CLR_CROSS_COMPONENTS_BUILD AND NOT CLR_CMAKE_TARGET_MACCATALYST AND NOT C
   add_dependencies(runtime singlefilehost)
 else()
     if(CLR_CMAKE_TARGET_BROWSER)
-      # TODOWASM set GEN_PINVOKE to 1 once we generate them. static entrypoint.c prevents C linker from trimming, when IL trimming trims PInvokes
+      # WASM-TODO set GEN_PINVOKE to 1 once we generate them. static entrypoint.c prevents C linker from trimming, when IL trimming trims PInvokes
       set(GEN_PINVOKE 0)
-      # TODOWASM should we start using System.Native.Browser and System.Runtime.InteropServices.JavaScript.Native also for Mono ?
+      # WASM-TODO should we start using System.Native.Browser and System.Runtime.InteropServices.JavaScript.Native also for Mono ?
       set(BUILD_LIBS_NATIVE_BROWSER 1)
       add_subdirectory(${CLR_SRC_NATIVE_DIR}/corehost browserhost)
       add_dependencies(runtime browserhost)

src/native/corehost/browserhost/CMakeLists.txt

@@ -78,7 +78,7 @@ target_link_libraries(
     ${END_WHOLE_ARCHIVE}
 )
 
-# TODOWASM: this is temporary locations
+# WASM-TODO: this is temporary locations
 set(SAMPLE_ASSETS
     sample/index.html
     sample/main.mjs

src/native/corehost/browserhost/browserhost.cpp

@@ -108,7 +108,7 @@ static pal::char_t buffer[STRING_LENGTH("0xffffffffffffffff")];
 
 extern "C" int browserHostInitializeCoreCLR(void)
 {
-    //TODOWASM: does getenv return UTF8 ?
+    //WASM-TODO: does getenv return UTF8 ?
     pal::getenv(HOST_PROPERTY_APP_PATHS, &app_path);
     pal::getenv(HOST_PROPERTY_NATIVE_DLL_SEARCH_DIRECTORIES, &search_paths);
     pal::getenv(HOST_PROPERTY_TRUSTED_PLATFORM_ASSEMBLIES, &tpa);
@@ -142,7 +142,7 @@ extern "C" int browserHostInitializeCoreCLR(void)
     return 0;
 }
 
-// TODOWASM: browser needs async entrypoint
+// WASM-TODO: browser needs async entrypoint
 extern "C" int browserHostExecuteAssembly(const char* assemblyPath)
 {
     int exit_code;
@@ -162,11 +162,11 @@ extern "C" int browserHostExecuteAssembly(const char* assemblyPath)
     {
         std::fprintf(stderr, "coreclr_shutdown_2 failed - Error: 0x%08x\n", retval);
         exit_code = -1;
-        // TODOWASM: this is too trivial
+        // WASM-TODO: this is too trivial
         browserHostRejectMain("coreclr_shutdown_2 failed");
     }
 
-    // TODOWASM: this is too trivial
+    // WASM-TODO: this is too trivial
     // because nothing runs continuations yet and also coreclr_execute_assembly is sync looping
     browserHostResolveMain(exit_code);
     return retval;

src/native/corehost/browserhost/loader/bootstrap.ts

@@ -18,11 +18,11 @@ const nativeModulePromiseController = createPromiseController<EmscriptenModuleIn
     updateInternals();
 });
 
-// TODOWASM: retry logic
-// TODOWASM: throttling logic
-// TODOWASM: load icu data
-// TODOWASM: invokeLibraryInitializers
-// TODOWASM: webCIL
+// WASM-TODO: retry logic
+// WASM-TODO: throttling logic
+// WASM-TODO: load icu data
+// WASM-TODO: invokeLibraryInitializers
+// WASM-TODO: webCIL
 
 export async function createRuntime(downloadOnly: boolean, loadBootResource?: LoadBootResourceCallback): Promise<any> {
     const config = getConfig();

src/native/corehost/browserhost/sample/dotnet.boot.js

@@ -1,7 +1,7 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
-// TODOWASM: this is dummy configuration until WASM SDK for corehost
+// WASM-TODO: this is dummy configuration until WASM SDK for corehost
 
 export const config = /*json-start*/{
   "mainAssemblyName": "HelloWorld.dll",

src/native/libs/System.Native.Browser/CMakeLists.txt

@@ -42,11 +42,11 @@ set(ROLLUP_OUTPUTS
     "${CLR_ARTIFACTS_OBJ_DIR}/native/browser-${CMAKE_BUILD_TYPE}-wasm/System.Runtime.InteropServices.JavaScript.Native/libSystem.Runtime.InteropServices.JavaScript.Native.js.map"
 )
 if(NOT PRODUCT_VERSION_JS)
-    # TODOWASM get dotnet ProductVersion from MSBuild
+    # WASM-TODO get dotnet ProductVersion from MSBuild
     set(PRODUCT_VERSION_JS "10.0.0-dev")
 endif()
 if(NOT CI_BUILD_JS)
-    # TODOWASM get ContinuousIntegrationBuild from MSBuild
+    # WASM-TODO get ContinuousIntegrationBuild from MSBuild
     set(CI_BUILD_JS "false")
 endif()