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

Fixes markup issues #391

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Fixes markup issues #391

wants to merge 6 commits into from

Conversation

funkyfuture
Copy link

a part of the fixes targets invalid XML where parenthesis are used as xml:id values. that makes it impossible for strict XML processors to load the affected document.

some Windows OS-based editors seem to been used as one URL ended with a LF entitity which is unexpected and depending on the client code / used URL parser might lead to problems.

there are also several occurances of &#xD (CR) in change records without any semantical significance, certainly stemming from the Windows editor world, and possibly an annoyance for users. let me know if i should amend an removal of these as well.

as parenthesis aren't valid within NCNAMEs
As there's no abbreviation provided in the context, most likely the fragment was meant
to tag an unclear transcription.
As the start of the surrounding lines seem to contain hardly recognizable contents,
it's likely that this was meant in the wrongly encoded part as well.
as it was situated before a closing tag.
That the following literal was the proper starting position is a plausible guess.
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.

None yet

1 participant