Yishay, a few other issues became apparent as I worked on that. So I added tests for them and fixed them also.
I can confirm that a lot of the older implementation was failing in many of the tests, because I reverted to the older implementation locally and checked against the same tests (which run in parallel with swf). So I do know that this represents progress. But what I cannot know is if there are other things that do not work after these changes outside of the current test coverage (which is why it is essential to enhance that coverage over time). However I also have no desire to cause many sudden changes needed in your project that we could let you manage more gradually if this is causing a problem, because I have a feeling that you and Harbs are the main users of XML at the moment. If you are still encountering problems after my latest changes then we could revert the XML project to how it was before all my recent changes and I could move everything to a branch. I haven't done that to date simply because in terms of what I can see in the tests so far, I can only consider it to be progress in terms of getting the implementation closer (than it was before) to the original as3 XML/XMList. But as I said... that is only as good as the test coverage. Anyhow, let me know what you think... I have fingers crossed that the latest changes should address things for you. Greg On Mon, Jun 24, 2019 at 12:35 AM Greg Dove <[email protected]> wrote: > Hi Yishay, thanks for the new details. That's another bug. I have a local > fix for that now, and new tests to cover this and more variations of the > document level parsing. > It's late for me now, so I will check everything thoroughly and push the > fix in my morning. > > > > On Sun, Jun 23, 2019 at 9:57 PM Yishay Weiss <[email protected]> > wrote: > >> Hi Greg, >> >> I’m still having issues with xml that were not there before. Can you see >> why this [1] app, where my.xml is [2] is giving me an exception [3]? >> >> >> >> [1] https://paste.apache.org/ld1L >> [2] https://paste.apache.org/d6VX >> [3] https://paste.apache.org/qpub >> >> Yishay >> >
