New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show the line number and the actual code for dangling gref/arity invalid error if possible. #288

Merged
merged 1 commit into from Jun 9, 2017

Conversation

Projects
None yet
2 participants
@nkoguro
Contributor

nkoguro commented Jun 9, 2017

This pull request changes the error message for dangling grefs and arity invalid like this.

testing bindings in script ./foo.scm ... ERROR: symbols referenced but not defined: bar(316:(u8vector-copy! xxx yyy bar))

The suspicious code detection logic in closure-grefs can mistake. But I think it is better than the current error message (just showing closure itself) in most cases.

@shirok

This comment has been minimized.

Owner

shirok commented Jun 9, 2017

Thanks. The only thing is that we don't want gauche.test to depend on util.match, for we use gauche.test to test util.match. But I'll fix it later.

@shirok shirok merged commit 23421a7 into shirok:master Jun 9, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment