X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=src%2FgdcmHeader.cxx;h=bac92ce95997e8092fc801018dddf1cb5a983d26;hb=d4f7a5d74be3a4a332380d24631f3d0d96f98f5f;hp=361aedd878f997036677fccddfeca4cf6d8dc870;hpb=81ed6935092ed908ca5c1af450f04c84baacf568;p=gdcm.git diff --git a/src/gdcmHeader.cxx b/src/gdcmHeader.cxx index 361aedd8..bac92ce9 100644 --- a/src/gdcmHeader.cxx +++ b/src/gdcmHeader.cxx @@ -1,99 +1,89 @@ // gdcmHeader.cxx -#include "gdcm.h" #include +#include // For nthos: #ifdef _MSC_VER #include #else #include #endif -#include // for isalpha -#include +#include // for isalpha #include #include "gdcmUtil.h" +#include "gdcmHeader.h" -#define HEADER_LENGTH_TO_READ 256 // on ne lit plus que le debut -#define _MaxSizeLoadElementValue_ 1024 // longueur au dela de laquelle on ne charge plus les valeurs - -namespace Error { - struct FileReadError { - FileReadError(FILE* fp, const char* Mesg) { - if (feof(fp)) - dbg.Verbose(1, "EOF encountered :", Mesg); - if (ferror(fp)) - dbg.Verbose(1, "Error on reading :", Mesg); - } - }; -} - -//FIXME: this looks dirty to me... - -#define str2num(str, typeNum) *((typeNum *)(str)) - -// str est un pointeur dans un tableau de caractères, qui doit contenir, -// à cet endroit la, la représentation binaire d'un entier (16 ou 32 bits) -// je veux récupérer ça ... dans un entier. -// s'il y a une autre solution, évitant des cast et les indirections, -// je suis preneur +// Refer to gdcmHeader::CheckSwap() +#define HEADER_LENGTH_TO_READ 256 +// Refer to gdcmHeader::SetMaxSizeLoadElementValue() +#define _MaxSizeLoadElementValue_ 1024 VRHT * gdcmHeader::dicom_vr = (VRHT*)0; -gdcmDictSet* gdcmHeader::Dicts = new gdcmDictSet(); void gdcmHeader::Initialise(void) { - if (!gdcmHeader::dicom_vr) - InitVRDict(); - RefPubDict = gdcmHeader::Dicts->GetDefaultPublicDict(); - RefShaDict = (gdcmDict*)0; + if (!gdcmHeader::dicom_vr) + InitVRDict(); + Dicts = new gdcmDictSet(); + RefPubDict = Dicts->GetDefaultPubDict(); + RefShaDict = (gdcmDict*)0; } -gdcmHeader::gdcmHeader (const char* InFilename) { - SetMaxSizeLoadElementValue(_MaxSizeLoadElementValue_); - filename = InFilename; - Initialise(); - fp=fopen(InFilename,"rw"); - dbg.Error(!fp, "gdcmHeader::gdcmHeader cannot open file", InFilename); - ParseHeader(); +gdcmHeader::gdcmHeader(const char *InFilename, bool exception_on_error) + throw(gdcmFileError) { + SetMaxSizeLoadElementValue(_MaxSizeLoadElementValue_); + filename = InFilename; + Initialise(); + fp=fopen(InFilename,"rb"); + if(exception_on_error) { + if(!fp) + throw gdcmFileError("gdcmHeader::gdcmHeader(const char *, bool)"); + } + else + dbg.Error(!fp, "gdcmHeader::gdcmHeader cannot open file", InFilename); + ParseHeader(); + AddAndDefaultElements(); } + gdcmHeader::~gdcmHeader (void) { - fclose(fp); - return; + //FIXME obviously there is much to be done here ! + fclose(fp); + return; } void gdcmHeader::InitVRDict (void) { - if (dicom_vr) { - dbg.Verbose(0, "gdcmHeader::InitVRDict:", "VR dictionary allready set"); - return; - } - VRHT *vr = new VRHT; - (*vr)["AE"] = "Application Entity"; // At most 16 bytes - (*vr)["AS"] = "Age String"; // Exactly 4 bytes - (*vr)["AT"] = "Attribute Tag"; // 2 16-bit unsigned short integers - (*vr)["CS"] = "Code String"; // At most 16 bytes - (*vr)["DA"] = "Date"; // Exactly 8 bytes - (*vr)["DS"] = "Decimal String"; // At most 16 bytes - (*vr)["DT"] = "Date Time"; // At most 26 bytes - (*vr)["FL"] = "Floating Point Single"; // 32-bit IEEE 754:1985 float - (*vr)["FD"] = "Floating Point Double"; // 64-bit IEEE 754:1985 double - (*vr)["IS"] = "Integer String"; // At most 12 bytes - (*vr)["LO"] = "Long String"; // At most 64 chars - (*vr)["LT"] = "Long Text"; // At most 10240 chars - (*vr)["OB"] = "Other Byte String"; // String of bytes (vr independant) - (*vr)["OW"] = "Other Word String"; // String of 16-bit words (vr dep) - (*vr)["PN"] = "Person Name"; // At most 64 chars - (*vr)["SH"] = "Short String"; // At most 16 chars - (*vr)["SL"] = "Signed Long"; // Exactly 4 bytes - (*vr)["SQ"] = "Sequence of Items"; // Not Applicable - (*vr)["SS"] = "Signed Short"; // Exactly 2 bytes - (*vr)["ST"] = "Short Text"; // At most 1024 chars - (*vr)["TM"] = "Time"; // At most 16 bytes - (*vr)["UI"] = "Unique Identifier"; // At most 64 bytes - (*vr)["UL"] = "Unsigned Long "; // Exactly 4 bytes - (*vr)["UN"] = "Unknown"; // Any length of bytes - (*vr)["US"] = "Unsigned Short "; // Exactly 2 bytes - (*vr)["UT"] = "Unlimited Text"; // At most 2^32 -1 chars - dicom_vr = vr; + if (dicom_vr) { + dbg.Verbose(0, "gdcmHeader::InitVRDict:", "VR dictionary allready set"); + return; + } + VRHT *vr = new VRHT; + (*vr)["AE"] = "Application Entity"; // At most 16 bytes + (*vr)["AS"] = "Age String"; // Exactly 4 bytes + (*vr)["AT"] = "Attribute Tag"; // 2 16-bit unsigned short integers + (*vr)["CS"] = "Code String"; // At most 16 bytes + (*vr)["DA"] = "Date"; // Exactly 8 bytes + (*vr)["DS"] = "Decimal String"; // At most 16 bytes + (*vr)["DT"] = "Date Time"; // At most 26 bytes + (*vr)["FL"] = "Floating Point Single"; // 32-bit IEEE 754:1985 float + (*vr)["FD"] = "Floating Point Double"; // 64-bit IEEE 754:1985 double + (*vr)["IS"] = "Integer String"; // At most 12 bytes + (*vr)["LO"] = "Long String"; // At most 64 chars + (*vr)["LT"] = "Long Text"; // At most 10240 chars + (*vr)["OB"] = "Other Byte String"; // String of bytes (vr independant) + (*vr)["OW"] = "Other Word String"; // String of 16-bit words (vr dep) + (*vr)["PN"] = "Person Name"; // At most 64 chars + (*vr)["SH"] = "Short String"; // At most 16 chars + (*vr)["SL"] = "Signed Long"; // Exactly 4 bytes + (*vr)["SQ"] = "Sequence of Items"; // Not Applicable + (*vr)["SS"] = "Signed Short"; // Exactly 2 bytes + (*vr)["ST"] = "Short Text"; // At most 1024 chars + (*vr)["TM"] = "Time"; // At most 16 bytes + (*vr)["UI"] = "Unique Identifier"; // At most 64 bytes + (*vr)["UL"] = "Unsigned Long "; // Exactly 4 bytes + (*vr)["UN"] = "Unknown"; // Any length of bytes + (*vr)["US"] = "Unsigned Short "; // Exactly 2 bytes + (*vr)["UT"] = "Unlimited Text"; // At most 2^32 -1 chars + dicom_vr = vr; } /** @@ -104,151 +94,154 @@ void gdcmHeader::InitVRDict (void) { */ void gdcmHeader::CheckSwap() { - // The only guaranted way of finding the swap code is to find a - // group tag since we know it's length has to be of four bytes i.e. - // 0x00000004. Finding the swap code in then straigthforward. Trouble - // occurs when we can't find such group... - guint32 s; - guint32 x=4; // x : pour ntohs - bool net2host; // true when HostByteOrder is the same as NetworkByteOrder - - int lgrLue; - char * entCur; - char deb[HEADER_LENGTH_TO_READ]; - - // First, compare HostByteOrder and NetworkByteOrder in order to - // determine if we shall need to swap bytes (i.e. the Endian type). - if (x==ntohs(x)) - net2host = true; - else - net2host = false; - - // The easiest case is the one of a DICOM header, since it possesses a - // file preamble where it suffice to look for the sting "DICM". - lgrLue = fread(deb, 1, HEADER_LENGTH_TO_READ, fp); - - entCur = deb + 128; - if(memcmp(entCur, "DICM", (size_t)4) == 0) { - filetype = TrueDicom; - dbg.Verbose(1, "gdcmHeader::CheckSwap:", "looks like DICOM Version3"); - } else { - filetype = Unknown; - dbg.Verbose(1, "gdcmHeader::CheckSwap:", "not a DICOM Version3 file"); - } - - if(filetype == TrueDicom) { - // Next, determine the value representation (VR). Let's skip to the - // first element (0002, 0000) and check there if we find "UL", in - // which case we (almost) know it is explicit VR. - // WARNING: if it happens to be implicit VR then what we will read - // is the length of the group. If this ascii representation of this - // length happens to be "UL" then we shall believe it is explicit VR. - // FIXME: in order to fix the above warning, we could read the next - // element value (or a couple of elements values) in order to make - // sure we are not commiting a big mistake. - // We need to skip : - // * the 128 bytes of File Preamble (often padded with zeroes), - // * the 4 bytes of "DICM" string, - // * the 4 bytes of the first tag (0002, 0000), - // i.e. a total of 136 bytes. - entCur = deb + 136; - if(memcmp(entCur, "UL", (size_t)2) == 0) { - filetype = ExplicitVR; - dbg.Verbose(1, "gdcmHeader::CheckSwap:", - "explicit Value Representation"); - } else { - filetype = ImplicitVR; - dbg.Verbose(1, "gdcmHeader::CheckSwap:", - "not an explicit Value Representation"); - } - - if (net2host) { - sw = 4321; - dbg.Verbose(1, "gdcmHeader::CheckSwap:", - "HostByteOrder != NetworkByteOrder"); - } else { - sw = 0; - dbg.Verbose(1, "gdcmHeader::CheckSwap:", - "HostByteOrder = NetworkByteOrder"); - } - - // Position the file position indicator at first tag (i.e. - // after the file preamble and the "DICM" string). - rewind(fp); - fseek (fp, 132L, SEEK_SET); - return; - } // End of TrueDicom - - // Alas, this is not a DicomV3 file and whatever happens there is no file - // preamble. We can reset the file position indicator to where the data - // is (i.e. the beginning of the file). - rewind(fp); - - // Our next best chance would be to be considering a 'clean' ACR/NEMA file. - // By clean we mean that the length of the first tag is written down. - // If this is the case and since the length of the first group HAS to be - // four (bytes), then determining the proper swap code is straightforward. - - entCur = deb + 4; - s = str2num(entCur, guint32); - - switch (s) { - case 0x00040000 : - sw = 3412; - filetype = ACR; - return; - case 0x04000000 : - sw = 4321; - filetype = ACR; - return; - case 0x00000400 : - sw = 2143; - filetype = ACR; - return; - case 0x00000004 : - sw = 0; - filetype = ACR; - return; - default : - dbg.Verbose(0, "gdcmHeader::CheckSwap:", - "ACR/NEMA unfound swap info (time to raise bets)"); - } - - // We are out of luck. It is not a DicomV3 nor a 'clean' ACR/NEMA file. - // It is time for despaired wild guesses. So, let's assume this file - // happens to be 'dirty' ACR/NEMA, i.e. the length of the group is - // not present. Then the only info we have is the net2host one. - if (! net2host ) - sw = 0; - else - sw = 4321; - return; + // The only guaranted way of finding the swap code is to find a + // group tag since we know it's length has to be of four bytes i.e. + // 0x00000004. Finding the swap code in then straigthforward. Trouble + // occurs when we can't find such group... + guint32 s; + guint32 x=4; // x : pour ntohs + bool net2host; // true when HostByteOrder is the same as NetworkByteOrder + + int lgrLue; + char * entCur; + char deb[HEADER_LENGTH_TO_READ]; + + // First, compare HostByteOrder and NetworkByteOrder in order to + // determine if we shall need to swap bytes (i.e. the Endian type). + if (x==ntohs(x)) + net2host = true; + else + net2host = false; + + // The easiest case is the one of a DICOM header, since it possesses a + // file preamble where it suffice to look for the string "DICM". + lgrLue = fread(deb, 1, HEADER_LENGTH_TO_READ, fp); + + entCur = deb + 128; + if(memcmp(entCur, "DICM", (size_t)4) == 0) { + filetype = TrueDicom; + dbg.Verbose(1, "gdcmHeader::CheckSwap:", "looks like DICOM Version3"); + } else { + filetype = Unknown; + dbg.Verbose(1, "gdcmHeader::CheckSwap:", "not a DICOM Version3 file"); + } + + if(filetype == TrueDicom) { + // Next, determine the value representation (VR). Let's skip to the + // first element (0002, 0000) and check there if we find "UL", in + // which case we (almost) know it is explicit VR. + // WARNING: if it happens to be implicit VR then what we will read + // is the length of the group. If this ascii representation of this + // length happens to be "UL" then we shall believe it is explicit VR. + // FIXME: in order to fix the above warning, we could read the next + // element value (or a couple of elements values) in order to make + // sure we are not commiting a big mistake. + // We need to skip : + // * the 128 bytes of File Preamble (often padded with zeroes), + // * the 4 bytes of "DICM" string, + // * the 4 bytes of the first tag (0002, 0000), + // i.e. a total of 136 bytes. + entCur = deb + 136; + if(memcmp(entCur, "UL", (size_t)2) == 0) { + filetype = ExplicitVR; + dbg.Verbose(1, "gdcmHeader::CheckSwap:", + "explicit Value Representation"); + } else { + filetype = ImplicitVR; + dbg.Verbose(1, "gdcmHeader::CheckSwap:", + "not an explicit Value Representation"); + } + + if (net2host) { + sw = 4321; + dbg.Verbose(1, "gdcmHeader::CheckSwap:", + "HostByteOrder != NetworkByteOrder"); + } else { + sw = 0; + dbg.Verbose(1, "gdcmHeader::CheckSwap:", + "HostByteOrder = NetworkByteOrder"); + } + + // Position the file position indicator at first tag (i.e. + // after the file preamble and the "DICM" string). + rewind(fp); + fseek (fp, 132L, SEEK_SET); + return; + } // End of TrueDicom + + // Alas, this is not a DicomV3 file and whatever happens there is no file + // preamble. We can reset the file position indicator to where the data + // is (i.e. the beginning of the file). + rewind(fp); + + // Our next best chance would be to be considering a 'clean' ACR/NEMA file. + // By clean we mean that the length of the first tag is written down. + // If this is the case and since the length of the first group HAS to be + // four (bytes), then determining the proper swap code is straightforward. + + entCur = deb + 4; + // We assume the array of char we are considering contains the binary + // representation of a 32 bits integer. Hence the following dirty + // trick : + s = *((guint32 *)(entCur)); + + switch (s) { + case 0x00040000 : + sw = 3412; + filetype = ACR; + return; + case 0x04000000 : + sw = 4321; + filetype = ACR; + return; + case 0x00000400 : + sw = 2143; + filetype = ACR; + return; + case 0x00000004 : + sw = 0; + filetype = ACR; + return; + default : + dbg.Verbose(0, "gdcmHeader::CheckSwap:", + "ACR/NEMA unfound swap info (time to raise bets)"); + } + + // We are out of luck. It is not a DicomV3 nor a 'clean' ACR/NEMA file. + // It is time for despaired wild guesses. So, let's assume this file + // happens to be 'dirty' ACR/NEMA, i.e. the length of the group is + // not present. Then the only info we have is the net2host one. + if (! net2host ) + sw = 0; + else + sw = 4321; + return; } void gdcmHeader::SwitchSwapToBigEndian(void) { - dbg.Verbose(1, "gdcmHeader::SwitchSwapToBigEndian", - "Switching to BigEndian mode."); - if ( sw == 0 ) { - sw = 4321; - return; - } - if ( sw == 4321 ) { - sw = 0; - return; - } - if ( sw == 3412 ) { - sw = 2143; - return; - } - if ( sw == 2143 ) - sw = 3412; + dbg.Verbose(1, "gdcmHeader::SwitchSwapToBigEndian", + "Switching to BigEndian mode."); + if ( sw == 0 ) { + sw = 4321; + return; + } + if ( sw == 4321 ) { + sw = 0; + return; + } + if ( sw == 3412 ) { + sw = 2143; + return; + } + if ( sw == 2143 ) + sw = 3412; } void gdcmHeader::GetPixels(size_t lgrTotale, void* _Pixels) { - size_t pixelsOffset; - pixelsOffset = GetPixelOffset(); - fseek(fp, pixelsOffset, SEEK_SET); - fread(_Pixels, 1, lgrTotale, fp); + size_t pixelsOffset; + pixelsOffset = GetPixelOffset(); + fseek(fp, pixelsOffset, SEEK_SET); + fread(_Pixels, 1, lgrTotale, fp); } @@ -256,94 +249,81 @@ void gdcmHeader::GetPixels(size_t lgrTotale, void* _Pixels) { /** * \ingroup gdcmHeader * \brief Find the value representation of the current tag. - * - * @param sw code swap - * @param skippedLength pointeur sur nombre d'octets que l'on a saute qd - * la lecture est finie - * @param longueurLue pointeur sur longueur (en nombre d'octets) - * effectivement lue - * @return longueur retenue pour le champ */ - -// --> -// --> Oops -// --> C'etait la description de quoi, ca? -// --> - void gdcmHeader::FindVR( ElValue *ElVal) { - if (filetype != ExplicitVR) - return; - - char VR[3]; - string vr; - int lgrLue; - long PositionOnEntry = ftell(fp); - // Warning: we believe this is explicit VR (Value Representation) because - // we used a heuristic that found "UL" in the first tag. Alas this - // doesn't guarantee that all the tags will be in explicit VR. In some - // cases (see e-film filtered files) one finds implicit VR tags mixed - // within an explicit VR file. Hence we make sure the present tag - // is in explicit VR and try to fix things if it happens not to be - // the case. - bool RealExplicit = true; - - lgrLue=fread (&VR, (size_t)2,(size_t)1, fp); - VR[2]=0; - vr = string(VR); - - // Assume we are reading a falsely explicit VR file i.e. we reached - // a tag where we expect reading a VR but are in fact we read the - // first to bytes of the length. Then we will interogate (through find) - // the dicom_vr dictionary with oddities like "\004\0" which crashes - // both GCC and VC++ implementations of the STL map. Hence when the - // expected VR read happens to be non-ascii characters we consider - // we hit falsely explicit VR tag. - - if ( (!isalpha(VR[0])) && (!isalpha(VR[1])) ) - RealExplicit = false; - - // CLEANME searching the dicom_vr at each occurence is expensive. - // PostPone this test in an optional integrity check at the end - // of parsing or only in debug mode. - if ( RealExplicit && !dicom_vr->count(vr) ) - RealExplicit = false; - - if ( RealExplicit ) { - if ( ElVal->IsVrUnknown() ) { - // When not a dictionary entry, we can safely overwrite the vr. - ElVal->SetVR(vr); - return; - } - if ( ElVal->GetVR() == vr ) { - // The vr we just read and the dictionary agree. Nothing to do. - return; - } - // The vr present in the file and the dictionary disagree. We assume - // the file writer knew best and use the vr of the file. Since it would - // be unwise to overwrite the vr of a dictionary (since it would - // compromise it's next user), we need to clone the actual DictEntry - // and change the vr for the read one. - gdcmDictEntry* NewTag = new gdcmDictEntry(ElVal->GetGroup(), - ElVal->GetElement(), - vr, - "FIXME", - ElVal->GetName()); - ElVal->SetDictEntry(NewTag); - return; - } - - // We thought this was explicit VR, but we end up with an - // implicit VR tag. Let's backtrack. - dbg.Verbose(1, "gdcmHeader::FindVR:", "Falsely explicit vr file"); - fseek(fp, PositionOnEntry, SEEK_SET); - // When this element is known in the dictionary we shall use, e.g. for - // the semantics (see the usage of IsAnInteger), the vr proposed by the - // dictionary entry. Still we have to flag the element as implicit since - // we know now our assumption on expliciteness is not furfilled. - // avoid . - if ( ElVal->IsVrUnknown() ) - ElVal->SetVR("Implicit"); - ElVal->SetImplicitVr(); + if (filetype != ExplicitVR) + return; + + char VR[3]; + string vr; + int lgrLue; + long PositionOnEntry = ftell(fp); + // Warning: we believe this is explicit VR (Value Representation) because + // we used a heuristic that found "UL" in the first tag. Alas this + // doesn't guarantee that all the tags will be in explicit VR. In some + // cases (see e-film filtered files) one finds implicit VR tags mixed + // within an explicit VR file. Hence we make sure the present tag + // is in explicit VR and try to fix things if it happens not to be + // the case. + bool RealExplicit = true; + + lgrLue=fread (&VR, (size_t)2,(size_t)1, fp); + VR[2]=0; + vr = string(VR); + + // Assume we are reading a falsely explicit VR file i.e. we reached + // a tag where we expect reading a VR but are in fact we read the + // first to bytes of the length. Then we will interogate (through find) + // the dicom_vr dictionary with oddities like "\004\0" which crashes + // both GCC and VC++ implementations of the STL map. Hence when the + // expected VR read happens to be non-ascii characters we consider + // we hit falsely explicit VR tag. + + if ( (!isalpha(VR[0])) && (!isalpha(VR[1])) ) + RealExplicit = false; + + // CLEANME searching the dicom_vr at each occurence is expensive. + // PostPone this test in an optional integrity check at the end + // of parsing or only in debug mode. + if ( RealExplicit && !dicom_vr->count(vr) ) + RealExplicit = false; + + if ( RealExplicit ) { + if ( ElVal->IsVrUnknown() ) { + // When not a dictionary entry, we can safely overwrite the vr. + ElVal->SetVR(vr); + return; + } + if ( ElVal->GetVR() == vr ) { + // The vr we just read and the dictionary agree. Nothing to do. + return; + } + // The vr present in the file and the dictionary disagree. We assume + // the file writer knew best and use the vr of the file. Since it would + // be unwise to overwrite the vr of a dictionary (since it would + // compromise it's next user), we need to clone the actual DictEntry + // and change the vr for the read one. + gdcmDictEntry* NewTag = new gdcmDictEntry(ElVal->GetGroup(), + ElVal->GetElement(), + vr, + "FIXME", + ElVal->GetName()); + ElVal->SetDictEntry(NewTag); + return; + } + + // We thought this was explicit VR, but we end up with an + // implicit VR tag. Let's backtrack. + dbg.Verbose(1, "gdcmHeader::FindVR:", "Falsely explicit vr file"); + fseek(fp, PositionOnEntry, SEEK_SET); + // When this element is known in the dictionary we shall use, e.g. for + // the semantics (see the usage of IsAnInteger), the vr proposed by the + // dictionary entry. Still we have to flag the element as implicit since + // we know now our assumption on expliciteness is not furfilled. + // avoid . + if ( ElVal->IsVrUnknown() ) + ElVal->SetVR("Implicit"); + ElVal->SetImplicitVr(); } /** @@ -354,14 +334,14 @@ void gdcmHeader::FindVR( ElValue *ElVal) { * @return True when ImplicitVRLittleEndian found. False in all other cases. */ bool gdcmHeader::IsImplicitVRLittleEndianTransferSyntax(void) { - ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); - if ( !Element ) - return false; - LoadElementValueSafe(Element); - string Transfer = Element->GetValue(); - if ( Transfer == "1.2.840.10008.1.2" ) - return true; - return false; + ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); + if ( !Element ) + return false; + LoadElementValueSafe(Element); + string Transfer = Element->GetValue(); + if ( Transfer == "1.2.840.10008.1.2" ) + return true; + return false; } /** @@ -372,14 +352,14 @@ bool gdcmHeader::IsImplicitVRLittleEndianTransferSyntax(void) { * @return True when ExplicitVRLittleEndian found. False in all other cases. */ bool gdcmHeader::IsExplicitVRLittleEndianTransferSyntax(void) { - ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); - if ( !Element ) - return false; - LoadElementValueSafe(Element); - string Transfer = Element->GetValue(); - if ( Transfer == "1.2.840.10008.1.2.1" ) - return true; - return false; + ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); + if ( !Element ) + return false; + LoadElementValueSafe(Element); + string Transfer = Element->GetValue(); + if ( Transfer == "1.2.840.10008.1.2.1" ) + return true; + return false; } /** @@ -390,17 +370,16 @@ bool gdcmHeader::IsExplicitVRLittleEndianTransferSyntax(void) { * @return True when DeflatedExplicitVRLittleEndian found. False in all other cases. */ bool gdcmHeader::IsDeflatedExplicitVRLittleEndianTransferSyntax(void) { - ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); - if ( !Element ) - return false; - LoadElementValueSafe(Element); - string Transfer = Element->GetValue(); - if ( Transfer == "1.2.840.10008.1.2.1.99" ) - return true; - return false; + ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); + if ( !Element ) + return false; + LoadElementValueSafe(Element); + string Transfer = Element->GetValue(); + if ( Transfer == "1.2.840.10008.1.2.1.99" ) + return true; + return false; } - /** * \ingroup gdcmHeader * \brief Determines if the Transfer Syntax was allready encountered @@ -409,17 +388,16 @@ bool gdcmHeader::IsDeflatedExplicitVRLittleEndianTransferSyntax(void) { * @return True when big endian found. False in all other cases. */ bool gdcmHeader::IsExplicitVRBigEndianTransferSyntax(void) { - ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); - if ( !Element ) - return false; - LoadElementValueSafe(Element); - string Transfer = Element->GetValue(); - if ( Transfer == "1.2.840.10008.1.2.2" ) - return true; - return false; + ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); + if ( !Element ) + return false; + LoadElementValueSafe(Element); + string Transfer = Element->GetValue(); + if ( Transfer == "1.2.840.10008.1.2.2" ) + return true; + return false; } - /** * \ingroup gdcmHeader * \brief Determines if the Transfer Syntax was allready encountered @@ -428,14 +406,14 @@ bool gdcmHeader::IsExplicitVRBigEndianTransferSyntax(void) { * @return True when JPEGBaseLineProcess1found. False in all other cases. */ bool gdcmHeader::IsJPEGBaseLineProcess1TransferSyntax(void) { - ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); - if ( !Element ) - return false; - LoadElementValueSafe(Element); - string Transfer = Element->GetValue(); - if ( Transfer == "1.2.840.10008.1.2.4.50" ) - return true; - return false; + ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); + if ( !Element ) + return false; + LoadElementValueSafe(Element); + string Transfer = Element->GetValue(); + if ( Transfer == "1.2.840.10008.1.2.4.50" ) + return true; + return false; } /** @@ -446,17 +424,16 @@ bool gdcmHeader::IsJPEGBaseLineProcess1TransferSyntax(void) { * @return True when JPEGExtendedProcess2-4 found. False in all other cases. */ bool gdcmHeader::IsJPEGExtendedProcess2_4TransferSyntax(void) { - ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); - if ( !Element ) - return false; - LoadElementValueSafe(Element); - string Transfer = Element->GetValue(); - if ( Transfer == "1.2.840.10008.1.2.4.51" ) - return true; - return false; + ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); + if ( !Element ) + return false; + LoadElementValueSafe(Element); + string Transfer = Element->GetValue(); + if ( Transfer == "1.2.840.10008.1.2.4.51" ) + return true; + return false; } - /** * \ingroup gdcmHeader * \brief Determines if the Transfer Syntax was allready encountered @@ -465,14 +442,14 @@ bool gdcmHeader::IsJPEGExtendedProcess2_4TransferSyntax(void) { * @return True when JPEGExtendedProcess3-5 found. False in all other cases. */ bool gdcmHeader::IsJPEGExtendedProcess3_5TransferSyntax(void) { - ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); - if ( !Element ) - return false; - LoadElementValueSafe(Element); - string Transfer = Element->GetValue(); - if ( Transfer == "1.2.840.10008.1.2.4.52" ) - return true; - return false; + ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); + if ( !Element ) + return false; + LoadElementValueSafe(Element); + string Transfer = Element->GetValue(); + if ( Transfer == "1.2.840.10008.1.2.4.52" ) + return true; + return false; } /** @@ -480,154 +457,160 @@ bool gdcmHeader::IsJPEGExtendedProcess3_5TransferSyntax(void) { * \brief Determines if the Transfer Syntax was allready encountered * and if it corresponds to a JPEGSpectralSelectionProcess6-8 one. * - * @return True when JPEGSpectralSelectionProcess6-8 found. False in all other cases. + * @return True when JPEGSpectralSelectionProcess6-8 found. False in all + * other cases. */ bool gdcmHeader::IsJPEGSpectralSelectionProcess6_8TransferSyntax(void) { - ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); - if ( !Element ) - return false; - LoadElementValueSafe(Element); - string Transfer = Element->GetValue(); - if ( Transfer == "1.2.840.10008.1.2.4.53" ) - return true; - return false; + ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010); + if ( !Element ) + return false; + LoadElementValueSafe(Element); + string Transfer = Element->GetValue(); + if ( Transfer == "1.2.840.10008.1.2.4.53" ) + return true; + return false; } -// -// Euhhhhhhh -// Il y en a encore DIX-SEPT, comme ça. -// Il faudrait trouver qq chose + rusé ... -// -// --> probablement TOUS les supprimer (Eric dixit) -// +/** + * \ingroup gdcmHeader + * \brief When the length of an element value is obviously wrong (because + * the parser went Jabberwocky) one can hope improving things by + * applying this heuristic. + */ void gdcmHeader::FixFoundLength(ElValue * ElVal, guint32 FoundLength) { - // Heuristic: a final fix. - if ( FoundLength == 0xffffffff) - FoundLength = 0; - ElVal->SetLength(FoundLength); + if ( FoundLength == 0xffffffff) + FoundLength = 0; + ElVal->SetLength(FoundLength); } guint32 gdcmHeader::FindLengthOB(void) { - // See PS 3.5-2001, section A.4 p. 49 on encapsulation of encoded pixel data. - guint16 g; - guint16 n; - long PositionOnEntry = ftell(fp); - bool FoundSequenceDelimiter = false; - guint32 TotalLength = 0; - guint32 ItemLength; - - while ( ! FoundSequenceDelimiter) { - g = ReadInt16(); - n = ReadInt16(); - TotalLength += 4; // We even have to decount the group and element - if ( g != 0xfffe ) { - dbg.Verbose(1, "gdcmHeader::FindLengthOB: ", - "wrong group for an item sequence."); - throw Error::FileReadError(fp, "gdcmHeader::FindLengthOB"); - } - if ( n == 0xe0dd ) - FoundSequenceDelimiter = true; - else if ( n != 0xe000) { - dbg.Verbose(1, "gdcmHeader::FindLengthOB: ", - "wrong element for an item sequence."); - throw Error::FileReadError(fp, "gdcmHeader::FindLengthOB"); - } - ItemLength = ReadInt32(); - TotalLength += ItemLength + 4; // We add 4 bytes since we just read - // the ItemLength with ReadInt32 - SkipBytes(ItemLength); - } - fseek(fp, PositionOnEntry, SEEK_SET); - return TotalLength; + // See PS 3.5-2001, section A.4 p. 49 on encapsulation of encoded pixel data. + guint16 g; + guint16 n; + long PositionOnEntry = ftell(fp); + bool FoundSequenceDelimiter = false; + guint32 TotalLength = 0; + guint32 ItemLength; + + while ( ! FoundSequenceDelimiter) { + g = ReadInt16(); + n = ReadInt16(); + if (errno == 1) + return 0; + TotalLength += 4; // We even have to decount the group and element + if ( g != 0xfffe ) { + dbg.Verbose(1, "gdcmHeader::FindLengthOB: ", + "wrong group for an item sequence."); + errno = 1; + return 0; + } + if ( n == 0xe0dd ) + FoundSequenceDelimiter = true; + else if ( n != 0xe000) { + dbg.Verbose(1, "gdcmHeader::FindLengthOB: ", + "wrong element for an item sequence."); + errno = 1; + return 0; + } + ItemLength = ReadInt32(); + TotalLength += ItemLength + 4; // We add 4 bytes since we just read + // the ItemLength with ReadInt32 + SkipBytes(ItemLength); + } + fseek(fp, PositionOnEntry, SEEK_SET); + return TotalLength; } void gdcmHeader::FindLength(ElValue * ElVal) { - guint16 element = ElVal->GetElement(); - string vr = ElVal->GetVR(); - guint16 length16; - - if ( (filetype == ExplicitVR) && ! ElVal->IsImplicitVr() ) { - - if ( (vr=="OB") || (vr=="OW") || (vr=="SQ") || (vr=="UN") ) { - // The following reserved two bytes (see PS 3.5-2001, section - // 7.1.2 Data element structure with explicit vr p27) must be - // skipped before proceeding on reading the length on 4 bytes. - fseek(fp, 2L, SEEK_CUR); - guint32 length32 = ReadInt32(); - if ( (vr == "OB") && (length32 == 0xffffffff) ) { - ElVal->SetLength(FindLengthOB()); - return; - } - FixFoundLength(ElVal, length32); - return; - } - - // Length is encoded on 2 bytes. - length16 = ReadInt16(); - - // We can tell the current file is encoded in big endian (like - // Data/US-RGB-8-epicard) when we find the "Transfer Syntax" tag - // and it's value is the one of the encoding of a big endian file. - // In order to deal with such big endian encoded files, we have - // (at least) two strategies: - // * when we load the "Transfer Syntax" tag with value of big endian - // encoding, we raise the proper flags. Then we wait for the end - // of the META group (0x0002) among which is "Transfer Syntax", - // before switching the swap code to big endian. We have to postpone - // the switching of the swap code since the META group is fully encoded - // in little endian, and big endian coding only starts at the next - // group. The corresponding code can be hard to analyse and adds - // many additional unnecessary tests for regular tags. - // * the second strategy consists in waiting for trouble, that shall appear - // when we find the first group with big endian encoding. This is - // easy to detect since the length of a "Group Length" tag (the - // ones with zero as element number) has to be of 4 (0x0004). When we - // encouter 1024 (0x0400) chances are the encoding changed and we - // found a group with big endian encoding. - // We shall use this second strategy. In order make sure that we - // can interpret the presence of an apparently big endian encoded - // length of a "Group Length" without committing a big mistake, we - // add an additional check: we look in the allready parsed elements - // for the presence of a "Transfer Syntax" whose value has to be "big - // endian encoding". When this is the case, chances are we got our - // hands on a big endian encoded file: we switch the swap code to - // big endian and proceed... - if ( (element == 0x000) && (length16 == 0x0400) ) { - if ( ! IsExplicitVRBigEndianTransferSyntax() ) - throw Error::FileReadError(fp, "gdcmHeader::FindLength"); - length16 = 4; - SwitchSwapToBigEndian(); - // Restore the unproperly loaded values i.e. the group, the element - // and the dictionary entry depending on them. - guint16 CorrectGroup = SwapShort(ElVal->GetGroup()); - guint16 CorrectElem = SwapShort(ElVal->GetElement()); - gdcmDictEntry * NewTag = IsInDicts(CorrectGroup, CorrectElem); - if (!NewTag) { - // This correct tag is not in the dictionary. Create a new one. - NewTag = new gdcmDictEntry(CorrectGroup, CorrectElem); - } - // FIXME this can create a memory leaks on the old entry that be - // left unreferenced. - ElVal->SetDictEntry(NewTag); - } - - // Heuristic: well some files are really ill-formed. - if ( length16 == 0xffff) { - length16 = 0; - dbg.Verbose(0, "gdcmHeader::FindLength", - "Erroneous element length fixed."); - } - FixFoundLength(ElVal, (guint32)length16); - return; - } - - // Either implicit VR or a non DICOM conformal (see not below) explicit - // VR that ommited the VR of (at least) this element. Farts happen. - // [Note: according to the part 5, PS 3.5-2001, section 7.1 p25 - // on Data elements "Implicit and Explicit VR Data Elements shall - // not coexist in a Data Set and Data Sets nested within it".] - // Length is on 4 bytes. - FixFoundLength(ElVal, ReadInt32()); + guint16 element = ElVal->GetElement(); + string vr = ElVal->GetVR(); + guint16 length16; + + if ( (filetype == ExplicitVR) && ! ElVal->IsImplicitVr() ) { + + if ( (vr=="OB") || (vr=="OW") || (vr=="SQ") || (vr=="UN") ) { + // The following reserved two bytes (see PS 3.5-2001, section + // 7.1.2 Data element structure with explicit vr p27) must be + // skipped before proceeding on reading the length on 4 bytes. + fseek(fp, 2L, SEEK_CUR); + guint32 length32 = ReadInt32(); + if ( (vr == "OB") && (length32 == 0xffffffff) ) { + ElVal->SetLength(FindLengthOB()); + return; + } + FixFoundLength(ElVal, length32); + return; + } + + // Length is encoded on 2 bytes. + length16 = ReadInt16(); + + // We can tell the current file is encoded in big endian (like + // Data/US-RGB-8-epicard) when we find the "Transfer Syntax" tag + // and it's value is the one of the encoding of a big endian file. + // In order to deal with such big endian encoded files, we have + // (at least) two strategies: + // * when we load the "Transfer Syntax" tag with value of big endian + // encoding, we raise the proper flags. Then we wait for the end + // of the META group (0x0002) among which is "Transfer Syntax", + // before switching the swap code to big endian. We have to postpone + // the switching of the swap code since the META group is fully encoded + // in little endian, and big endian coding only starts at the next + // group. The corresponding code can be hard to analyse and adds + // many additional unnecessary tests for regular tags. + // * the second strategy consists in waiting for trouble, that shall + // appear when we find the first group with big endian encoding. This + // is easy to detect since the length of a "Group Length" tag (the + // ones with zero as element number) has to be of 4 (0x0004). When we + // encouter 1024 (0x0400) chances are the encoding changed and we + // found a group with big endian encoding. + // We shall use this second strategy. In order to make sure that we + // can interpret the presence of an apparently big endian encoded + // length of a "Group Length" without committing a big mistake, we + // add an additional check: we look in the allready parsed elements + // for the presence of a "Transfer Syntax" whose value has to be "big + // endian encoding". When this is the case, chances are we have got our + // hands on a big endian encoded file: we switch the swap code to + // big endian and proceed... + if ( (element == 0x000) && (length16 == 0x0400) ) { + if ( ! IsExplicitVRBigEndianTransferSyntax() ) { + dbg.Verbose(0, "gdcmHeader::FindLength", "not explicit VR"); + errno = 1; + return; + } + length16 = 4; + SwitchSwapToBigEndian(); + // Restore the unproperly loaded values i.e. the group, the element + // and the dictionary entry depending on them. + guint16 CorrectGroup = SwapShort(ElVal->GetGroup()); + guint16 CorrectElem = SwapShort(ElVal->GetElement()); + gdcmDictEntry * NewTag = GetDictEntryByKey(CorrectGroup, CorrectElem); + if (!NewTag) { + // This correct tag is not in the dictionary. Create a new one. + NewTag = new gdcmDictEntry(CorrectGroup, CorrectElem); + } + // FIXME this can create a memory leaks on the old entry that be + // left unreferenced. + ElVal->SetDictEntry(NewTag); + } + + // Heuristic: well some files are really ill-formed. + if ( length16 == 0xffff) { + length16 = 0; + dbg.Verbose(0, "gdcmHeader::FindLength", + "Erroneous element length fixed."); + } + FixFoundLength(ElVal, (guint32)length16); + return; + } + + // Either implicit VR or a non DICOM conformal (see not below) explicit + // VR that ommited the VR of (at least) this element. Farts happen. + // [Note: according to the part 5, PS 3.5-2001, section 7.1 p25 + // on Data elements "Implicit and Explicit VR Data Elements shall + // not coexist in a Data Set and Data Sets nested within it".] + // Length is on 4 bytes. + FixFoundLength(ElVal, ReadInt32()); } /** @@ -638,27 +621,27 @@ void gdcmHeader::FindLength(ElValue * ElVal) { * @return The suggested integer. */ guint32 gdcmHeader::SwapLong(guint32 a) { - // FIXME: il pourrait y avoir un pb pour les entiers negatifs ... - switch (sw) { - case 0 : - break; - case 4321 : - a=( ((a<<24) & 0xff000000) | ((a<<8) & 0x00ff0000) | - ((a>>8) & 0x0000ff00) | ((a>>24) & 0x000000ff) ); - break; - - case 3412 : - a=( ((a<<16) & 0xffff0000) | ((a>>16) & 0x0000ffff) ); - break; - - case 2143 : - a=( ((a<<8) & 0xff00ff00) | ((a>>8) & 0x00ff00ff) ); - break; - default : - dbg.Error(" gdcmHeader::SwapLong : unset swap code"); - a=0; - } - return(a); + // FIXME: il pourrait y avoir un pb pour les entiers negatifs ... + switch (sw) { + case 0 : + break; + case 4321 : + a=( ((a<<24) & 0xff000000) | ((a<<8) & 0x00ff0000) | + ((a>>8) & 0x0000ff00) | ((a>>24) & 0x000000ff) ); + break; + + case 3412 : + a=( ((a<<16) & 0xffff0000) | ((a>>16) & 0x0000ffff) ); + break; + + case 2143 : + a=( ((a<<8) & 0xff00ff00) | ((a>>8) & 0x00ff00ff) ); + break; + default : + dbg.Error(" gdcmHeader::SwapLong : unset swap code"); + a=0; + } + return(a); } /** @@ -667,138 +650,129 @@ guint32 gdcmHeader::SwapLong(guint32 a) { * @return The properly swaped 16 bits integer. */ guint16 gdcmHeader::SwapShort(guint16 a) { - if ( (sw==4321) || (sw==2143) ) - a =(((a<<8) & 0x0ff00) | ((a>>8)&0x00ff)); - return (a); + if ( (sw==4321) || (sw==2143) ) + a =(((a<<8) & 0x0ff00) | ((a>>8)&0x00ff)); + return (a); } void gdcmHeader::SkipBytes(guint32 NBytes) { - //FIXME don't dump the returned value - (void)fseek(fp, (long)NBytes, SEEK_CUR); + //FIXME don't dump the returned value + (void)fseek(fp, (long)NBytes, SEEK_CUR); } void gdcmHeader::SkipElementValue(ElValue * ElVal) { - SkipBytes(ElVal->GetLength()); + SkipBytes(ElVal->GetLength()); } void gdcmHeader::SetMaxSizeLoadElementValue(long NewSize) { - if (NewSize < 0) - return; - if ((guint32)NewSize >= (guint32)0xffffffff) { - MaxSizeLoadElementValue = 0xffffffff; - return; - } - MaxSizeLoadElementValue = NewSize; + if (NewSize < 0) + return; + if ((guint32)NewSize >= (guint32)0xffffffff) { + MaxSizeLoadElementValue = 0xffffffff; + return; + } + MaxSizeLoadElementValue = NewSize; } /** * \ingroup gdcmHeader - * \brief Loads the element if it's size is not to big. - * @param ElVal Element whose value shall be loaded. - * @param MaxSize Size treshold above which the element value is not - * loaded in memory. The element value is allways loaded - * when MaxSize is equal to UINT32_MAX. - * @return + * \brief Loads the element content if it's length is not bigger + * than the value specified with + * gdcmHeader::SetMaxSizeLoadElementValue() */ void gdcmHeader::LoadElementValue(ElValue * ElVal) { - size_t item_read; - guint16 group = ElVal->GetGroup(); - guint16 elem = ElVal->GetElement(); - string vr = ElVal->GetVR(); - guint32 length = ElVal->GetLength(); - bool SkipLoad = false; - - fseek(fp, (long)ElVal->GetOffset(), SEEK_SET); - - // Sequences not treated yet ! - // - // Ne faudrait-il pas au contraire trouver immediatement - // une maniere 'propre' de traiter les sequences (vr = SQ) - // car commencer par les ignorer risque de conduire a qq chose - // qui pourrait ne pas etre generalisable - // - if( vr == "SQ" ) - SkipLoad = true; - - // Heuristic : a sequence "contains" a set of tags (called items). It looks - // like the last tag of a sequence (the one that terminates the sequence) - // has a group of 0xfffe (with a dummy length). - if( group == 0xfffe ) - SkipLoad = true; - - // The group length doesn't represent data to be loaded in memory, since - // each element of the group shall be loaded individualy. - if( elem == 0 ) - //SkipLoad = true; // modif sauvage JPR - - if ( SkipLoad ) { - // FIXME the following skip is not necessary - SkipElementValue(ElVal); - ElVal->SetLength(0); - ElVal->SetValue("gdcm::Skipped"); - return; - } - - // When the length is zero things are easy: - if ( length == 0 ) { - ElVal->SetValue(""); - return; - } - - // Values bigger than specified are not loaded. - // - // En fait, c'est les elements dont la longueur est superieure - // a celle fixee qui ne sont pas charges - // - if (length > MaxSizeLoadElementValue) { - ostringstream s; - s << "gdcm::NotLoaded."; - s << " Address:" << (long)ElVal->GetOffset(); - s << " Length:" << ElVal->GetLength(); - //mesg += " Length:" + ElVal->GetLength(); - ElVal->SetValue(s.str()); - return; - } - - // When an integer is expected, read and convert the following two or - // four bytes properly i.e. as an integer as opposed to a string. - if ( IsAnInteger(ElVal) ) { - guint32 NewInt; - if( length == 2 ) { - NewInt = ReadInt16(); - } else if( length == 4 ) { - NewInt = ReadInt32(); - } else - dbg.Error(true, "LoadElementValue: Inconsistency when reading Int."); - - //FIXME: make the following an util fonction - ostringstream s; - s << NewInt; - ElVal->SetValue(s.str()); - return; - } - - // FIXME The exact size should be length if we move to strings or whatever - - // - // QUESTION : y a-t-il une raison pour ne pas utiliser g_malloc ici ? - // - - char* NewValue = (char*)malloc(length+1); - if( !NewValue) { - dbg.Verbose(1, "LoadElementValue: Failed to allocate NewValue"); - return; - } - NewValue[length]= 0; - - item_read = fread(NewValue, (size_t)length, (size_t)1, fp); - if ( item_read != 1 ) { - free(NewValue); - Error::FileReadError(fp, "gdcmHeader::LoadElementValue"); - ElVal->SetValue("gdcm::UnRead"); - return; - } - ElVal->SetValue(NewValue); + size_t item_read; + guint16 group = ElVal->GetGroup(); + guint16 elem = ElVal->GetElement(); + string vr = ElVal->GetVR(); + guint32 length = ElVal->GetLength(); + bool SkipLoad = false; + + fseek(fp, (long)ElVal->GetOffset(), SEEK_SET); + + // Sequences not treated yet ! + // + // Ne faudrait-il pas au contraire trouver immediatement + // une maniere 'propre' de traiter les sequences (vr = SQ) + // car commencer par les ignorer risque de conduire a qq chose + // qui pourrait ne pas etre generalisable + // + if( vr == "SQ" ) + SkipLoad = true; + + // Heuristic : a sequence "contains" a set of tags (called items). It looks + // like the last tag of a sequence (the one that terminates the sequence) + // has a group of 0xfffe (with a dummy length). + if( group == 0xfffe ) + SkipLoad = true; + + // The group length doesn't represent data to be loaded in memory, since + // each element of the group shall be loaded individualy. + if( elem == 0 ) + //SkipLoad = true; // modif sauvage JPR + // On charge la longueur du groupe + // quand l'element 0x0000 est présent ! + + if ( SkipLoad ) { + // FIXME the following skip is not necessary + SkipElementValue(ElVal); + ElVal->SetLength(0); + ElVal->SetValue("gdcm::Skipped"); + return; + } + + // When the length is zero things are easy: + if ( length == 0 ) { + ElVal->SetValue(""); + return; + } + + // The elements whose length is bigger than the specified upper bound + // are not loaded. Instead we leave a short notice of the offset of + // the element content and it's length. + if (length > MaxSizeLoadElementValue) { + ostringstream s; + s << "gdcm::NotLoaded."; + s << " Address:" << (long)ElVal->GetOffset(); + s << " Length:" << ElVal->GetLength(); + ElVal->SetValue(s.str()); + return; + } + + // When an integer is expected, read and convert the following two or + // four bytes properly i.e. as an integer as opposed to a string. + if ( IsAnInteger(ElVal) ) { + guint32 NewInt; + if( length == 2 ) { + NewInt = ReadInt16(); + } else if( length == 4 ) { + NewInt = ReadInt32(); + } else + dbg.Error(true, "LoadElementValue: Inconsistency when reading Int."); + + //FIXME: make the following an util fonction + ostringstream s; + s << NewInt; + ElVal->SetValue(s.str()); + return; + } + + // FIXME The exact size should be length if we move to strings or whatever + char* NewValue = (char*)malloc(length+1); + if( !NewValue) { + dbg.Verbose(1, "LoadElementValue: Failed to allocate NewValue"); + return; + } + NewValue[length]= 0; + + item_read = fread(NewValue, (size_t)length, (size_t)1, fp); + if ( item_read != 1 ) { + free(NewValue); + dbg.Verbose(1, "gdcmHeader::LoadElementValue","unread element value"); + ElVal->SetValue("gdcm::UnRead"); + return; + } + ElVal->SetValue(NewValue); } /** @@ -810,129 +784,159 @@ void gdcmHeader::LoadElementValue(ElValue * ElVal) { * @return */ void gdcmHeader::LoadElementValueSafe(ElValue * ElVal) { - long PositionOnEntry = ftell(fp); - LoadElementValue(ElVal); - fseek(fp, PositionOnEntry, SEEK_SET); + long PositionOnEntry = ftell(fp); + LoadElementValue(ElVal); + fseek(fp, PositionOnEntry, SEEK_SET); } guint16 gdcmHeader::ReadInt16(void) { - guint16 g; - size_t item_read; - item_read = fread (&g, (size_t)2,(size_t)1, fp); - if ( item_read != 1 ) - throw Error::FileReadError(fp, "gdcmHeader::ReadInt16"); - g = SwapShort(g); - return g; + guint16 g; + size_t item_read; + item_read = fread (&g, (size_t)2,(size_t)1, fp); + errno = 0; + if ( item_read != 1 ) { + dbg.Verbose(1, "gdcmHeader::ReadInt16", " File read error"); + errno = 1; + return 0; + } + g = SwapShort(g); + return g; } guint32 gdcmHeader::ReadInt32(void) { - guint32 g; - size_t item_read; - item_read = fread (&g, (size_t)4,(size_t)1, fp); - if ( item_read != 1 ) - throw Error::FileReadError(fp, "gdcmHeader::ReadInt32"); - g = SwapLong(g); - return g; + guint32 g; + size_t item_read; + item_read = fread (&g, (size_t)4,(size_t)1, fp); + errno = 0; + if ( item_read != 1 ) { + dbg.Verbose(1, "gdcmHeader::ReadInt32", " File read error"); + errno = 1; + return 0; + } + g = SwapLong(g); + return g; } /** * \ingroup gdcmHeader - * \brief Read the next tag without loading it's value - * @return On succes the newly created ElValue, NULL on failure. + * \brief Build a new Element Value from all the low level arguments. + * Check for existence of dictionary entry, and build + * a default one when absent. + * @param Group group of the underlying DictEntry + * @param Elem element of the underlying DictEntry + */ +ElValue* gdcmHeader::NewElValueByKey(guint16 Group, guint16 Elem) { + // Find out if the tag we encountered is in the dictionaries: + gdcmDictEntry * NewTag = GetDictEntryByKey(Group, Elem); + if (!NewTag) + NewTag = new gdcmDictEntry(Group, Elem); + + ElValue* NewElVal = new ElValue(NewTag); + if (!NewElVal) { + dbg.Verbose(1, "gdcmHeader::NewElValueByKey", + "failed to allocate ElValue"); + return (ElValue*)0; + } + return NewElVal; +} + +/** + * \ingroup gdcmHeader + * \brief Build a new Element Value from all the low level arguments. + * Check for existence of dictionary entry, and build + * a default one when absent. + * @param Name Name of the underlying DictEntry */ +ElValue* gdcmHeader::NewElValueByName(string Name) { + + gdcmDictEntry * NewTag = GetDictEntryByName(Name); + if (!NewTag) + NewTag = new gdcmDictEntry(0xffff, 0xffff, "LO", "Unknown", Name); + ElValue* NewElVal = new ElValue(NewTag); + if (!NewElVal) { + dbg.Verbose(1, "gdcmHeader::ObtainElValueByName", + "failed to allocate ElValue"); + return (ElValue*)0; + } + return NewElVal; +} + +/** + * \ingroup gdcmHeader + * \brief Read the next tag but WITHOUT loading it's value + * @return On succes the newly created ElValue, NULL on failure. + */ ElValue * gdcmHeader::ReadNextElement(void) { - guint16 g; - guint16 n; - ElValue * NewElVal; - - try { - g = ReadInt16(); - n = ReadInt16(); - } - catch ( Error::FileReadError ) { - // We reached the EOF (or an error occured) and header parsing - // has to be considered as finished. - return (ElValue *)0; - } - - // Find out if the tag we encountered is in the dictionaries: - gdcmDictEntry * NewTag = IsInDicts(g, n); - if (!NewTag) - NewTag = new gdcmDictEntry(g, n); - - NewElVal = new ElValue(NewTag); - if (!NewElVal) { - dbg.Verbose(1, "ReadNextElement: failed to allocate ElValue"); - return (ElValue*)0; - } - - FindVR(NewElVal); - try { FindLength(NewElVal); } - catch ( Error::FileReadError ) { // Call it quits - return (ElValue *)0; - } - NewElVal->SetOffset(ftell(fp)); - return NewElVal; + guint16 g; + guint16 n; + ElValue * NewElVal; + + g = ReadInt16(); + n = ReadInt16(); + if (errno == 1) + // We reached the EOF (or an error occured) and header parsing + // has to be considered as finished. + return (ElValue *)0; + + NewElVal = NewElValueByKey(g, n); + FindVR(NewElVal); + FindLength(NewElVal); + if (errno == 1) + // Call it quits + return (ElValue *)0; + NewElVal->SetOffset(ftell(fp)); + return NewElVal; } +/** + * \ingroup gdcmHeader + * \brief Apply some heuristics to predict wether the considered + * element value contains/represents an integer or not. + * @param ElVal The element value on which to apply the predicate. + * @return The result of the heuristical predicate. + */ bool gdcmHeader::IsAnInteger(ElValue * ElVal) { - guint16 group = ElVal->GetGroup(); - guint16 element = ElVal->GetElement(); - string vr = ElVal->GetVR(); - guint32 length = ElVal->GetLength(); - - // When we have some semantics on the element we just read, and if we - // a priori know we are dealing with an integer, then we shall be - // able to swap it's element value properly. - if ( element == 0 ) { // This is the group length of the group - if (length == 4) - return true; - else - dbg.Error("gdcmHeader::IsAnInteger", - "Erroneous Group Length element length."); - } - - if ( group % 2 != 0 ) - // We only have some semantics on documented elements, which are - // the even ones. - return false; - - if ( (length != 4) && ( length != 2) ) - // Swapping only make sense on integers which are 2 or 4 bytes long. - return false; - - if ( (vr == "UL") || (vr == "US") || (vr == "SL") || (vr == "SS") ) - return true; - - if ( (group == 0x0028) && (element == 0x0005) ) - // This tag is retained from ACR/NEMA - // CHECKME Why should "Image Dimensions" be a single integer ? - // - // "Image Dimensions", c'est en fait le 'nombre de dimensions' - // de l'objet ACR-NEMA stocké - // 1 : Signal - // 2 : Image - // 3 : Volume - // 4 : Sequence - // - // DICOM V3 ne retient pas cette information - // Par defaut, tout est 'Image', - // C'est a l'utilisateur d'explorer l'ensemble des entetes - // pour savoir à quoi il a a faire - // - // Le Dicom Multiframe peut etre utilise pour stocker, - // dans un seul fichier, une serie temporelle (cardio vasculaire GE, p.ex) - // ou un volume (medecine Nucleaire, p.ex) - // - return true; - - if ( (group == 0x0028) && (element == 0x0200) ) - // This tag is retained from ACR/NEMA - return true; - - return false; + guint16 group = ElVal->GetGroup(); + guint16 element = ElVal->GetElement(); + string vr = ElVal->GetVR(); + guint32 length = ElVal->GetLength(); + + // When we have some semantics on the element we just read, and if we + // a priori know we are dealing with an integer, then we shall be + // able to swap it's element value properly. + if ( element == 0 ) { // This is the group length of the group + if (length == 4) + return true; + else + dbg.Error("gdcmHeader::IsAnInteger", + "Erroneous Group Length element length."); + } + + if ( group % 2 != 0 ) + // We only have some semantics on documented elements, which are + // the even ones. + return false; + + if ( (length != 4) && ( length != 2) ) + // Swapping only make sense on integers which are 2 or 4 bytes long. + return false; + + if ( (vr == "UL") || (vr == "US") || (vr == "SL") || (vr == "SS") ) + return true; + + if ( (group == 0x0028) && (element == 0x0005) ) + // The "Image Dimensions" tag is retained from ACR/NEMA and contains + // the number of dimensions of the contained object (1 for Signal, + // 2 for Image, 3 for Volume, 4 for Sequence). + return true; + + if ( (group == 0x0028) && (element == 0x0200) ) + // This tag is retained from ACR/NEMA + return true; + + return false; } /** @@ -940,213 +944,391 @@ bool gdcmHeader::IsAnInteger(ElValue * ElVal) { * \brief Recover the offset (from the beginning of the file) of the pixels. */ size_t gdcmHeader::GetPixelOffset(void) { - // If this file complies with the norm we should encounter the - // "Image Location" tag (0x0028, 0x0200). This tag contains the - // the group that contains the pixel data (hence the "Pixel Data" - // is found by indirection through the "Image Location"). - // Inside the group pointed by "Image Location" the searched element - // is conventionally the element 0x0010 (when the norm is respected). - // When the "Image Location" is absent we default to group 0x7fe0. - guint16 grPixel; - guint16 numPixel; - string ImageLocation = GetPubElValByName("Image Location"); - if ( ImageLocation == "gdcm::Unfound" ) { - grPixel = 0x7fe0; - } else { - grPixel = (guint16) atoi( ImageLocation.c_str() ); - } - if (grPixel != 0x7fe0) - // FIXME is this still necessary ? - // Now, this looks like an old dirty fix for Philips imager - numPixel = 0x1010; - else - numPixel = 0x0010; - ElValue* PixelElement = PubElVals.GetElementByNumber(grPixel, numPixel); - if (PixelElement) - return PixelElement->GetOffset(); - else - return 0; -} - -gdcmDictEntry * gdcmHeader::IsInDicts(guint32 group, guint32 element) { - // - // Y a-t-il une raison de lui passer des guint32 - // alors que group et element sont des guint16? - // - gdcmDictEntry * found = (gdcmDictEntry*)0; - if (!RefPubDict && !RefShaDict) { - //FIXME build a default dictionary ! - printf("FIXME in gdcmHeader::IsInDicts\n"); - } - if (RefPubDict) { - found = RefPubDict->GetTag(group, element); - if (found) - return found; - } - if (RefShaDict) { - found = RefShaDict->GetTag(group, element); - if (found) - return found; - } - return found; + // If this file complies with the norm we should encounter the + // "Image Location" tag (0x0028, 0x0200). This tag contains the + // the group that contains the pixel data (hence the "Pixel Data" + // is found by indirection through the "Image Location"). + // Inside the group pointed by "Image Location" the searched element + // is conventionally the element 0x0010 (when the norm is respected). + // When the "Image Location" is absent we default to group 0x7fe0. + guint16 grPixel; + guint16 numPixel; + string ImageLocation = GetPubElValByName("Image Location"); + if ( ImageLocation == "gdcm::Unfound" ) { + grPixel = 0x7fe0; + } else { + grPixel = (guint16) atoi( ImageLocation.c_str() ); + } + if (grPixel != 0x7fe0) + // FIXME is this still necessary ? + // Now, this looks like an old dirty fix for Philips imager + numPixel = 0x1010; + else + numPixel = 0x0010; + ElValue* PixelElement = PubElVals.GetElementByNumber(grPixel, numPixel); + if (PixelElement) + return PixelElement->GetOffset(); + else + return 0; } -list * gdcmHeader::GetPubTagNames(void) { - list * Result = new list; - TagHT entries = RefPubDict->GetEntries(); - - for (TagHT::iterator tag = entries.begin(); tag != entries.end(); ++tag){ - Result->push_back( tag->second->GetName() ); - } - return Result; +/** + * \ingroup gdcmHeader + * \brief Searches both the public and the shadow dictionary (when they + * exist) for the presence of the DictEntry with given + * group and element. The public dictionary has precedence on the + * shadow one. + * @param group group of the searched DictEntry + * @param element element of the searched DictEntry + * @return Corresponding DictEntry when it exists, NULL otherwise. + */ +gdcmDictEntry * gdcmHeader::GetDictEntryByKey(guint16 group, guint16 element) { + gdcmDictEntry * found = (gdcmDictEntry*)0; + if (!RefPubDict && !RefShaDict) { + dbg.Verbose(0, "FIXME in gdcmHeader::GetDictEntry", + "we SHOULD have a default dictionary"); + } + if (RefPubDict) { + found = RefPubDict->GetTagByKey(group, element); + if (found) + return found; + } + if (RefShaDict) { + found = RefShaDict->GetTagByKey(group, element); + if (found) + return found; + } + return found; } -map > * gdcmHeader::GetPubTagNamesByCategory(void) { - map > * Result = new map >; - TagHT entries = RefPubDict->GetEntries(); - - for (TagHT::iterator tag = entries.begin(); tag != entries.end(); ++tag){ - (*Result)[tag->second->GetFourth()].push_back(tag->second->GetName()); - } - return Result; +/** + * \ingroup gdcmHeader + * \brief Searches both the public and the shadow dictionary (when they + * exist) for the presence of the DictEntry with given name. + * The public dictionary has precedence on the shadow one. + * @param Name name of the searched DictEntry + * @return Corresponding DictEntry when it exists, NULL otherwise. + */ +gdcmDictEntry * gdcmHeader::GetDictEntryByName(string Name) { + gdcmDictEntry * found = (gdcmDictEntry*)0; + if (!RefPubDict && !RefShaDict) { + dbg.Verbose(0, "FIXME in gdcmHeader::GetDictEntry", + "we SHOULD have a default dictionary"); + } + if (RefPubDict) { + found = RefPubDict->GetTagByName(Name); + if (found) + return found; + } + if (RefShaDict) { + found = RefShaDict->GetTagByName(Name); + if (found) + return found; + } + return found; } +/** + * \ingroup gdcmHeader + * \brief Searches within the public dictionary for element value of + * a given tag. + * @param group Group of the researched tag. + * @param element Element of the researched tag. + * @return Corresponding element value when it exists, and the string + * "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetPubElValByNumber(guint16 group, guint16 element) { - return PubElVals.GetElValueByNumber(group, element); + return PubElVals.GetElValueByNumber(group, element); } +/** + * \ingroup gdcmHeader + * \brief Searches within the public dictionary for element value + * representation of a given tag. + * + * Obtaining the VR (Value Representation) might be needed by caller + * to convert the string typed content to caller's native type + * (think of C++ vs Python). The VR is actually of a higher level + * of semantics than just the native C++ type. + * @param group Group of the researched tag. + * @param element Element of the researched tag. + * @return Corresponding element value representation when it exists, + * and the string "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetPubElValRepByNumber(guint16 group, guint16 element) { - ElValue* elem = PubElVals.GetElementByNumber(group, element); - if ( !elem ) - return "gdcm::Unfound"; - return elem->GetVR(); + ElValue* elem = PubElVals.GetElementByNumber(group, element); + if ( !elem ) + return "gdcm::Unfound"; + return elem->GetVR(); } +/** + * \ingroup gdcmHeader + * \brief Searches within the public dictionary for element value of + * a given tag. + * @param TagName name of the researched element. + * @return Corresponding element value when it exists, and the string + * "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetPubElValByName(string TagName) { - return PubElVals.GetElValueByName(TagName); + return PubElVals.GetElValueByName(TagName); } +/** + * \ingroup gdcmHeader + * \brief Searches within the elements parsed with the public dictionary for + * the element value representation of a given tag. + * + * Obtaining the VR (Value Representation) might be needed by caller + * to convert the string typed content to caller's native type + * (think of C++ vs Python). The VR is actually of a higher level + * of semantics than just the native C++ type. + * @param TagName name of the researched element. + * @return Corresponding element value representation when it exists, + * and the string "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetPubElValRepByName(string TagName) { - ElValue* elem = PubElVals.GetElementByName(TagName); - if ( !elem ) - return "gdcm::Unfound"; - return elem->GetVR(); + ElValue* elem = PubElVals.GetElementByName(TagName); + if ( !elem ) + return "gdcm::Unfound"; + return elem->GetVR(); } +/** + * \ingroup gdcmHeader + * \brief Searches within elements parsed with the SHADOW dictionary + * for the element value of a given tag. + * @param group Group of the researched tag. + * @param element Element of the researched tag. + * @return Corresponding element value representation when it exists, + * and the string "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetShaElValByNumber(guint16 group, guint16 element) { - return ShaElVals.GetElValueByNumber(group, element); + return ShaElVals.GetElValueByNumber(group, element); } +/** + * \ingroup gdcmHeader + * \brief Searches within the elements parsed with the SHADOW dictionary + * for the element value representation of a given tag. + * + * Obtaining the VR (Value Representation) might be needed by caller + * to convert the string typed content to caller's native type + * (think of C++ vs Python). The VR is actually of a higher level + * of semantics than just the native C++ type. + * @param group Group of the researched tag. + * @param element Element of the researched tag. + * @return Corresponding element value representation when it exists, + * and the string "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetShaElValRepByNumber(guint16 group, guint16 element) { - ElValue* elem = ShaElVals.GetElementByNumber(group, element); - if ( !elem ) - return "gdcm::Unfound"; - return elem->GetVR(); + ElValue* elem = ShaElVals.GetElementByNumber(group, element); + if ( !elem ) + return "gdcm::Unfound"; + return elem->GetVR(); } +/** + * \ingroup gdcmHeader + * \brief Searches within the elements parsed with the shadow dictionary + * for an element value of given tag. + * @param TagName name of the researched element. + * @return Corresponding element value when it exists, and the string + * "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetShaElValByName(string TagName) { - return ShaElVals.GetElValueByName(TagName); + return ShaElVals.GetElValueByName(TagName); } +/** + * \ingroup gdcmHeader + * \brief Searches within the elements parsed with the shadow dictionary for + * the element value representation of a given tag. + * + * Obtaining the VR (Value Representation) might be needed by caller + * to convert the string typed content to caller's native type + * (think of C++ vs Python). The VR is actually of a higher level + * of semantics than just the native C++ type. + * @param TagName name of the researched element. + * @return Corresponding element value representation when it exists, + * and the string "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetShaElValRepByName(string TagName) { - ElValue* elem = ShaElVals.GetElementByName(TagName); - if ( !elem ) - return "gdcm::Unfound"; - return elem->GetVR(); + ElValue* elem = ShaElVals.GetElementByName(TagName); + if ( !elem ) + return "gdcm::Unfound"; + return elem->GetVR(); } - +/** + * \ingroup gdcmHeader + * \brief Searches within elements parsed with the public dictionary + * and then within the elements parsed with the shadow dictionary + * for the element value of a given tag. + * @param group Group of the researched tag. + * @param element Element of the researched tag. + * @return Corresponding element value representation when it exists, + * and the string "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetElValByNumber(guint16 group, guint16 element) { - string pub = GetPubElValByNumber(group, element); - if (pub.length()) - return pub; - return GetShaElValByNumber(group, element); + string pub = GetPubElValByNumber(group, element); + if (pub.length()) + return pub; + return GetShaElValByNumber(group, element); } +/** + * \ingroup gdcmHeader + * \brief Searches within elements parsed with the public dictionary + * and then within the elements parsed with the shadow dictionary + * for the element value representation of a given tag. + * + * Obtaining the VR (Value Representation) might be needed by caller + * to convert the string typed content to caller's native type + * (think of C++ vs Python). The VR is actually of a higher level + * of semantics than just the native C++ type. + * @param group Group of the researched tag. + * @param element Element of the researched tag. + * @return Corresponding element value representation when it exists, + * and the string "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetElValRepByNumber(guint16 group, guint16 element) { - string pub = GetPubElValRepByNumber(group, element); - if (pub.length()) - return pub; - return GetShaElValRepByNumber(group, element); + string pub = GetPubElValRepByNumber(group, element); + if (pub.length()) + return pub; + return GetShaElValRepByNumber(group, element); } +/** + * \ingroup gdcmHeader + * \brief Searches within elements parsed with the public dictionary + * and then within the elements parsed with the shadow dictionary + * for the element value of a given tag. + * @param TagName name of the researched element. + * @return Corresponding element value when it exists, + * and the string "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetElValByName(string TagName) { - string pub = GetPubElValByName(TagName); - if (pub.length()) - return pub; - return GetShaElValByName(TagName); + string pub = GetPubElValByName(TagName); + if (pub.length()) + return pub; + return GetShaElValByName(TagName); } +/** + * \ingroup gdcmHeader + * \brief Searches within elements parsed with the public dictionary + * and then within the elements parsed with the shadow dictionary + * for the element value representation of a given tag. + * + * Obtaining the VR (Value Representation) might be needed by caller + * to convert the string typed content to caller's native type + * (think of C++ vs Python). The VR is actually of a higher level + * of semantics than just the native C++ type. + * @param TagName name of the researched element. + * @return Corresponding element value representation when it exists, + * and the string "gdcm::Unfound" otherwise. + */ string gdcmHeader::GetElValRepByName(string TagName) { - string pub = GetPubElValRepByName(TagName); - if (pub.length()) - return pub; - return GetShaElValRepByName(TagName); + string pub = GetPubElValRepByName(TagName); + if (pub.length()) + return pub; + return GetShaElValRepByName(TagName); } /** * \ingroup gdcmHeader - * \brief Modifie la valeur d'un ElValue déja existant - * \ dans le PubElVals du gdcmHeader, - * \ accédé par ses numero de groupe et d'element. + * \brief Accesses an existing ElValue in the PubElVals of this instance + * through it's (group, element) and modifies it's content with + * the given value. + * @param content new value to substitute with + * @param group group of the ElVal to modify + * @param element element of the ElVal to modify */ -int gdcmHeader::SetPubElValByNumber(string content, guint16 group, guint16 element) { - //TagKey key = gdcmDictEntry::TranslateToKey(group, element); - //PubElVals.tagHt[key]->SetValue(content); - - return ( PubElVals.SetElValueByNumber (content, group, element) ); +int gdcmHeader::SetPubElValByNumber(string content, guint16 group, + guint16 element) +{ + return ( PubElVals.SetElValueByNumber (content, group, element) ); } - /** * \ingroup gdcmHeader - * \brief Modifie la valeur d'un ElValue déja existant - * \ dans le PubElVals du gdcmHeader, - * \ accédé par son nom + * \brief Accesses an existing ElValue in the PubElVals of this instance + * through tag name and modifies it's content with the given value. + * @param content new value to substitute with + * @param TagName name of the tag to be modified */ int gdcmHeader::SetPubElValByName(string content, string TagName) { - //TagKey key = gdcmDictEntry::TranslateToKey(group, element); - //PubElVals.tagHt[key]->SetValue(content); - - return ( PubElVals.SetElValueByName (content, TagName) ); + return ( PubElVals.SetElValueByName (content, TagName) ); } - /** * \ingroup gdcmHeader - * \brief Modifie la valeur d'un ElValue déja existant - * \ dans le ShaElVals du gdcmHeader, - * \ accédé par ses numero de groupe et d'element. + * \brief Accesses an existing ElValue in the ShaElVals of this instance + * through it's (group, element) and modifies it's content with + * the given value. + * @param content new value to substitute with + * @param group group of the ElVal to modify + * @param element element of the ElVal to modify */ -int gdcmHeader::SetShaElValByNumber(string content, guint16 group, guint16 element) { - - return ( ShaElVals.SetElValueByNumber (content, group, element) ); +int gdcmHeader::SetShaElValByNumber(string content, + guint16 group, guint16 element) +{ + return ( ShaElVals.SetElValueByNumber (content, group, element) ); } - /** * \ingroup gdcmHeader - * \brief Modifie la valeur d'un ElValue déja existant - * \ dans le ShaElVals du gdcmHeader, - * \ accédé par son nom + * \brief Accesses an existing ElValue in the ShaElVals of this instance + * through tag name and modifies it's content with the given value. + * @param content new value to substitute with + * @param TagName name of the tag to be modified */ int gdcmHeader::SetShaElValByName(string content, string TagName) { - - return ( ShaElVals.SetElValueByName (content, TagName) ); + return ( ShaElVals.SetElValueByName (content, TagName) ); +} + +/** + * \ingroup gdcmHeader + * \brief Parses the header of the file but WITHOUT loading element values. + */ +void gdcmHeader::ParseHeader(bool exception_on_error) throw(gdcmFormatError) { + ElValue * newElValue = (ElValue *)0; + + rewind(fp); + CheckSwap(); + while ( (newElValue = ReadNextElement()) ) { + SkipElementValue(newElValue); + PubElVals.Add(newElValue); + } } + /** * \ingroup gdcmHeader - * \brief Parses the header of the file but does NOT load element values. + * \brief Once the header is parsed add some gdcm convenience/helper elements + * in the ElValSet. For example add: + * - gdcmImageType which is an entry containing a short for the + * type of image and whose value ranges in + * I8 (unsigned 8 bit image) + * I16 (unsigned 8 bit image) + * IS16 (signed 8 bit image) + * - gdcmXsize, gdcmYsize, gdcmZsize whose values are respectively + * the ones of the official DICOM fields Rows, Columns and Planes. */ -void gdcmHeader::ParseHeader(void) { - ElValue * newElValue = (ElValue *)0; - - rewind(fp); - CheckSwap(); - while ( (newElValue = ReadNextElement()) ) { - SkipElementValue(newElValue); - PubElVals.Add(newElValue); - } +void gdcmHeader::AddAndDefaultElements(void) { + ElValue* NewEntry = (ElValue*)0; + + NewEntry = NewElValueByName("gdcmXSize"); + NewEntry->SetValue(GetElValByName("Rows")); + PubElVals.Add(NewEntry); + + NewEntry = NewElValueByName("gdcmYSize"); + NewEntry->SetValue(GetElValByName("Columns")); + PubElVals.Add(NewEntry); + + NewEntry = NewElValueByName("gdcmZSize"); + NewEntry->SetValue(GetElValByName("Planes")); + PubElVals.Add(NewEntry); } /** @@ -1155,20 +1337,17 @@ void gdcmHeader::ParseHeader(void) { * public tag based hash table. */ void gdcmHeader::LoadElements(void) { - - rewind(fp); - - TagElValueHT ht = PubElVals.GetTagHt(); - - for (TagElValueHT::iterator tag = ht.begin(); tag != ht.end(); ++tag) { - LoadElementValue(tag->second); - } + rewind(fp); + TagElValueHT ht = PubElVals.GetTagHt(); + for (TagElValueHT::iterator tag = ht.begin(); tag != ht.end(); ++tag) { + LoadElementValue(tag->second); + } } void gdcmHeader::PrintPubElVal(ostream & os) { - PubElVals.Print(os); + PubElVals.Print(os); } void gdcmHeader::PrintPubDict(ostream & os) { - RefPubDict->Print(os); + RefPubDict->Print(os); }