Skip to content

Conversation

@SingingBush
Copy link
Contributor

This PR focuses on AbstractDOMParser & AbstractXMLDocumentParser and the related interfaces XMLDTDContentModelHandler and XMLDTDHandler

/** Default constructor. */
/**
* Default constructor.
* <p>Constructs a DOM parser using default features</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

period

* internal entities or a document entity that is
* parsed from a java.io.Reader).
* @param augs Additional information that may include infoset augmentations
* @param name the name of the entity
Copy link
Contributor

Choose a reason for hiding this comment

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

general entity

*
* @throws XNIException Thrown by handler to signal an error.
* @param name the name of the entity. Parameter entity names start with '%', whereas
* the name of a general entity is just the entity name
Copy link
Contributor

Choose a reason for hiding this comment

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

needs a period since this ends a sentence

* @param text the value of the entity
* @param nonNormalizedText The non-normalized value of the entity. This value contains
* the same sequence of characters that was in the internal entity declaration,
* without any entity references expanded
Copy link
Contributor

Choose a reason for hiding this comment

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

period

@SingingBush SingingBush requested a review from elharo December 1, 2025 22:41
@elharo elharo merged commit 6b07564 into apache:main Dec 2, 2025
5 checks passed
@SingingBush SingingBush deleted the javadoc/abstract-parsers-and-handler-interfaces branch December 2, 2025 12: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