[GitHub] [xerces-c] rouault commented on pull request #39: [XERCESC-2227] Memleak fixes in ContentSpecNode and ComplexTypeInfo classes
rouault commented on pull request #39: URL: https://github.com/apache/xerces-c/pull/39#issuecomment-948891203 maybe this can be merged ? and https://github.com/apache/xerces-c/pull/40 as well ? -- 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: c-dev-unsubscr...@xerces.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: c-dev-unsubscr...@xerces.apache.org For additional commands, e-mail: c-dev-h...@xerces.apache.org
[GitHub] [xerces-c] rouault commented on pull request #39: [XERCESC-2227] Memleak fixes in ContentSpecNode and ComplexTypeInfo classes
rouault commented on pull request #39: URL: https://github.com/apache/xerces-c/pull/39#issuecomment-926102766 > Where you changed `retNode` to `specNode`, am I correct in the assumption that this was always buggy? no the code was fine. The change was a result from some temporary confusion, but at line 741 which I believe the only occurence, the 2 were aliases. I've pushed a commit to restore the original use of retNode -- 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: c-dev-unsubscr...@xerces.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: c-dev-unsubscr...@xerces.apache.org For additional commands, e-mail: c-dev-h...@xerces.apache.org