expreduce
expreduce copied to clipboard
Production cases should use symbolic constants.
@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.
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.
Thanks for the change! This is very helpful for the parser.