follow-up to 8241294: Examine input checking in ClassLoader::package_from_class_name

[Andrey Petushkov]

Dear Claes, All,

one question related to the subject. Even though it's considered
legitimate for the method to accept '['s at the start of the input
IMHO the check for subsequent 'L' is wrong and the related comment is
misleading.

JLS does not prohibit class and package names to start from capital L,
so it's unclear why arrays of those, and only those (considering these
appear here in internal form) being banned?

Moreover, given that this function is widely used it's even more
important that if there is some internal special case which passes
such forms as argument it's better to be clearly documented, instead
of saying false things like "Fully qualified class names should not
contain a 'L'." (1).

However this this code seem to pass quite number of editorials by
different people and still has the same check. So there might be a
reason. Given that may I ask for kind clarification of the matter?

Thank you,
Andrey

[1] https://hg.openjdk.java.net/jdk/jdk/file/5efafa45f3b8/src/hotspot/share/classfile/classLoader.cpp#l201