- file->SetImageData(imageData, dataSize);
- file->WriteDcmExplVR( "TestReadWriteReadCompare.dcm" );
- std::cout << " 2...";
+ /// \todo Following line commented out because gdcmFile::SetImageData() is
+ /// brain dead: it sets ImageDataSize to its argument and PixelRead to a.
+ /// Later on, when writing gdcmFile::WriteBase()
+ /// and because PixelRead == 1 we call
+ /// PixelElement->SetLength( ImageDataSizeRaw );
+ /// where we use ImageDataSizeRAW instead of ImageDataSize !
+ /// But when the original image made the transformation LUT -> RGB,
+ /// ImageDataSizeRaw is the third of ImageDataSize, and there is no
+ /// reason (since we called gdcmFile::SetImageData) to use the Raw image
+ /// size... This "bug" in gdcmFile made that we had to black list
+ /// images 8BitsUncompressedColor.dcm, OT-PAL-8-face.dcm and
+ /// US-PAL-8-10x-echo.dcm...
+ /// In conclusion fix gdcmFile, and then uncomment the following line.
+
+ // --> I did. ctest doesn't break. But ... is it enought to say it's OK ?
+
+ file->SetImageData(imageData, dataSize);
+
+ file->WriteDcmExplVR( "TestReadWriteReadCompare.dcm" );
+ std::cout << "2...";