afs commented on issue #1864: URL: https://github.com/apache/jena/issues/1864#issuecomment-1537371665
Hi @Marcono1234 -- yes, it's excessively safe code (likely either the optimizer will remove it or will be a zero cost instruction in the execution pipeline because the value is also used in the rest of the expression). xmlinput0 is legacy - the original code for the parser before the current efforts maintaining the RDF/XML parser. xmlinput is the RDF/XML parser in-use in release Jena 4.8.0. There is development work to simplify the structure. ARP (the internalname for the parse) It has many features that have been added or existed when it was a standalone codebase. I've made the change on my current active development so it does not get forgotten. This code, when finished, will become "xmlinput" and the current xmlinput (renamed to xmlinput1) will be the new legacy. As a project Jena has always tried to support legacy behaviour for as long as possible. Each situation is treated individually. Jena has a long-tail of version usage so it helps users when they jump several versions. More: https://lists.apache.org/thread/zom28q9dqc7h4mx8s4v6h28c6b9sx2qh -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
