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

[Bug] zzEOFDone initialized to self [sf#113] #114

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

Comments

Projects
None yet
2 participants
@lsf37
Member

lsf37 commented Feb 15, 2015

*Reported by anonymous on 2010-12-15 11:57 UTC
In generated code, I see:

/** sets all values stored in this class */
ZzFlexStreamInfo(java.io.Reader zzReader, int zzEndRead, int zzStartRead,
int zzCurrentPos, int zzMarkedPos,
char [] zzBuffer, boolean zzAtEOF, int yyline, int yycolumn) {
this.zzReader = zzReader;
this.zzEndRead = zzEndRead;
this.zzStartRead = zzStartRead;
this.zzCurrentPos = zzCurrentPos;
this.zzMarkedPos = zzMarkedPos;
this.zzBuffer = zzBuffer;
this.zzAtEOF = zzAtEOF;
this.zzEOFDone = zzEOFDone;
this.yyline = yyline;
this.yycolumn = yycolumn;
}

note that zzEOFDone is not passed as an argument to the method, so it is apparently assigned to itself.

Found using FindBugs.

@lsf37 lsf37 changed the title from zzEOFDone initialized to self to [Bug] zzEOFDone initialized to self [sf#113] Feb 15, 2015

@lsf37 lsf37 added this to the feature request milestone 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 steve_rowe on 2013-12-23 05:32 UTC
I committed a fix in r727.

Member

lsf37 commented Feb 15, 2015

Commented by steve_rowe on 2013-12-23 05:32 UTC
I committed a fix in r727.

@lsf37

This comment has been minimized.

Show comment
Hide comment
@lsf37

lsf37 Feb 15, 2015

Member

Updated by lsf37 on 2014-01-16 06:48 UTC

  • status: open --> open-fixed
  • Group: --> feature request
Member

lsf37 commented Feb 15, 2015

Updated by lsf37 on 2014-01-16 06:48 UTC

  • status: open --> open-fixed
  • Group: --> feature request
@lsf37

This comment has been minimized.

Show comment
Hide comment
@lsf37

lsf37 Feb 15, 2015

Member

Updated by lsf37 on 2014-03-01 00:26 UTC

  • status: open-fixed --> closed
Member

lsf37 commented Feb 15, 2015

Updated by lsf37 on 2014-03-01 00:26 UTC

  • status: open-fixed --> closed

@lsf37 lsf37 added the bug label Feb 17, 2015

@lsf37 lsf37 removed this from the feature request milestone Feb 17, 2015

@regisd regisd added this to the 1.5.0 milestone Nov 4, 2017

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