Skip to content

Improved command line argument checking and exception handling for pr…#129

Merged
krperry merged 1 commit into
mainfrom
cmd_crash_fix
Jun 4, 2026
Merged

Improved command line argument checking and exception handling for pr…#129
krperry merged 1 commit into
mainfrom
cmd_crash_fix

Conversation

@krperry
Copy link
Copy Markdown
Collaborator

@krperry krperry commented Jun 4, 2026

…oblems opening files

  • Checks file arguments as early as possible and prints errors to stdout
  • makes output more consistent no matter the reason for the file not opening (missing or unopenable)
  • Catch startup BBNotifyException in Main to prevent no output but an exception.
  • fixed a bug with wp manager that would crash if it was partially initialized and failed.
  • cleaned up exception handling in both wp manager and brialle manager.

Files that are changed:

  • brailleblaster-core/src/main/java/org/brailleblaster/Main.kt
  • brailleblaster-core/src/main/java/org/brailleblaster/perspectives/braille/Manager.java
  • brailleblaster-core/src/main/java/org/brailleblaster/wordprocessor/WPManager.kt

…oblems opening files

- Checks file arguments as early as possible  and prints errors to stdout
- makes output more consistent no matter the reason for the file not opening (missing or unopenable)
- Catch startup BBNotifyException in Main to prevent no output but an exception.
- fixed a bug with wp manager that would crash if  it was partially initialized and failed.
- cleaned up exception handling in both wp manager and brialle manager.

Files that are changed:

- brailleblaster-core/src/main/java/org/brailleblaster/Main.kt
- brailleblaster-core/src/main/java/org/brailleblaster/perspectives/braille/Manager.java
- brailleblaster-core/src/main/java/org/brailleblaster/wordprocessor/WPManager.kt
@krperry krperry merged commit b725a50 into main Jun 4, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant