On Thu, 28 Jul 2022 14:45:36 GMT, Tejesh R <t...@openjdk.org> wrote: > > I guess exposing no-args public constructor was wrong and it should have > > been protected from beginning. Also, please add a testcase. > > Yeah, its been used internally by other constructors after setting the > `graphics` instances. Exposing it as public causes user to create it without > `graphics` been set. I tried modifying it to protected and ran the test, but > some html test fails.
Even if it hadn't failed the test, you cannot simply change a method, constructor, or field in a public (or protected) class in an exported package from public to protected, since that would be an incompatible API change (at least not without a very compelling reason and a lot of discussion, and even then, probably not). ------------- PR: https://git.openjdk.org/jdk/pull/9673
