Skip to content

Commit 3637d31

Browse files
committed
Fix property initialization order in CodeRunnerViewModel
Ensure 'Extension' is initialized before 'SelectedCompiler' to avoid null reference and incorrect behavior.
1 parent c04b54d commit 3637d31

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/CodeSnip/Views/CodeRunnerView/CodeRunnerViewModel.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,12 +66,12 @@ public partial class CodeRunnerViewModel : ObservableObject
6666

6767
public CodeRunnerViewModel(string languageExtension, string code, Func<string> getLatestCode)
6868
{
69+
Extension = languageExtension;// set before triggering OnSelectedCompilerChanged beacause it uses it
6970
Compilers = _compilersSettings.GetCompilersByExtension(languageExtension);
7071
var defaultCompilerId = _compilersSettings.GetDefaultCompilerIdByExtension(languageExtension);
7172
SelectedCompiler = Compilers.FirstOrDefault(c => c.Id == defaultCompilerId) ?? Compilers.FirstOrDefault();
7273
Code = code;
7374
_getLatestCode = getLatestCode;
74-
Extension = languageExtension;
7575
_godboltService = new GodboltService(_httpClient);
7676

7777
}
@@ -88,7 +88,7 @@ private static string MapLanguageExtensionToAsmHighlighting(string languageExten
8888
return languageExtension.ToLowerInvariant() switch
8989
{
9090

91-
"java" => "java-bytecode",
91+
"java" => "javaopc",
9292
_ => "asm", // Default for C++, Rust, D, etc.
9393
};
9494
}

0 commit comments

Comments
 (0)