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

Delete xalan2jdoc.jar #135

Conversation

kriegaex
Copy link
Contributor

Follow-up to #134, because that one was merged without this commit:

Running javadoc generation with or without it on the stylebook classpath had absolutely no effect. Comparing javadocs with and without it revealed that both directory trees were binary identical.

Actually, on JDK 8 the JAR cannot have been working correctly anyway, because the JAR file contained Java 1.1 class files (major 45), which is OK. But it also uses several classes which are no longer available in JDK 8, i.e. the code cannot have been working there. For example, one class imported was

  • com.sun.tools.doclets.IndexBuilder,

which a web search found in JDKs 1.2 and 1.3. In 1.8, there is

  • com.sun.tools.doclets.internal.toolkit.util.IndexBuilder,

which might serve a similar purpose, but neither is public API nor the class referenced by the doclet code. I really think, this is just cruft that can go away.

@jkesselm

Running javadoc generation with or without it on the stylebook classpath
had absolutely no effect. Comparing javadocs with and without it
revealed that both directory trees were binary identical.

Actually, on JDK 8 the JAR cannot have been working correctly anyway,
because the JAR file contained Java 1.1 class files (major 45), which is
OK. But it also uses several classes which are no longer available in
JDK 8, i.e. the code cannot have been working there. For example, one
class imported was
  com.sun.tools.doclets.IndexBuilder,
which a web search found in JDKs 1.2 and 1.3. In 1.8, there is
  com.sun.tools.doclets.internal.toolkit.util.IndexBuilder,
which might serve a similar purpose, but neither is public API nor the
class referenced by the doclet code. I really think, this is just cruft
that can go away.
@jkesselm jkesselm merged commit 39cd970 into apache:xalan-java-mvn-refactored Nov 30, 2023
1 check failed
@kriegaex kriegaex deleted the delete-doclet-taglet-binaries branch December 1, 2023 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants