<br><font size=2 face="sans-serif">Bonjour Mathieu,</font>
<br>
<br><font size=2 face="sans-serif">>1. Pas de tabulation, STP:<br>
>http://www.creatis.insa-lyon.fr/Public/Gdcm/CodingStyle.html<br>
</font>
<br><font size=2 face="sans-serif">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... ;-(</font>
<br>
<br><font size=2 face="sans-serif"><br>
>2. Pour la lib jpeg, le mechanisme est un peu complexe je l'admet, a<br>
>chaque fois que tu modifie la lib jpeg, il faut aussi penser a mettre a<br>
>jour les fichier mangle (pour utilisation de jpeg8, 12 et 16).<br>
><br>
>$ cvs ci -m"ENH: jpeg_memory_src depend on the bit depth, need to<br>
>mangle the name" mangle_jpeg12bits.h mangle_jpeg16bits.h mangle_jpeg8bits.h<br>
</font>
<br><font size=2 face="sans-serif">Ok je le savais pas.</font>
<br>
<br><font size=2 face="sans-serif"><br>
>3. J'utilise 'cvs diff -u3 -p' pour faire un patch. C'est super pratique<br>
>car ca prend 3 lignes au dessus et 3 lignes en dessous. Ca permet<br>
>d'appliquer des patchs /presque/ a tres long terme (y'a une sorte<br>
>d'algorithme flou pour retrouver les lignes originales).<br>
</font>
<br><font size=2 face="sans-serif">Ok je le note</font>
<br><font size=2 face="sans-serif"><br>
>4. Dans src/gdcmPixelReadConvert.cxx, c'est a mon avis dangereux de faire:<br>
><br>
>#define JOCTET unsigned char<br>
><br>
>ca ne marche que pour les image 8bits, non ?</font>
<br>
<br><font size=2 face="sans-serif">Non ca defini ce que c'est qu'un int8_t c'est a dire un OCTET (8 bits) quelquesoit la machine:</font>
<br><font size=2 face="sans-serif">cf libjpeg.doc:</font>
<br><font size=2 face="Courier New">The work buffer is defined as an array of datatype JOCTET, which is generally</font>
<br><font size=2 face="Courier New">"char" or "unsigned char".  On a machine where char is not exactly 8 bits</font>
<br><font size=2 face="Courier New">wide, you must define JOCTET as a wider data type and then modify the data</font>
<br><font size=2 face="Courier New">source and destination modules to transcribe the work arrays into 8-bit units</font>
<br><font size=2 face="Courier New">on external storage.</font>
<br>
<br><font size=2 face="sans-serif">normalement on doit faire</font>
<br><font size=2 face="sans-serif">#define JOCTET uint8_t</font>
<br><font size=2 face="sans-serif">ou un truc du genre.</font>
<br>
<br><font size=2 face="sans-serif">>5. La testing suite semble etre cassee sur les images JPEG multiframes<br>
>(*). Est-ce que tu peux verifier que mon patch n'a rien casse par<br>
>rapport au tien:<br>
><br>
>http://www.creatis.insa-lyon.fr/~malaterre/gdcm/gdcm-jpeg.patch<br>
><br>
</font>
<br><font size=2 face="sans-serif">Je vais voir.</font>
<br>
<br>
<br><font size=2 face="sans-serif">A+</font>
<br><font size=2 face="sans-serif">JM</font>