Skip to content

Conversation

@ktgw0316
Copy link

@ktgw0316 ktgw0316 commented Oct 18, 2024

There is an error in GraphicsJAI when one attempts to create a GraphicsJAI from an existing Graphics2D. If the GraphicsJAI goes out of scope it can ruin the original Graphics2D and causes some method invokations, such as g2d.fill(shape), to be ignored.

This bug was reported as JAI_CORE-96.

@ktgw0316 ktgw0316 marked this pull request as ready for review October 18, 2024 06:34
@jodygarnett
Copy link
Contributor

Hello @ktgw0316 thanks for the PR; we are just in the process of restarting this project (it was in a holding pattern pending funding).

I will be working with @aaime and others to review outstanding PRs before we start work on a 1.0 release.

@jodygarnett
Copy link
Contributor

jodygarnett commented May 9, 2025

The report from eclipsefdn/eca seems to be triggered by using different emails in the commit messages:

Author(s) covered by necessary legal agreements
arctica0316@gmil DOT com
jody.garnett@gm
il DOT com

Author Eclipse account cannot be found.
Email: arctica0316@gm*il DOT com
GitHub ID: ktgw0316

Since it the email is listed as both "covered" and "not found" we may need to ask for help.

@ktgw0316 have you signed the https://www.eclipse.org/legal/eca/ thing? Does your profile have the [email protected] email address listed?

@jodygarnett
Copy link
Contributor

@pstankie you are listed as a contributor on the eclipsefdn repo. We have an issue with the eclipsefdn/eca bot both accepting and rejecting the email address "arctica0316@gmil DOT com".

@ktgw0316
Copy link
Author

ktgw0316 commented May 10, 2025

@jodygarnett

@ktgw0316 have you signed the https://www.eclipse.org/legal/eca/ thing? Does your profile have the [email protected] email address listed?

Yes, but I haven't connected my GitHub account to the Eclipse account. I've just connected it.

@ktgw0316 ktgw0316 force-pushed the java_net_JIRA/96 branch from 7a4850a to 85095f4 Compare May 10, 2025 04:11
@jodygarnett
Copy link
Contributor

Thanks @ktgw0316 that resolved the check!

Copy link
Contributor

@jodygarnett jodygarnett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revisit this change after JAI-Ext merged, check if still required.

@jodygarnett
Copy link
Contributor

JAI-Ext merged, please reopen if this problem still remains in 0.9.1 release.

@jodygarnett jodygarnett closed this Jan 6, 2026
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 this pull request may close these issues.

2 participants