Comment why defaul case in appendAttreibuteValue is never executed

Also add comment tags so that lcov can ignore unreachable code

Comment why encoding check in appendAttributeValue is never true

Also add comment tag so lcov can ignore unreachable code

Remove unreachable error handling code in character ref handling

Comment why internal entity in param entity is believe impossible.

Also add comment tags to allow lcov to ignore unreachable code

Comment why external subset name lookup cannot fail

Also add comment tag so that lcov can ignore unreachable code

Remove unreachable code in entityValueInitProcessor

Comment why doIgnoreSection's default case is never called.

Also add comment tags so that lcov can ignore unreachable code

Comment why doIgnoreSection is believed to always use the parser's encoding

Also add comment tags so that lcov can ignore the unreachable code

Comment why doCdataSection's default case is never executed.

Also add comment tags so that lcov can ignore the unreachable code

Comment why code in storeAtts() is never executed.

Also add comment tags so lcov can ignore the unreachable code

Comment why doContent's default case is never executed.

Also add comment markers so lcov can ignore the unreachable code.

Comment why tidy-up parse cannot be suspended

Also add tags so that lcov can ignore the unreachable code.

Test parsing element in namespace with a failing allocator

Test extremely long namespace prefix

Exercises a path in poolGrow that requires attempting to over-fill
a gap of more than 1024 characters in a non-empty pool.

Test large element group with a failing allocator

Test long notation value with external entity and failing allocator

Move external_entity_optioner up to be more generally available

No functionality changes in this commit

Test long entity value and external entity with duff allocator

Test external entity with long public ID and failing allocator

Test long base URI for external parser with duff allocator

Test long entity system ID in external parser with failing allocator

Test with long default attribute for ext parser with duff allocator

Test longer context URI with a failing reallocator

Test a long URI in the context string with a failing allocator

Test long namespace in external entity context with duff reallocator

Test long general entity name in external DTD with failing realloc

Test shorter long context URI with a failing reallocator

One last code path to exercise

Again test long context URI against a failing reallocator

Another different code path

Refactor context URI reallocation tests

Test shorter context URI in the face of a failing reallocator

Yet another different code path exercised

Test different length context URI with duff reallocator

Exercises another different code path

Test slightly shorter context URI with duff allocator

Exercises a different code path to previous context test

Test appropriately long context URI with a failing reallocator

Test not-so-long namespace with failing allocator

Test long namespace with failing allocator

Test long root element name with a failing allocator

Test longer namespace prefix with failing reallocator

Test long namespace prefix with a failing reallocator

Test handling long attribute list with a failing reallocator

Test that comments reach the default handler

Test processing instruction with long body and duff allocator

Test processing instruction reaches default handler if no PI handler

Test char entity extending param entity with failing reallocator

Silence overly-pessimistic compiler warning.

The parsed_version structure will be fully filled in if parse_version
returns a non-zero value, but initializing the variable to zero is a
trivial cost for the tests.

Test invalid character reference is faulted

Test expanding a param entity's string pool with a failing allocator

Test trailing CR in a PE value is interpreted as a newline

Test error in nested external entity substitution

Test another different code path for skipped external entity

Test different code path for skipping missing external entity

Test recursive parameter entity

Test missing external entity is skipped

Test general entity substitution in standalone internal param entity

Test attribute value of a predefined entity with a duff allocator

Test whitespace attribute value with dodgy allocator

Test attribute value with trailing CR

Test long attribute value with failing allocator

Test long attribute value with character ref and a failing allocator

Tests strictly that a character reference at the end of a long default
value for an attribute will trigger the relevant string pool to grow,
and will recover correctly if there is no memory for growth

Test incomplete param ref in attribute default is rejected

Test invalid character in default attribute is rejected

Test long attribute with a dodgy reallocator

This tests a specific code path through parameter substitution, and
requires the attribute value to come out to 1024 bytes PLUS a terminal
NUL to trigger growth of the string pool in use.  The NUL is added
separately, and it is that string append that we are triggering on.

Test restarting an errored parser returns the same error

Check that resumed general entity parse can be re-suspended

Test resuming a general entity catches XML errors

Test suspending and resuming parse in parameter entity substitution

Test suspending and resuming parse during entity substitution

Add test to ensure a hash collision while expanding hash table

This is purely an exercise in code coverage; there is no user-level
way of telling that the hash table has been expanded or that a
collision occurred.

Test suspending parse during epilog

Test aborting in the epilog not from a CR character

Test handling of partial characters at the end of the epilog

Test incomplete token in epilog

Test comment in epilog with failing allocator

Test PI handling in epilog with a failing allocator

Extend abort test to cover more epilog code

Test aborting parse during epilog processing

Test suspending parse during an XML declaration

Shuffle code to keep similar things together.

No functional changes

Test group with many elements and a failing reallocator

Test deeply nested group parsing with a failing allocator

Add test for external parameter entity with "not standalone" handler

Test external param entity with no further external parser

Test undefined system parameter entity in external DTD

Test recursive parameter entity is rejected

Test skipped parameter entity in external DTD triggers handler

Review 2017-03-13: refactor XML_Expat_Version test

Also revised to allow for the different definitions of XML_LChar.
Previous versions will not have worked in XML_UNICODE builds.

Review 2017-03-13: construct test string to avoid magic numbers

Review 2017-03-13: replace magic number with appropriate string len

Remove warning about potentially unset variable in tests

Also make test function external_entity_public() more robust
against misuse.

Test standalone parameter entity coverage

Test substituted public entity with long name against failing realloc

Test group with choice in element declaration

Add element decl handler to nested group reallocation test

Add element declaration handler to nested group test

Test nested groups with a failing reallocator

Test deeply-nested groups

Test public notation with no system ID

Test system notation with failing allocator

Test public notation with long system ID against failing allocator

Extend notation allocation test with notation declaration handler

This extends the code coverage a little, in particular checking
another allocation failure path.

Extend PE allocation test with long entity name