fix: wire up GUI browse buttons, make text boxes editable, pause CLI on error, register codepage provider

GUI:
- Attach Click event handlers to browse input/output buttons (were defined but never wired up)
- Remove ReadOnly=true from input/output text boxes so users can type or copy paths

CLI:
- Make Cli.Run return bool to signal errors
- Only pause with Console.ReadLine() when an error occurs (not on success)
- Fix default case to return true

Core:
- Add System.Text.Encoding.CodePages package for CP1252 SRT file support
- Call Encoding.RegisterProvider(CodePagesEncodingProvider.Instance) before reading files
This commit is contained in:
Kareem Horstink
2026-06-25 16:46:05 +00:00
parent 40ce11d068
commit 9b7b8df874
6 changed files with 26 additions and 7 deletions
+6 -4
View File
@@ -4,19 +4,19 @@ namespace ConvertSRTto3DASS;
internal static class Cli
{
public static void Run(string[] args)
public static bool Run(string[] args)
{
if (args.Length == 0 || args[0] is "--help" or "-h" or "/?")
{
PrintUsage();
return;
return false;
}
var inputPath = args[0];
if (!File.Exists(inputPath))
{
Console.Error.WriteLine($"Input file not found: {inputPath}");
return;
return true;
}
// Parse options
@@ -80,7 +80,7 @@ internal static class Cli
break;
default:
Console.Error.WriteLine($"Unknown argument: {args[i]}");
return;
return true;
}
i++;
}
@@ -106,11 +106,13 @@ internal static class Cli
var result = SrtConverter.Convert(options);
Console.WriteLine($"Parsed: {result.SubtitleCount} subtitle blocks");
Console.WriteLine("Conversion complete.");
return false;
}
catch (Exception ex)
{
Console.Error.WriteLine($"ERROR: {ex.Message}");
Console.Error.WriteLine(ex.ToString());
return true;
}
string GetNextArg(int index) => index + 1 < args.Length ? args[index + 1] : throw new ArgumentException($"Missing value for {args[index]}");