[Dcmlib] Comment se tirer une balle dans le pied

[Jean-Pierre ROUX]

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