Gerwin Klein

**Commented by jafl on 2007-07-30 20:28 UTC** Skeleton file with state stack functionality

**Commented by jafl on 2007-07-30 20:28 UTC** Logged In: YES user_id=1439262 Originator: YES I have the state stack working. File Added: skeleton.state_stack

**Commented by lsf37 on 2007-08-01 22:30 UTC** Logged In: YES user_id=93534 Originator: NO Thanks, you're right, of course.

**Updated by lsf37 on 2007-08-01 22:30 UTC** - **priority**: 5 --> 3

**Commented by jafl on 2007-08-03 22:48 UTC** Logged In: YES user_id=1439262 Originator: YES My suggestion would be to include the state stack in the default skeleton. The patch is very...

> IIRC, this can only be done with code generated with 1.8.0, hence can only be done in 1.9.0 Makes sense, let's leave this here until 1.8.0 is out.

This seems to be passing even though bazel is still building with 1.7.0. Is this Ok to merge?

No worries. Are the errors about fall-through or other jflex 1.8.0 things?

Thanks for reporting that, it looks like one of the refactors has forgotten to add a null pointer check.

They're equivalent, why change them?