binaryninja-api icon indicating copy to clipboard operation
binaryninja-api copied to clipboard

Warn loudly when settings.json isn't parseable

Open psifertex opened this issue 1 year ago • 1 comments

Ran into an issue where an error in my settings.json that was hard to troubleshoot because I didn't realize it was failing. Should either log warn or log error.

psifertex avatar Aug 14 '24 18:08 psifertex

I have a test branch with an ugly hack that implements this. But, after some discussion, we have decided we would prefer to add back previously-removed functionality that catches this class of problem and defers logging until the UI is available.

This is a bigger lift than my quick hack, so we'll need someone with more time/understanding to make this change instead.

fuzyll avatar Aug 14 '24 18:08 fuzyll

Fixed in 4.2.6056-dev.

bpotchik avatar Sep 13 '24 19:09 bpotchik