focus-editor / focus

A simple and fast text editor
GNU General Public License v3.0
1.81k stars 99 forks source link

[BUG] Choosing theme from dialog causes a crash #420

Closed maxamundsen closed 2 months ago

maxamundsen commented 2 months ago

Describe the bug As the title says. You cannot change themes via the theme dialog. You can still change themes by editing the config manually.

System Info

Stack Trace

Ignoring "watch_recursively = false" flag since macOS always watches directories recursively.
Started watching external directory '/Users/mta/Library/Application Support/dev.focus-editor'
/Users/mta/projects/focus/src/config_parser.jai:49,24: Assertion failed: parse_config must use a pool or a temp allocator

Stack trace:
/Users/mta/projects/focus/modules/Runtime_Support.jai:103: runtime_support_assertion_failed
/opt/jai/modules/Basic/module.jai:87: assert_helper
/Users/mta/projects/focus/src/config_parser.jai:49: parse_config
/Users/mta/projects/focus/src/widgets/theme_dialog.jai:59: apply_selected_theme
/Users/mta/projects/focus/src/widgets/generic_choice_dialog.jai:8: dialog_handle_event
/Users/mta/projects/focus/src/widgets/theme_dialog.jai:2: theme_dialog_handle_event
/Users/mta/projects/focus/src/main.jai:296: main
focus-editor commented 2 months ago

Should be fixed in the main branch - please reopen if not. Thanks for the report.