-
Notifications
You must be signed in to change notification settings - Fork 255
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ghidra addHighlightBrace has been renamed to addBraceHighlight in master #81
Comments
Hello Alexandre, Big thank you for the heads-up here. The highlighting code in the Ghidra plugin is really shaky right now. I need to dig more into it but a stable highlighter service would be great, it was missing so far with many classes not exposed. I'll keep the issue open until the 10.2 is released and I update ret-sync code accordingly. Thanks. |
Hi, Ideas for a workaround ? |
not really an issue atm but more a warning, addHighlightBrace in ghidra master / 10.2 has been rename to addBracehighlight here :
https://github.com/NationalSecurityAgency/ghidra/blob/bed44bd55dc88c7c6fab5f158cdb4b9daa087519/Ghidra/Features/Decompiler/src/main/java/ghidra/app/decompiler/component/ClangHighlightController.java#L610
compilation of the plugin against master fails here :
ret-sync/ext_ghidra/src/main/java/retsync/LocalColorizerService.java
Line 302 in 7602ddd
The text was updated successfully, but these errors were encountered: