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

[Feature] provide option to add arguments to constructor and yylex [sf#21] #156

Closed
lsf37 opened this Issue Feb 15, 2015 · 7 comments

Comments

Projects
None yet
2 participants
@lsf37
Member

lsf37 commented Feb 15, 2015

Reported by jafl on 2007-07-28 00:15 UTC
It would be nice to have something like %ctorarg and %yylexarg to specify additional arguments to the constructors (both of them, of course) and to yylex().

@lsf37 lsf37 changed the title from provide option to add arguments to constructor and yylex to [Feature] provide option to add arguments to constructor and yylex [sf#21] Feb 15, 2015

@lsf37 lsf37 closed this Feb 15, 2015

@lsf37

This comment has been minimized.

Show comment
Hide comment
@lsf37

lsf37 Feb 15, 2015

Member

Commented by jafl on 2007-08-03 22:51 UTC
Patch to LexScan.flex for ctor args

Member

lsf37 commented Feb 15, 2015

Commented by jafl on 2007-08-03 22:51 UTC
Patch to LexScan.flex for ctor args

@lsf37

This comment has been minimized.

Show comment
Hide comment
@lsf37

lsf37 Feb 15, 2015

Member

Commented by jafl on 2007-08-03 22:51 UTC
Logged In: YES
user_id=1439262
Originator: YES

File Added: LexScan.diff

Member

lsf37 commented Feb 15, 2015

Commented by jafl on 2007-08-03 22:51 UTC
Logged In: YES
user_id=1439262
Originator: YES

File Added: LexScan.diff

@lsf37

This comment has been minimized.

Show comment
Hide comment
@lsf37

lsf37 Feb 15, 2015

Member

Commented by jafl on 2007-08-03 22:51 UTC
Patch to Emitter.java for ctor args

Member

lsf37 commented Feb 15, 2015

Commented by jafl on 2007-08-03 22:51 UTC
Patch to Emitter.java for ctor args

@lsf37

This comment has been minimized.

Show comment
Hide comment
@lsf37

lsf37 Feb 15, 2015

Member

Commented by jafl on 2007-08-03 22:51 UTC
Logged In: YES
user_id=1439262
Originator: YES

File Added: Emitter.diff

Member

lsf37 commented Feb 15, 2015

Commented by jafl on 2007-08-03 22:51 UTC
Logged In: YES
user_id=1439262
Originator: YES

File Added: Emitter.diff

@lsf37

This comment has been minimized.

Show comment
Hide comment
@lsf37

lsf37 Feb 15, 2015

Member

Commented by decamps on 2007-11-02 11:47 UTC
Logged In: YES
user_id=698517
Originator: NO

I have commited this patch in branches/jflex-java5, revisions 250, 251 & 252

Member

lsf37 commented Feb 15, 2015

Commented by decamps on 2007-11-02 11:47 UTC
Logged In: YES
user_id=698517
Originator: NO

I have commited this patch in branches/jflex-java5, revisions 250, 251 & 252

@lsf37

This comment has been minimized.

Show comment
Hide comment
@lsf37

lsf37 Feb 15, 2015

Member

Commented by lsf37 on 2008-05-26 12:10 UTC
Logged In: YES
user_id=93534
Originator: NO

I have ported this patch now also to the trunk version of
JFlex and Java 1.1 in revision 339.

The option conflicts with %debug and %standalone. In these cases,
a warning and additional default constructors will be emitted
(without parameters or init code).

Member

lsf37 commented Feb 15, 2015

Commented by lsf37 on 2008-05-26 12:10 UTC
Logged In: YES
user_id=93534
Originator: NO

I have ported this patch now also to the trunk version of
JFlex and Java 1.1 in revision 339.

The option conflicts with %debug and %standalone. In these cases,
a warning and additional default constructors will be emitted
(without parameters or init code).

@lsf37

This comment has been minimized.

Show comment
Hide comment
@lsf37

lsf37 Feb 15, 2015

Member

Updated by lsf37 on 2008-05-27 11:47 UTC

  • status: open --> closed
Member

lsf37 commented Feb 15, 2015

Updated by lsf37 on 2008-05-27 11:47 UTC

  • status: open --> closed

@regisd regisd added the enhancement label Nov 4, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment