X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=src%2FgdcmRLEFramesInfo.cxx;h=8a8ed7cb8201a73b27816e0bf802e0187019a03d;hb=eeb624df6a1661e969566c200505f3dac84f4cd6;hp=d95e93a521919b540417f2587b277217f3d1531d;hpb=d1c68c2c2ae9fadf927053150f7fbc625a7c7366;p=gdcm.git diff --git a/src/gdcmRLEFramesInfo.cxx b/src/gdcmRLEFramesInfo.cxx index d95e93a5..8a8ed7cb 100644 --- a/src/gdcmRLEFramesInfo.cxx +++ b/src/gdcmRLEFramesInfo.cxx @@ -3,8 +3,8 @@ Program: gdcm Module: $RCSfile: gdcmRLEFramesInfo.cxx,v $ Language: C++ - Date: $Date: 2005/02/01 10:29:56 $ - Version: $Revision: 1.11 $ + Date: $Date: 2005/06/17 12:27:52 $ + Version: $Revision: 1.16 $ Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de l'Image). All rights reserved. See Doc/License.txt or @@ -18,6 +18,7 @@ #include "gdcmRLEFramesInfo.h" #include "gdcmDebug.h" +#include "gdcmUtil.h" namespace gdcm { @@ -61,10 +62,17 @@ RLEFrame *RLEFramesInfo::GetNextFrame() * \brief Reads from disk the Pixel Data of 'Run Length Encoded' * Dicom encapsulated file and decompress it. * @param fp already open File Pointer - * at which the pixel data should be copied + * from which the pixel data should be read + * @param raw raw + * @param xSize x Size + * @param ySize y Size + * @param zSize z Size + * @param bitsAllocated Bits allocated * @return Boolean */ -bool RLEFramesInfo::DecompressRLEFile( std::ifstream *fp , uint8_t *raw, int xSize, int ySize, int zSize, int bitsAllocated ) +bool RLEFramesInfo::DecompressRLEFile( std::ifstream *fp , uint8_t *raw, + int xSize, int ySize, int zSize, + int bitsAllocated ) { uint8_t *subRaw = raw; long rawSegmentSize = xSize * ySize; @@ -85,31 +93,47 @@ bool RLEFramesInfo::DecompressRLEFile( std::ifstream *fp , uint8_t *raw, int xSi } /** - * \brief Try to deal with RLE 16 Bits. - * We assume the RLE has already been parsed and loaded in - * Raw (through \ref ReadAndDecompressJPEGFile ). - * We here need to make 16 Bits Pixels from Low Byte and - * High Byte 'Planes'...(for what it may mean) - * @return Boolean + * \brief We assume Raw contains the decoded RLE pixels but as + * 8 bits per pixel. We convert those pixels to 16 bits + * per pixel. + * @param raw raw + * @param xSize x Size + * @param ySize y Size + * @param numberOfFrames number of frames + * @return Boolean always true */ -bool RLEFramesInfo::ConvertRLE16BitsFromRLE8Bits( uint8_t* raw, int xSize, - int ySize,int numberOfFrames ) +bool RLEFramesInfo::ConvertRLE16BitsFromRLE8Bits(uint8_t *raw, int xSize, + int ySize, int numberOfFrames) { size_t pixelNumber = xSize * ySize; - size_t rawSize = xSize * ySize * numberOfFrames; + size_t rawSize = pixelNumber * numberOfFrames * 2; // We assumed Raw contains the decoded RLE pixels but as // 8 bits per pixel. In order to convert those pixels to 16 bits // per pixel we cannot work in place within Raw and hence // we copy it in a safe place, say copyRaw. - uint8_t* copyRaw = new uint8_t[rawSize * 2]; - memmove( copyRaw, raw, rawSize * 2 ); + uint8_t *copyRaw = new uint8_t[rawSize]; + memmove( copyRaw, raw, rawSize ); + + uint8_t *x = raw; + uint8_t *a; + uint8_t *b; - uint8_t* x = raw; - uint8_t* a = copyRaw; - uint8_t* b = a + pixelNumber; + // Warning : unckecked patch to see the behaviour on Big Endian Processors + if ( !Util::IsCurrentProcessorBigEndian() ) + { + a = copyRaw; // begining of 'low bytes' + b = a + pixelNumber; // begining of 'hight bytes' + } + else + { + b = copyRaw; // begining of 'low bytes' + a = b + pixelNumber; // begining of 'hight bytes' + } + + // Re order bytes for ( int i = 0; i < numberOfFrames; i++ ) { for ( unsigned int j = 0; j < pixelNumber; j++ ) @@ -119,9 +143,9 @@ bool RLEFramesInfo::ConvertRLE16BitsFromRLE8Bits( uint8_t* raw, int xSize, } } + delete[] copyRaw; - - /// \todo check that operator new []didn't fail, and sometimes return false + return true; }