Skip to content

Patch ImageJMacroTokenMaker to accommodate RSyntaxtTextArea v3.6.0 #72

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

Merged
merged 1 commit into from
Apr 24, 2025

Conversation

tferr
Copy link
Collaborator

@tferr tferr commented Apr 24, 2025

This fixes the issue detailed in scijava/pom-scijava#288

Context:
In newer versions of RSyntaxtTextArea, AbstractJFlexCTokenMaker adds an abstract yystate() method intended to return the current state of a JFlex lexer. Since it is abstract, it must be implemented by any concrete subclass, so we patch ImageJMacroTokenMaker so that the current zzLexicalState is returned

Tested with: mvn clean package -Denforcer.skip

This fixes the issue detailed in
scijava/pom-scijava#288.

Context:
In newer versions of RSyntaxtTextArea, AbstractJFlexCTokenMaker adds an
abstract yystate() method intended to return the current state of a
JFlex lexer. Since it is abstract, it must be implemented by any
concrete subclass, so we patch ImageJMacroTokenMaker so that the current
zzLexicalState is returned.

Tested with: `mvn clean package -Denforcer.skip`

Signed-off-by: Curtis Rueden <ctrueden@wisc.edu>
@ctrueden ctrueden force-pushed the fix-pom-scijava#148 branch from ca392a2 to fdc8abd Compare April 24, 2025 16:27
@ctrueden ctrueden merged commit d5a80a7 into master Apr 24, 2025
1 check passed
@ctrueden ctrueden deleted the fix-pom-scijava#148 branch April 24, 2025 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants