[Dcmlib] Comment se tirer une balle dans le pied

[Mathieu Malaterre]

Ok j'aime beaucoup cette solution :)

Merci
Mathieu

Jean-Pierre ROUX wrote:
> At 13:26 -0400 20/06/05, Mathieu Malaterre wrote:
> 
>> Depuis un moment je rale que gdcm n'utilise pas {} pour les if.
> 
> 
> 
> Est-ce qq chose s'oppose a ce que, dans le cas release, on definisse, 
> gdcmWarningMacro comme '{}' et non pas comme '<vide>'?
> 
> Ca eviterait de devoir rajouter des {} derriere tous les if a une ligne.
> Risque de warning sur 'empty structure' ?
> 
> JPRx
> 
>> Voila un excellent exemple:
>>
>> (gdcmDocument.cxx)
>>
>> ...
>> if ( Fp->fail() )
>>   gdcmWarningMacro("--> fail");
>>
>> gdcmWarningMacro("Unread element value " << valEntry->GetKey()
>>                   << " lgt : " << valEntry->GetReadLength()
>>                   << " at " << std::hex << valEntry->GetOffset());
>> valEntry->SetValue(GDCM_UNREAD);
>> return;
>> ...
>>
>> Maintenant si je vous dis qu'en Release gdcmWarningMacro est remplace 
>> par vide, ca donne le code suivant:
>>
>> if ( Fp->fail() )
>>
>>
>> valEntry->SetValue(GDCM_UNREAD);
>> return;
>>
>>
>> J'ai besoin de detailler...
>>
>> Mathieu
>> _______________________________________________
>> Dcmlib mailing list
>> Dcmlib at creatis.insa-lyon.fr
>> http://www.creatis.insa-lyon.fr/mailman/listinfo/dcmlib
> 
> 
>   Jean-Pierre ROUX
>   CREATIS - CNRS UMR 5515, INSERM U 630
>   Laboratoire de Radiologie Experimentale
>   Hopital Cardiologique
>   28 Avenue du Doyen LEPINE
>   B.P. Lyon-Montchat
>   69394 Lyon Cedex 03
> 
>   Tel      : (+33) 04 72 35 74 12
>   Fax      : (+33) 04 72 68 49 16
>   URL      : http://www.creatis.univ-lyon1.fr
>   e-mail   : jpr at creatis.univ-lyon1.fr
>                                    
> _______________________________________________
> Dcmlib mailing list
> Dcmlib at creatis.insa-lyon.fr
> http://www.creatis.insa-lyon.fr/mailman/listinfo/dcmlib
>