tinyxml2
tinyxml2 copied to clipboard
Increase depth recursion limit
The depth recursion limit TINYXML2_MAX_ELEMENT_DEPTH = 100 is quite restrictive for some applications. We encountered it here, where it is limiting our users from forming simulated chains with more than 100 joints (here is a video of a simulated chain, for illustration). While we of course accept that some limit is required for safety, we feel that 100 is too small.
This PR proposes to increase it to 500, which matches the depth allowed by other standard parsers, see e.g. libxml2 and IBM DataPower.
Thanks for your time and consideration.
Any updates on this?
Max element nesting depth 100 feels way too small to me, too. Please see also this thread
@leethomason Is there any update on this pull request?
@leethomason Is there any update on this? Thank you!
There is no value for that to make everyone happy. :) But I'm good with trying 500.
That is more than enough! Thank you very much @leethomason ! 😁