umvarma / pynastran

Automatically exported from code.google.com/p/pynastran
1 stars 0 forks source link

Patch for /trunk/pyNastran/op4/op4.py #153

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
dense ascii op4 writing does not respect name_order; it writes it out in sorted 
order. I fixed this and refactored the code block to remove redundancy. Note 
that I swapped the sparse ascii and sparse binary blocks, since the logic 
seemed to be reversed there (if is_binary then write ascii?). Does this make 
sense?

Original issue reported on code.google.com by jeffrey....@gmail.com on 19 Jun 2014 at 10:39

Attachments:

GoogleCodeExporter commented 9 years ago
I didn't even see that logic bug.  It's certainly clearer now.

Original comment by mesheb82 on 20 Jun 2014 at 6:30