Skip to content
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

Improve error message on method not found #558

Merged
merged 1 commit into from
Oct 20, 2020

Conversation

cmacdonald
Copy link
Contributor

This addresses a suggestion in #538

Example before PR:

>>> Float = autoclass("java.lang.Float")
>>> Float.floatValue()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "jnius/jnius_export_class.pxi", line 1142, in jnius.JavaMultipleMethod.__call__
jnius.JavaException: No methods matching your arguments, requested: (), available: []

Example after PR:

>>> Float = autoclass("java.lang.Float")
>>> Float.floatValue()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "jnius/jnius_export_class.pxi", line 1142, in jnius.JavaMultipleMethod.__call__
jnius.JavaException: No static methods called floatValue in java/lang/Float matching your arguments, requested: (), available: []

@cmacdonald
Copy link
Contributor Author

I think that github actions are broken. I wonder if #541 works?

@obfusk
Copy link
Contributor

obfusk commented Oct 20, 2020

I think that github actions are broken.

Looks like the CI is configured to run on push but not for PRs.

@cmacdonald
Copy link
Contributor Author

I think that github actions are broken.

Looks like the CI is configured to run on push but not for PRs.

But it runs on my fork instead, and failed for x86:
https://github.com/cmacdonald/pyjnius/runs/1282554361

Craig

@obfusk
Copy link
Contributor

obfusk commented Oct 20, 2020

See #559.

@tshirtman tshirtman merged commit a89929c into kivy:master Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants