[Dcmlib] Bug in File::GetZOrigin

[Benoit Regrain]

Not for the origin and spacing... only for the image size

Benoit


----- Original Message ----- 
From: "Mathieu Malaterre" <mathieu.malaterre at kitware.com>
To: "Marco Feuerstein" <feuerste at cs.tum.edu>
Cc: <Dcmlib at creatis.insa-lyon.fr>
Sent: Friday, October 21, 2005 3:54 PM
Subject: Re: [Dcmlib] Bug in File::GetZOrigin


> Marco Feuerstein wrote:
>> Hi,
>> 
>> 
>> I just realized that the current CVS version of gdcm has a bug in
>> File::GetZOrigin()
>> In lines 822 and 846 it should be
>> 
>> return (float)entry->GetValue(2);
>> 
>> instead of
>> 
>> return (float)entry->GetValue(0);
>> 
>> 
>> Can you please fix it? Thank you!
>> Please CC me, when you fixed it, since I am not subscribed to the list.
> 
> Definitely ! Thanks a lot for the bug report. I cannot believe none of 
> the tests realize the value was wrong...
> 
> Benoit, any comment on why test might not have seen the problem ? We are 
> supposed to read the Z value in the 'tst' file and compare it, right ?
> 
> Mathieu
> _______________________________________________
> Dcmlib mailing list
> Dcmlib at creatis.insa-lyon.fr
> http://www.creatis.insa-lyon.fr/mailman/listinfo/dcmlib