search

pulseenergy / vacation

Handy little app to help you keep track of your accumulated and remaining vacation days.
GNU General Public License v2.0
5 stars 2 forks source link

Don't crash if user tries to take stat holiday off #3

Closed jollyra closed 8 years ago

jollyra commented 8 years ago

I tried to "vacation dec 24,25" and it crashing. I think because 25 is a stat holiday.

supermitch commented 8 years ago

Can't reproduce this. Taking stat holidays should make absolutely no difference (to that part of the code, anyway.)

What is broken is Dec 24,25 (without a space after the comma). Is that what broken during the demo? That's a lexer error.

supermitch commented 8 years ago

Gonna close this issue since @jollyra won't test it for me.