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

since 1.3.22 setColor(java.awt.Color.SOMECOLOR) does not work anymore in jpms program #439

Closed
chrismaster opened this issue Oct 27, 2020 · 6 comments · Fixed by #440
Closed
Labels

Comments

@chrismaster
Copy link
Contributor

in 1.3.20
fontCell.setColor(Color.BLACK) works
in 1.3.21 and 1.3.22 I get
error: cannot access Color
fontCell.setColor(BLACK);
^
class file for java.awt.Color not found

in module-info.java is
requires java.desktop;

without module-info.java it compiles....

@chrismaster
Copy link
Contributor Author

fontCell.setColor(0,0,0);
compiles too

@chrismaster
Copy link
Contributor Author

Color BLACK = Color.BLACK;
compiles too, so java.awt.Color is accessible
but if i add
fontCell.setColor(BLACK);
it gives
error: cannot access Color
fontCell.setColor(BLACK);
^
class file for java.awt.Color not found

@asturio
Copy link
Member

asturio commented Oct 27, 2020

Thanks. Pull Requests welcome!

@chrismaster
Copy link
Contributor Author

Sorry, got no real java skills. And my git skills are 0.
I cloned openpdf and added to
modules-info.java
requires java.desktop;

with this new openpdf-1.3.23-SNAPSHOT.jar
fontCell.setColor(java.awt.Color.SOMECOLOR) gives no error

in the module java.desktop is java.awt.Color.*
requires static java.desktop; would also do the job.....

@chrismaster
Copy link
Contributor Author

#440

@asturio
Copy link
Member

asturio commented Oct 27, 2020

I'll merge your PR soon.

@asturio asturio linked a pull request Oct 27, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants