stevedlawrence commented on a change in pull request #505:
URL: https://github.com/apache/daffodil/pull/505#discussion_r623964227



##########
File path: 
daffodil-cli/src/it/resources/org/apache/daffodil/CLI/output/output9.txt
##########
@@ -1,4 +1,4 @@
-<base14:rabbitHole xmlns:b14="http://b14.com"; xmlns:a14="http://a14.com"; 
xmlns:base14="http://baseSchema.com";>
+<base14:rabbitHole xmlns:a14="http://a14.com"; xmlns:b14="http://b14.com"; 
xmlns:base14="http://baseSchema.com";>

Review comment:
       As to you other question, I'm not really sure why the serialization 
changes causes namespace mapping ordering changes. I suspect copying the list 
caused some things to be evaluated in a slightly different order and because we 
used Set/Maps, however things got lazily evaluated changed the order. It's not 
entirely clear to me, and I couldn't track down the underlying change. But 
2.12.13 did backport some Map changes from 2.13.x, so a change in ordering 
isn't too surprising. 




-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to