[Dcmlib] TestVR back again
Jean-Pierre Roux
Jean-Pierre.Roux at creatis.insa-lyon.fr
Fri Oct 21 12:07:44 CEST 2005
>
>
>>> Hello,
>>>
>>> I just modify the TestVR to fail on a very bizarre case. Looks like
>>> the switch in gdcm::VR that read on the fly the dict is not working.
>>> Whereas if you open gdcmDefaultDict.cxx in the gdcm-bin dir you'll
>>> see that there is not '\000/' VR.
>>>
>>> Instead of fixing the parser how about we always use the other case:
>>> FillDefault ? This is a very nice feature in gdcm to load on the fly
>>> a new dicom dict but how about a new VR dict ? Does it really make
>>> sense ? How often does it change ? Did it ever change ?
>>
VR list didn't change since *ages*§
-One of the few things permanent in Dicom-
Sure we'll never try to create VR on the fly !
When a image is supposed to be 'Explicit VR' -we *know* some Entries
are *not*-, the only way we have to guess if the length is coded on 2 or
4 bytes is to check the 2 first bytes of the 'area' taht may contain,
either [any VR but OB, OW, SQ + length, coded on 2 bytes] or [OB, OW or
SQ + 0x0000] or [ length, coded on 4 bytes]
An pray -or throw curses on %$@ Dicom- we never get, in an illegal
Implicit VR Entry, a length starting with 0x5352
(this entry will be considered as a Sequence :-( )
JP
>>>
>>> Mathieu
>>>
More information about the Dcmlib
mailing list