Skip to content

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 <[email protected]>
@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.

3 participants