Skip to content

Commit

Permalink
Fix build warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
TheJoeFin committed Jun 1, 2024
1 parent f5c6ea5 commit 70c7c54
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Text-Grab/Pages/GeneralSettings.xaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
x:Name="VersionTextblock"
VerticalAlignment="Center"
Style="{StaticResource TextBodyNormal}"
Text="Version 4.X.X WPFUI Tray" />
Text="Version 4.5-beta1" />

<ui:HyperlinkButton
x:Name="OpenExeFolderButton"
Expand Down
3 changes: 2 additions & 1 deletion Text-Grab/Pages/LanguageSettings.xaml.cs
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,8 @@ public async Task CopyFileWithElevatedPermissions(string sourcePath, string dest
Process? process = Process.Start(startInfo);
// string errors = process?.StandardError.ReadToEnd();
// string output = process?.StandardOutput.ReadToEnd();
await process?.WaitForExitAsync();
if (process is not null)
await process.WaitForExitAsync();

// if (!string.IsNullOrEmpty(errors))
// ErrorsAndOutputText.Text += Environment.NewLine + errors;
Expand Down
2 changes: 1 addition & 1 deletion Text-Grab/Services/SettingsService.cs
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ private void MigrateLocalSettingsToClassic()
foreach (KeyValuePair<string, object> localSetting in _localSettings.Values)
{
try { ClassicSettings[localSetting.Key] = localSetting.Value; }
catch (SettingsPropertyNotFoundException ex) { Debug.WriteLine($"Failed to migrate {localSetting.Key} of value {localSetting.Value}"); } // continue, just skip the setting
catch (SettingsPropertyNotFoundException ex) { Debug.WriteLine($"Failed to migrate {localSetting.Key} of value {localSetting.Value}, exception message: {ex.Message}"); } // continue, just skip the setting
catch (Exception ex)
{
Debug.WriteLine($"Failed to migrate setting {localSetting.Key} from ApplicationDataContainer {ex.Message}");
Expand Down

0 comments on commit 70c7c54

Please sign in to comment.