Hi Sean, After adding some missing colons :-), the patch has been pushed:
http://itk.org/gitweb?p=ITK.git;a=commit;h=a9fd63e26738f3c1449133c8077c67e072d1202a Thanks, Matt On Mon, Mar 25, 2013 at 7:16 PM, Matt McCormick <[email protected]> wrote: > Hi Sean, > > Thanks again for the patch. I have applied it to the 'release-3.20' > branch, and I will push after a clean build/test. > > Thanks, > Matt > > On Mon, Mar 25, 2013 at 4:13 PM, Sean McBride <[email protected]> wrote: >> Hans, Matt, >> >> Thanks for merging this into ITK master. As I'm stuck on ITK 3 currently, >> I'd like too see it merged in there too. IIRC the gerrit workflow does not >> apply in the ITK 3 case, or am I remembering wrong? Attached is a patch for >> ITK3, but I can go via gerrit if that's the correct approach. >> >> Thanks, >> >> Sean >> >> >> On Wed, 20 Mar 2013 21:45:46 +0000, Matt McCormick said: >> >>>Hi Sean, >>> >>>It looks like a good fix. Thanks for submitting. >>> >>>Matt >>> >>>On Wed, Mar 20, 2013 at 5:49 PM, Sean McBride <[email protected]> >>>wrote: >>>> Hi all, >>>> >>>> A customer of ours has a 'flipped' Analyze 7.5 file. >>>> >>>> We created a patch for 4.x here: >>>> <http://review.source.kitware.com/#/c/10327/1/Modules/IO/NIFTI/src/ >>>itkNiftiImageIO.cxx> >>>> >>>> and for 3.x (attached). >>>> >>>> Does this seem like a correct/acceptable fix? _______________________________________________ Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Kitware offers ITK Training Courses, for more information visit: http://kitware.com/products/protraining.php Please keep messages on-topic and check the ITK FAQ at: http://www.itk.org/Wiki/ITK_FAQ Follow this link to subscribe/unsubscribe: http://www.itk.org/mailman/listinfo/insight-developers
