Skip to content
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

Misidentified Javadocs #2

Closed
fennifith opened this issue Oct 4, 2018 · 0 comments
Closed

Misidentified Javadocs #2

fennifith opened this issue Oct 4, 2018 · 0 comments
Labels
bug Something isn't working

Comments

@fennifith
Copy link
Owner

To be clear, RegEx isn't exactly the problem here, but it is part of it. I've been told that I shouldn't use RegEx for something like this. I don't know if that advice is valid or not, but it remains the easiest option regardless, so I would prefer not to switch from it if possible.

The RegEx statements should be modified to only accept javadocs that aren't escaped with a preceding "" or contained within a string. Currently, the following lines are identified by the regex statement as a javadoc:

System.out.println("/**");
System.out.println("*/");

Guaranteed, this wouldn't normally show up in an application, but it's still something to look out for.

@fennifith fennifith added the bug Something isn't working label Oct 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant