[Dcmlib] Re: Hack pour BIGENDIAN

[Mathieu Malaterre]

Mauvais nouvelle ca ne semble pas marcher:

  1/ 20 Testing TestBug                          Passed
   2/ 20 Testing TestHash                         Passed
   3/ 20 Testing TestTS                           Passed
   4/ 20 Testing TestVR                           Passed
   5/ 20 Testing TestUtil                         Passed
   6/ 20 Testing TestDicomString                  Passed
   7/ 20 Testing TestDict                         Passed
   8/ 20 Testing TestReadWriteReadCompare      ***Failed


JP, t'es sur que ton patch de swapper les pixels ca se fait que sur les 
buffer apres decompression JPEG ? Je vois pas de if(JPEG) dans 
gdcmBinEntry ...

Mathieu


Mathieu Malaterre wrote:
> JP,
> 
> Ok ca compile, je relance un dashboard sur Sun. Dis moi t'as meme pas 
> essaye' de le compiler sous linux (il suffisait de commenter les 
> ifdef...) :P
> 
> Mathieu
> 
> Mathieu Malaterre wrote:
> 
>> Houla houla,
>>
>> ca passe pas les chgt de ce matin:
>>
>> http://public.kitware.com/Public/Sites/dashsun1.kitware/GDCM-SunOS-CC/20050114-1602-Experimental/BuildError.html 
>>
>>
>> Site:dashsun1.kitware
>>
>> Build Name:GDCM-SunOS-CC
>> Found 16 Errors
>>
>> Warnings are here.
>> Build Log line 13
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 134
>>
>> cmake.depends is up-to-date
>> /.../gdcm-CC/src: building default_target
>> `cmake.depends' is up to date.
>> Building dependencies cmake.check_depends...
>> `cmake.depends' is up to date.
>> Building object file gdcmBinEntry.o...
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 134: Error: The operation 
>> "unsigned short(gdcm::DocEntry::*)() == int" is illegal.
>>
>> Build Log line 14
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 136
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 136: Error: unit16_t is not 
>> defined.
>>
>> Build Log line 15
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 136
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 136: Error: currPosition is not 
>> defined.
>>
>> Build Log line 16
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 140
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 140: Error: unit16_t is not 
>> defined.
>>
>> Build Log line 17
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 140
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 140: Error: buffer is not defined.
>>
>> Build Log line 18
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 140
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 140: Error: Type name expected 
>> instead of "uint16".
>>
>> Build Log line 19
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 148
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 148: Error: buffer is not defined.
>>
>> Build Log line 20
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 148
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 148: Error: The type "void" is 
>> incomplete.
>>
>> Build Log line 21
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 148
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 148: Error: A value of type 
>> void is not allowed.
>>
>> Build Log line 22
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 149
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 149: Error: The type "void" is 
>> incomplete.
>>
>> Build Log line 23
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 149
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 149: Error: A value of type 
>> void is not allowed.
>>
>> Build Log line 24
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 151
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 151: Error: currPosition is not 
>> defined.
>>
>> Build Log line 25
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 152
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 152: Error: currPosition is not 
>> defined.
>>
>> Build Log line 26
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 157
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 157: Error: currPosition is not 
>> defined.
>>
>> Build Log line 27
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 159
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 159: Error: buffer is not defined.
>>
>> Build Log line 28
>>
>> File: /.../gdcm/src/gdcmBinEntry.cxx Line: 159
>>
>> "/.../gdcm/src/gdcmBinEntry.cxx", line 159: Error: Badly formed 
>> expression.
>> 16 Error(s) detected.
>> *** Error code 16 (ignored)
>> The following command caused the error:
>> CC -o gdcmBinEntry.o -Dgdcm_EXPORTS    -KPIC  -I/.../gdcm/src 
>> -I/.../gdcm-CC/ -I/.../gdcm-CC/src  -c /.../gdcm/src/gdcmBinEntry.cxx
>> Building object file gdcmDicomDir.o...
>> Building object file gdcmDicomDirElement.o...
>>
>>
>>
>> Jean-Pierre Roux wrote:
>>
>>> Salut, Mathieu
>>>
>>> J'ai deporte dans BinEntry::WriteContent l'equivalent de ton Hack 
>>> pour le pb des processeurs BIGENDIAN.
>>> Un jour, il faudra faire la même chose pour toutes les BinEntry de VR 
>>> = OW
>>> Ce sont des 'chaines' de short int, pour le moment on les charge -ou 
>>> non, ca depend de leur longueur, et on ne les traite pas, car on n'en 
>>> fait rien. Pas la peine, donc, de les swapper, ni dans un sens ni 
>>> dans l'autre.
>>>
>>> Le code est tres simple, mais je n'ai pas pu tester faute de machine ...
>>>
>>> JP
>>>
>>
>>
>>
>> _______________________________________________
>> Dcmlib mailing list
>> Dcmlib at creatis.insa-lyon.fr
>> http://www.creatis.insa-lyon.fr/mailman/listinfo/dcmlib
>>
> 
> 
> 
> _______________________________________________
> Dcmlib mailing list
> Dcmlib at creatis.insa-lyon.fr
> http://www.creatis.insa-lyon.fr/mailman/listinfo/dcmlib
>