In the example
b = str.indexOf(")");
the editor highlights the closing parentheses within the text quotes. The editor similarly highlights braces and brackets.
Perhaps it would be appropriate to ignore all such opening/closing elements located within quotes?
You are right. I'll fix that.
Just checking in for the First Patch program... I didn't see an implementation plan posted, though I realize sometimes this is done through e-mail between the mentor and developer. I'd appreciate it if you could let me know which milestone you're at according to this page:
and if you're stuck, just let Anuradha (and/or me) know and we'll do our best to help.
Created attachment 112203 [details]
I have tested the proposed patch (sent by e-mail; I will attach a final version later today which leaves out unnecessary changes).
My testing has demonstrated to me that it solves the problem described in this issue and I have not noticed any negative consequences. This change also adds a unit test to the existing MasterMatcherTest and I have found that this test passes 100% of the time on both Linux and Windows XP.
However, I do want to point out that unit tests in the editor.bracesmatching module were already broken before this change. I have filed issue #205033 (Test error/failure in editor.bracesmatching module) about those problems, but since they existed long before this patch, there is no reason to make this issue depend on that one.
Created attachment 113153 [details]
Revised patch which contains changes needed to resolve this issue plus a unit test to prove the solution
I revised Maxim's patch to remove unintentional modifications.
I have tested that the latest patch (attached to this issue) can be applied to the Mercurial repository cleanly, doesn't contain any unnecessary changes and solves the problem identified by this issue.
In my opinion, this patch is ready to be included in NetBeans. I have e-mailed David Strupl from Oracle's NetBeans team to ask which engineer should push this change, and offered to push the change myself if they are not available to do so.
I have never heard from David Strupl, but Svata Dedic (who fixed related issue #205033) told me that this should probably be assigned to Editor/Painting & Printing and that Miloslav Metelka would be the likely person to integrate it. I will change this now.
This old bug may not be relevant anymore. If you can still reproduce it in 8.2 development builds please reopen this issue.
Thanks for your cooperation,
NetBeans IDE 8.2 Release Boss