[Dcmlib] GE_RHAPSODE-16-MONO2-JPEG-Fragments.dcm

[jean-michel.rouet at philips.com]

Bonjour Mathieu,

>1. Pas de tabulation, STP:
>http://www.creatis.insa-lyon.fr/Public/Gdcm/CodingStyle.html

Pas de pb... normalement je n'utilise pas les tabs non plus, mais la 
j'avais ecrit ca a la maison, et mon vi etait pas configuré 
correctement... ;-(


>2. Pour la lib jpeg, le mechanisme est un peu complexe je l'admet, a
>chaque fois que tu modifie la lib jpeg, il faut aussi penser a mettre a
>jour les fichier mangle (pour utilisation de jpeg8, 12 et 16).
>
>$ cvs ci -m"ENH: jpeg_memory_src depend on the bit depth, need to
>mangle the name" mangle_jpeg12bits.h mangle_jpeg16bits.h 
mangle_jpeg8bits.h

Ok je le savais pas.


>3. J'utilise 'cvs diff -u3 -p' pour faire un patch. C'est super pratique
>car ca prend 3 lignes au dessus et 3 lignes en dessous. Ca permet
>d'appliquer des patchs /presque/ a tres long terme (y'a une sorte
>d'algorithme flou pour retrouver les lignes originales).

Ok je le note

>4. Dans src/gdcmPixelReadConvert.cxx, c'est a mon avis dangereux de 
faire:
>
>#define JOCTET unsigned char
>
>ca ne marche que pour les image 8bits, non ?

Non ca defini ce que c'est qu'un int8_t c'est a dire un OCTET (8 bits) 
quelquesoit la machine:
cf libjpeg.doc:
The work buffer is defined as an array of datatype JOCTET, which is 
generally
"char" or "unsigned char".  On a machine where char is not exactly 8 bits
wide, you must define JOCTET as a wider data type and then modify the data
source and destination modules to transcribe the work arrays into 8-bit 
units
on external storage.

normalement on doit faire
#define JOCTET uint8_t
ou un truc du genre.

>5. La testing suite semble etre cassee sur les images JPEG multiframes
>(*). Est-ce que tu peux verifier que mon patch n'a rien casse par
>rapport au tien:
>
>http://www.creatis.insa-lyon.fr/~malaterre/gdcm/gdcm-jpeg.patch
>

Je vais voir.


A+
JM
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.creatis.insa-lyon.fr/pipermail/dcmlib/attachments/20041208/3c75ac49/attachment.html>