[Dcmlib] Comment se tirer une balle dans le pied

[Mathieu Malaterre]

Depuis un moment je rale que gdcm n'utilise pas {} pour les if. 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