racket / zuo

A tiny Racket for scripting
Other
263 stars 27 forks source link

Fix problem in call/prompt #15

Closed soegaard closed 11 months ago

soegaard commented 11 months ago

The loop invariant is that count equals length of args, but here args were updated without updating count.

soegaard commented 11 months ago

Added a little repair for the description of module paths. [It should have been a separate pull request, but I forgot to make a separate branch.]

soegaard commented 11 months ago

Btw - the source has:

static int zuo_is_symbol_module_char(int c) {
  return (c != 0) && (isalpha(c) || isdigit(c) || strchr("-_+/", c));
}

But the documentation doesn't mention underscore. Is that intentional?

mflatt commented 11 months ago

No, underscore should be mentioned, too.

Thanks for these repairs! Will merge soon, but maybe you'll want to add one more fix for underscore.

soegaard commented 11 months ago

Done.