jkotlinski / durexforth

Modern C64 Forth
Other
230 stars 28 forks source link

Added POSTPONE error handling. Drive-by code simplifications. #446

Closed jkotlinski closed 2 years ago

lonetech commented 2 years ago

Looks like a move of rot from base.fs to core.asm was mixed in. That code change also looks okay, though it wasn't mentioned. I suppose it will build faster, but there's no asm calling it, no label to do so, and base.fs still depends on asm words. The only difference at run time will be the order of words.

jkotlinski commented 2 years ago

The move of rot was so it could be used in postpone!

lonetech commented 2 years ago

Thank you for the rot clarification, and pardon my confusion. I didn't spot that dependency. And asm in turn depends on does> etc so I guess this was the smaller move. Not a problem it is done, I just didn't see why at the time.