You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@daffodil.apache.org by GitBox <gi...@apache.org> on 2018/07/16 14:41:40 UTC

[GitHub] marshalljmiller commented on issue #80: added a layer transform that swaps bytes within an array of 32 bit words

marshalljmiller commented on issue #80: added a layer transform that swaps bytes within an array of 32 bit words
URL: https://github.com/apache/incubator-daffodil/pull/80#issuecomment-405270564
 
 
   I will update the commit message so that it is formatted correctly and with more detail.  Should I amend the commit and force push it and then add follow up commits for the necessary code changes?
   
   You are correct that the bug describes implementing the fourth combination of bit+byte order, but after discussing the problem with Mike on the list (https://lists.apache.org/thread.html/99bcd6fe3d75fbcaa70413ee643b7d3ce247260dc0224a78dc45bd23@%3Cdev.daffodil.apache.org%3E) we both concluded that this approach would be simpler and would achieve the desired support.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services