expreduce icon indicating copy to clipboard operation
expreduce copied to clipboard

Production cases should use symbolic constants.

Open cznic opened this issue 8 years ago • 2 comments
trafficstars

@corywalker yy now supports symbolic names for productions case numbers. I've addedd some "easy" names to wl at https://github.com/cznic/wl/commit/3af89c3e68a786cb9d126f44df52a20a9403941e.

IIRC, we've discussed this option earlier and it's finally available. Please test if it still works for you and let me know if I've accidentaly broken something. If it's ok, you might want to replace the hard-coded case numbers in expreduce with their names. Please feel free to add other names to WL productions as needed, thank you.

cznic avatar Oct 23 '17 13:10 cznic

I converted about 50% of the cases to use the new symbolic constants in 40ceb5e0aaff7758dd9baa292c27f5265e475755. I'm not able to add the other constant names into wl at this time, but going forward we'll include the names. And if the case numbers ever shift considerably, I'll take that time to update the rest of them.

corywalker avatar Nov 02 '17 05:11 corywalker

Thanks for the change! This is very helpful for the parser.

corywalker avatar Nov 02 '17 05:11 corywalker