[Dcmlib] Bug in File::GetZOrigin

Mathieu Malaterre mathieu.malaterre at kitware.com
Fri Oct 21 15:54:50 CEST 2005


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



More information about the Dcmlib mailing list