stevedlawrence opened a new pull request #104: Make maxOccursBounds limit a fatal error URL: https://github.com/apache/incubator-daffodil/pull/104 - Move the existing and unused TunableLimitExceeedError to SDE and treat it more like an SDE rather than a Diagnostic. This requires less special casing. - Instead of treating an array larger than maxOccursBounds as a ParseError, treat it as a fatal TunableLimitExceeded error to make it more obvious to the user that either the tunable needs to be increased or the schema needs to be modified - Move checkN inside the try/catch block that cleans up mark pool leaks since this now throws an exception. - Bump the maxOccursBoudns tunable to MaxInt to reduce the chance of valid schemas/data from hitting this bound. If there is invalid data, it may now just take longer before we realize things go off the rails and hits the larger maxOccursBounds. - Modify existing code to throw a TunableLimitExceededError instead of an SDE where appropriate DAFFODIL-1519
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services