Author jeff.allen
Recipients jeff.allen, zyasoft
Date 2018-04-08.08:31:23
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <>
In fact we do something similar when we encounter a non-public class here:

However, the role that "public" plays here needs to be replaced by "accessible", in the sense of modular accessibility from the unnamed module. I wonder if that can be done without invoking Java 9 specifics?

There's a bug in that if-statement too. The name field for core classes, has already been changed from the Java class name at:
and so the prefix tested is no longer there, which means we perfrom this unnecessarily for core classes. That should be fixed, but for now it gives me lots of opportunity to see it in action :/

Also, the logic of this seems odd:
We check to see if the class is non-public, after the merge?
Date User Action Args
2018-04-08 08:31:24jeff.allensetmessageid: <>
2018-04-08 08:31:24jeff.allensetrecipients: + jeff.allen, zyasoft
2018-04-08 08:31:24jeff.allenlinkissue2662 messages
2018-04-08 08:31:23jeff.allencreate