X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;ds=sidebyside;f=src%2FgdcmDocument.cxx;h=8937fa3ea97638ab36e1d647d066bc983c8867fd;hb=827d0d72b115d9e54aac8f0b054856dffff862ca;hp=313cee7ae760a6a61a9873a0415a5fb8a5c18bdb;hpb=9ca4fd0ccbfea12ed7760856de3a8faf41625869;p=gdcm.git diff --git a/src/gdcmDocument.cxx b/src/gdcmDocument.cxx index 313cee7a..8937fa3e 100644 --- a/src/gdcmDocument.cxx +++ b/src/gdcmDocument.cxx @@ -3,8 +3,8 @@ Program: gdcm Module: $RCSfile: gdcmDocument.cxx,v $ Language: C++ - Date: $Date: 2005/01/07 16:45:51 $ - Version: $Revision: 1.164 $ + Date: $Date: 2005/01/11 00:21:48 $ + Version: $Revision: 1.172 $ Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de l'Image). All rights reserved. See Doc/License.txt or @@ -47,7 +47,7 @@ namespace gdcm static const char *TransferSyntaxStrings[] = { // Implicit VR Little Endian "1.2.840.10008.1.2", - // Implicit VR Big Endian DLX G.E? + // Implicit VR Big Endian DLX (G.E Private) "1.2.840.113619.5.2", // Explicit VR Little Endian "1.2.840.10008.1.2.1", @@ -106,8 +106,7 @@ Document::Document( std::string const &filename ) : ElementSet(-1) return; } - Debug::Verbose(0, "Document::Document: starting parsing of file: ", - Filename.c_str()); + gdcmVerboseMacro( "Starting parsing of file: " << Filename.c_str()); Fp->seekg( 0, std::ios::beg); Fp->seekg(0, std::ios::end); @@ -124,7 +123,7 @@ Document::Document( std::string const &filename ) : ElementSet(-1) // Load 'non string' values - std::string PhotometricInterpretation = GetEntryByNumber(0x0028,0x0004); + std::string PhotometricInterpretation = GetEntry(0x0028,0x0004); if( PhotometricInterpretation == "PALETTE COLOR " ) { LoadEntryBinArea(0x0028,0x1200); // gray LUT @@ -170,16 +169,16 @@ Document::Document( std::string const &filename ) : ElementSet(-1) // we switch lineNumber and columnNumber // std::string RecCode; - RecCode = GetEntryByNumber(0x0008, 0x0010); // recognition code + RecCode = GetEntry(0x0008, 0x0010); // recognition code if (RecCode == "ACRNEMA_LIBIDO_1.1" || RecCode == "CANRME_AILIBOD1_1." ) // for brain-damaged softwares // with "little-endian strings" { Filetype = ACR_LIBIDO; - std::string rows = GetEntryByNumber(0x0028, 0x0010); - std::string columns = GetEntryByNumber(0x0028, 0x0011); - SetEntryByNumber(columns, 0x0028, 0x0010); - SetEntryByNumber(rows , 0x0028, 0x0011); + std::string rows = GetEntry(0x0028, 0x0010); + std::string columns = GetEntry(0x0028, 0x0011); + SetEntry(columns, 0x0028, 0x0010); + SetEntry(rows , 0x0028, 0x0011); } // ----------------- End of ACR-LibIDO kludge ------------------ } @@ -283,14 +282,13 @@ bool Document::IsReadable() { if( Filetype == Unknown) { - Debug::Verbose(0, "Document::IsReadable: wrong filetype"); + gdcmVerboseMacro( "Wrong filetype"); return false; } if( TagHT.empty() ) { - Debug::Verbose(0, "Document::IsReadable: no tags in internal" - " hash table."); + gdcmVerboseMacro( "No tags in internal hash table."); return false; } @@ -305,7 +303,7 @@ bool Document::IsReadable() */ TransferSyntaxType Document::GetTransferSyntax() { - DocEntry *entry = GetDocEntryByNumber(0x0002, 0x0010); + DocEntry *entry = GetDocEntry(0x0002, 0x0010); if ( !entry ) { return UnknownTS; @@ -397,7 +395,7 @@ bool Document::IsDicomV3() // Anyway, it's to late check if the 'Preamble' was found ... // And ... would it be a rich idea to check ? // (some 'no Preamble' DICOM images exist !) - return GetDocEntryByNumber(0x0002, 0x0010) != NULL; + return GetDocEntry(0x0002, 0x0010) != NULL; } /** @@ -424,17 +422,13 @@ std::ifstream *Document::OpenFile() if(Fp) { - Debug::Verbose( 0, - "Document::OpenFile is already opened when opening: ", - Filename.c_str()); + gdcmVerboseMacro( "Is already opened when opening: " << Filename.c_str()); } Fp = new std::ifstream(Filename.c_str(), std::ios::in | std::ios::binary); if( ! *Fp ) { - Debug::Verbose( 0, - "Document::OpenFile cannot open file: ", - Filename.c_str()); + gdcmVerboseMacro( "Cannot open file: " << Filename.c_str()); delete Fp; Fp = 0; return 0; @@ -455,6 +449,9 @@ std::ifstream *Document::OpenFile() zero == 0x0005 || zero == 0x0500 || zero == 0x0006 || zero == 0x0600 || zero == 0x0007 || zero == 0x0700 || zero == 0x0008 || zero == 0x0800 ) { + std::string msg + = Util::Format("ACR/DICOM with no preamble: (%04x)\n", zero); + gdcmVerboseMacro( msg.c_str() ); return Fp; } @@ -473,9 +470,7 @@ std::ifstream *Document::OpenFile() } CloseFile(); - Debug::Verbose( 0, - "Document::OpenFile not DICOM/ACR (missing preamble)", - Filename.c_str()); + gdcmVerboseMacro( "Not DICOM/ACR (missing preamble)" << Filename.c_str()); return 0; } @@ -544,13 +539,13 @@ void Document::WriteContent(std::ofstream *fp, FileType filetype) * \return pointer to the modified/created Header Entry (NULL when creation * failed). */ -ValEntry *Document::ReplaceOrCreateByNumber(std::string const &value, - uint16_t group, - uint16_t elem, - TagName const &vr ) +ValEntry *Document::ReplaceOrCreate(std::string const &value, + uint16_t group, + uint16_t elem, + TagName const &vr ) { ValEntry *valEntry = 0; - DocEntry *currentEntry = GetDocEntryByNumber( group, elem); + DocEntry *currentEntry = GetDocEntry( group, elem); if (currentEntry) { @@ -566,8 +561,7 @@ ValEntry *Document::ReplaceOrCreateByNumber(std::string const &value, { if (!RemoveEntry(currentEntry)) { - Debug::Verbose(0, "Document::ReplaceOrCreateByNumber: removal" - " of previous DocEntry failed."); + gdcmVerboseMacro( "Removal of previous DocEntry failed."); return NULL; } @@ -577,12 +571,11 @@ ValEntry *Document::ReplaceOrCreateByNumber(std::string const &value, // Create a new valEntry if necessary if (!valEntry) { - valEntry = NewValEntryByNumber(group, elem, vr); + valEntry = NewValEntry(group, elem, vr); if ( !AddEntry(valEntry)) { - Debug::Verbose(0, "Document::ReplaceOrCreateByNumber: AddEntry" - " failed allthough this is a creation."); + gdcmVerboseMacro("AddEntry failed although this is a creation."); delete valEntry; return NULL; @@ -605,14 +598,14 @@ ValEntry *Document::ReplaceOrCreateByNumber(std::string const &value, * \return pointer to the modified/created Header Entry (NULL when creation * failed). */ -BinEntry *Document::ReplaceOrCreateByNumber(uint8_t *binArea, - int lgth, - uint16_t group, - uint16_t elem, - TagName const &vr ) +BinEntry *Document::ReplaceOrCreate(uint8_t *binArea, + int lgth, + uint16_t group, + uint16_t elem, + TagName const &vr ) { BinEntry *binEntry = 0; - DocEntry *currentEntry = GetDocEntryByNumber( group, elem); + DocEntry *currentEntry = GetDocEntry( group, elem); // Verify the currentEntry if (currentEntry) @@ -629,8 +622,7 @@ BinEntry *Document::ReplaceOrCreateByNumber(uint8_t *binArea, { if (!RemoveEntry(currentEntry)) { - Debug::Verbose(0, "Document::ReplaceOrCreateByNumber: removal" - " of previous DocEntry failed."); + gdcmVerboseMacro( "Removal of previous DocEntry failed."); return NULL; } @@ -640,12 +632,11 @@ BinEntry *Document::ReplaceOrCreateByNumber(uint8_t *binArea, // Create a new binEntry if necessary if (!binEntry) { - binEntry = NewBinEntryByNumber(group, elem, vr); + binEntry = NewBinEntry(group, elem, vr); if ( !AddEntry(binEntry)) { - Debug::Verbose(0, "Document::ReplaceOrCreateByNumber: AddEntry" - " failed allthough this is a creation."); + gdcmVerboseMacro( "AddEntry failed allthough this is a creation."); delete binEntry; return NULL; @@ -682,10 +673,10 @@ BinEntry *Document::ReplaceOrCreateByNumber(uint8_t *binArea, * \return pointer to the modified/created SeqEntry (NULL when creation * failed). */ -SeqEntry *Document::ReplaceOrCreateByNumber( uint16_t group, uint16_t elem) +SeqEntry *Document::ReplaceOrCreate( uint16_t group, uint16_t elem) { SeqEntry *seqEntry = 0; - DocEntry *currentEntry = GetDocEntryByNumber( group, elem); + DocEntry *currentEntry = GetDocEntry( group, elem); // Verify the currentEntry if (currentEntry) @@ -702,8 +693,7 @@ SeqEntry *Document::ReplaceOrCreateByNumber( uint16_t group, uint16_t elem) { if (!RemoveEntry(currentEntry)) { - Debug::Verbose(0, "Document::ReplaceOrCreateByNumber: removal" - " of previous DocEntry failed."); + gdcmVerboseMacro( "Removal of previous DocEntry failed."); return NULL; } @@ -713,12 +703,11 @@ SeqEntry *Document::ReplaceOrCreateByNumber( uint16_t group, uint16_t elem) // Create a new seqEntry if necessary if (!seqEntry) { - seqEntry = NewSeqEntryByNumber(group, elem); + seqEntry = NewSeqEntry(group, elem); if ( !AddEntry(seqEntry)) { - Debug::Verbose(0, "Document::ReplaceOrCreateByNumber: AddEntry" - " failed allthough this is a creation."); + gdcmVerboseMacro( "AddEntry failed allthough this is a creation."); delete seqEntry; return NULL; @@ -736,10 +725,10 @@ SeqEntry *Document::ReplaceOrCreateByNumber( uint16_t group, uint16_t elem) * @param elem element number of the Entry * \return boolean */ -bool Document::ReplaceIfExistByNumber(std::string const &value, - uint16_t group, uint16_t elem ) +bool Document::ReplaceIfExist(std::string const &value, + uint16_t group, uint16_t elem ) { - SetEntryByNumber(value, group, elem); + SetEntry(value, group, elem); return true; } @@ -758,7 +747,7 @@ std::string Document::GetTransferSyntaxValue(TransferSyntaxType type) * @param element Element number of the searched Dicom Element * @return true is found */ -bool Document::CheckIfEntryExistByNumber(uint16_t group, uint16_t element ) +bool Document::CheckIfEntryExist(uint16_t group, uint16_t element ) { const std::string &key = DictEntry::TranslateToKey(group, element ); return TagHT.count(key) != 0; @@ -774,7 +763,7 @@ bool Document::CheckIfEntryExistByNumber(uint16_t group, uint16_t element ) * @return Corresponding element value representation when it exists, * and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise. */ -std::string Document::GetEntryByNumber(uint16_t group, uint16_t element) +std::string Document::GetEntry(uint16_t group, uint16_t element) { TagKey key = DictEntry::TranslateToKey(group, element); if ( !TagHT.count(key)) @@ -799,9 +788,9 @@ std::string Document::GetEntryByNumber(uint16_t group, uint16_t element) * @return Corresponding element value representation when it exists, * and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise. */ -std::string Document::GetEntryVRByNumber(uint16_t group, uint16_t element) +std::string Document::GetEntryVR(uint16_t group, uint16_t element) { - DocEntry *elem = GetDocEntryByNumber(group, element); + DocEntry *elem = GetDocEntry(group, element); if ( !elem ) { return GDCM_UNFOUND; @@ -817,9 +806,9 @@ std::string Document::GetEntryVRByNumber(uint16_t group, uint16_t element) * @param element Element number of the searched tag. * @return Corresponding element length; -2 if not found */ -int Document::GetEntryLengthByNumber(uint16_t group, uint16_t element) +int Document::GetEntryLength(uint16_t group, uint16_t element) { - DocEntry *elem = GetDocEntryByNumber(group, element); + DocEntry *elem = GetDocEntry(group, element); if ( !elem ) { return -2; //magic number @@ -835,14 +824,13 @@ int Document::GetEntryLengthByNumber(uint16_t group, uint16_t element) * @param group group number of the Dicom Element to modify * @param element element number of the Dicom Element to modify */ -bool Document::SetEntryByNumber(std::string const& content, - uint16_t group, uint16_t element) +bool Document::SetEntry(std::string const& content, + uint16_t group, uint16_t element) { - ValEntry *entry = GetValEntryByNumber(group, element); + ValEntry *entry = GetValEntry(group, element); if (!entry ) { - Debug::Verbose(0, "Document::SetEntryByNumber: no corresponding", - " ValEntry (try promotion first)."); + gdcmVerboseMacro( "No corresponding ValEntry (try promotion first)."); return false; } return SetEntry(content,entry); @@ -857,14 +845,13 @@ bool Document::SetEntryByNumber(std::string const& content, * @param group group number of the Dicom Element to modify * @param element element number of the Dicom Element to modify */ -bool Document::SetEntryByNumber(uint8_t*content, int lgth, - uint16_t group, uint16_t element) +bool Document::SetEntry(uint8_t*content, int lgth, + uint16_t group, uint16_t element) { - BinEntry *entry = GetBinEntryByNumber(group, element); + BinEntry *entry = GetBinEntry(group, element); if (!entry ) { - Debug::Verbose(0, "Document::SetEntryByNumber: no corresponding", - " ValEntry (try promotion first)."); + gdcmVerboseMacro( "No corresponding ValEntry (try promotion first)."); return false; } @@ -920,12 +907,12 @@ bool Document::SetEntry(uint8_t *content, int lgth, BinEntry *entry) * @param elem element number of the Entry * @return Pointer to the 'non string' area */ -void *Document::GetEntryBinAreaByNumber(uint16_t group, uint16_t elem) +void *Document::GetEntryBinArea(uint16_t group, uint16_t elem) { - DocEntry *entry = GetDocEntryByNumber(group, elem); + DocEntry *entry = GetDocEntry(group, elem); if (!entry) { - Debug::Verbose(1, "Document::GetDocEntryByNumber: no entry"); + gdcmVerboseMacro( "No entry"); return 0; } if ( BinEntry *binEntry = dynamic_cast(entry) ) @@ -945,7 +932,7 @@ void *Document::GetEntryBinAreaByNumber(uint16_t group, uint16_t elem) void Document::LoadEntryBinArea(uint16_t group, uint16_t elem) { // Search the corresponding DocEntry - DocEntry *docElement = GetDocEntryByNumber(group, elem); + DocEntry *docElement = GetDocEntry(group, elem); if ( !docElement ) return; @@ -977,13 +964,13 @@ void Document::LoadEntryBinArea(BinEntry *element) uint8_t *a = new uint8_t[l]; if( !a ) { - Debug::Verbose(0, "Document::LoadEntryBinArea cannot allocate a"); + gdcmVerboseMacro( "Cannot allocate a"); return; } /// \todo check the result Fp->read((char*)a, l); - if( Fp->fail() || Fp->eof()) //Fp->gcount() == 1 + if( Fp->fail() || Fp->eof()) { delete[] a; return; @@ -1002,10 +989,10 @@ void Document::LoadEntryBinArea(BinEntry *element) * @param element Element number of the searched Dicom Element * @return */ -/*bool Document::SetEntryBinAreaByNumber(uint8_t *area, - uint16_t group, uint16_t element) +/*bool Document::SetEntryBinArea(uint8_t *area, + uint16_t group, uint16_t element) { - DocEntry *currentEntry = GetDocEntryByNumber(group, element); + DocEntry *currentEntry = GetDocEntry(group, element); if ( !currentEntry ) { return false; @@ -1030,7 +1017,7 @@ void Document::LoadEntryBinArea(BinEntry *element) * @param element Element number of the searched Dicom Element * @return */ -DocEntry *Document::GetDocEntryByNumber(uint16_t group, uint16_t element) +DocEntry *Document::GetDocEntry(uint16_t group, uint16_t element) { TagKey key = DictEntry::TranslateToKey(group, element); if ( !TagHT.count(key)) @@ -1041,14 +1028,14 @@ DocEntry *Document::GetDocEntryByNumber(uint16_t group, uint16_t element) } /** - * \brief Same as \ref Document::GetDocEntryByNumber except it only + * \brief Same as \ref Document::GetDocEntry except it only * returns a result when the corresponding entry is of type * ValEntry. * @return When present, the corresponding ValEntry. */ -ValEntry *Document::GetValEntryByNumber(uint16_t group, uint16_t element) +ValEntry *Document::GetValEntry(uint16_t group, uint16_t element) { - DocEntry *currentEntry = GetDocEntryByNumber(group, element); + DocEntry *currentEntry = GetDocEntry(group, element); if ( !currentEntry ) { return 0; @@ -1057,20 +1044,20 @@ ValEntry *Document::GetValEntryByNumber(uint16_t group, uint16_t element) { return entry; } - Debug::Verbose(0, "Document::GetValEntryByNumber: unfound ValEntry."); + gdcmVerboseMacro( "Unfound ValEntry."); return 0; } /** - * \brief Same as \ref Document::GetDocEntryByNumber except it only + * \brief Same as \ref Document::GetDocEntry except it only * returns a result when the corresponding entry is of type * BinEntry. * @return When present, the corresponding BinEntry. */ -BinEntry *Document::GetBinEntryByNumber(uint16_t group, uint16_t element) +BinEntry *Document::GetBinEntry(uint16_t group, uint16_t element) { - DocEntry *currentEntry = GetDocEntryByNumber(group, element); + DocEntry *currentEntry = GetDocEntry(group, element); if ( !currentEntry ) { return 0; @@ -1079,7 +1066,7 @@ BinEntry *Document::GetBinEntryByNumber(uint16_t group, uint16_t element) { return entry; } - Debug::Verbose(0, "Document::GetBinEntryByNumber: unfound BinEntry."); + gdcmVerboseMacro( "Unfound BinEntry."); return 0; } @@ -1125,8 +1112,7 @@ uint32_t Document::SwapLong(uint32_t a) a=( ((a<< 8) & 0xff00ff00) | ((a>>8) & 0x00ff00ff) ); break; default : - //std::cout << "swapCode= " << SwapCode << std::endl; - Debug::Error(" Document::SwapLong : unset swap code"); + gdcmErrorMacro( "Unset swap code:" << SwapCode ); a = 0; } return a; @@ -1207,8 +1193,8 @@ void Document::ParseDES(DocEntrySet *set, long offset, if ( ! Global::GetVR()->IsVROfBinaryRepresentable(vr) ) { ////// Neither ValEntry NOR BinEntry: should mean UNKOWN VR - Debug::Verbose(0, "Document::ParseDES: neither Valentry, " - "nor BinEntry. Probably unknown VR."); + gdcmVerboseMacro( "Neither Valentry, nor BinEntry." + "Probably unknown VR."); } //////////////////// BinEntry or UNKOWN VR: @@ -1478,8 +1464,8 @@ void Document::LoadDocEntry(DocEntry *entry) else { // fusible - std::cout<< "MaxSizeLoadEntry exceeded, neither a BinEntry " - << "nor a ValEntry ?! Should never print that !" << std::endl; + gdcmErrorMacro( "MaxSizeLoadEntry exceeded, neither a BinEntry " + << "nor a ValEntry ?! Should never print that !" ); } // to be sure we are at the end of the value ... @@ -1555,8 +1541,8 @@ void Document::LoadDocEntry(DocEntry *entry) if( length % 2 ) { newValue = Util::DicomString(str, length+1); - //Debug::Verbose(0, "Warning: bad length: ", length ); - Debug::Verbose(0, "For string :", newValue.c_str()); + gdcmVerboseMacro("Warning: bad length: " << length << + ",For string :" << newValue.c_str()); // Since we change the length of string update it length //entry->SetReadLength(length+1); } @@ -1568,10 +1554,9 @@ void Document::LoadDocEntry(DocEntry *entry) if ( ValEntry *valEntry = dynamic_cast(entry) ) { - if ( Fp->fail() || Fp->eof())//Fp->gcount() == 1 + if ( Fp->fail() || Fp->eof()) { - Debug::Verbose(1, "Document::LoadDocEntry", - "unread element value"); + gdcmVerboseMacro("Unread element value"); valEntry->SetValue(GDCM_UNREAD); return; } @@ -1588,8 +1573,7 @@ void Document::LoadDocEntry(DocEntry *entry) } else { - Debug::Error(true, "Document::LoadDocEntry" - "Should have a ValEntry, here !"); + gdcmErrorMacro( "Should have a ValEntry, here !"); } } @@ -1693,8 +1677,7 @@ void Document::FindDocEntryLength( DocEntry *entry ) // and the dictionary entry depending on them. uint16_t correctGroup = SwapShort( entry->GetGroup() ); uint16_t correctElem = SwapShort( entry->GetElement() ); - DictEntry *newTag = GetDictEntryByNumber( correctGroup, - correctElem ); + DictEntry *newTag = GetDictEntry( correctGroup, correctElem ); if ( !newTag ) { // This correct tag is not in the dictionary. Create a new one. @@ -1735,7 +1718,7 @@ void Document::FindDocEntryLength( DocEntry *entry ) std::string Document::FindDocEntryVR() { if ( Filetype != ExplicitVR ) - return(GDCM_UNKNOWN); + return GDCM_UNKNOWN; long positionOnEntry = Fp->tellg(); // Warning: we believe this is explicit VR (Value Representation) because @@ -1753,9 +1736,9 @@ std::string Document::FindDocEntryVR() if( !CheckDocEntryVR(vr) ) { Fp->seekg(positionOnEntry, std::ios::beg); - return(GDCM_UNKNOWN); + return GDCM_UNKNOWN; } - return(vr); + return vr; } /** @@ -1921,14 +1904,12 @@ void Document::SkipDocEntry(DocEntry *entry) /** * \brief Skips to the begining of the next Header Entry * \warning NOT end user intended method ! - * @param offset start of skipping - * @param readLgth length to skip - + * @param currentDocEntry entry to skip */ -void Document::SkipToNextDocEntry(DocEntry *newDocEntry) +void Document::SkipToNextDocEntry(DocEntry *currentDocEntry) { - Fp->seekg((long)(newDocEntry->GetOffset()), std::ios::beg); - Fp->seekg( (long)(newDocEntry->GetReadLength()),std::ios::cur); + Fp->seekg((long)(currentDocEntry->GetOffset()), std::ios::beg); + Fp->seekg( (long)(currentDocEntry->GetReadLength()),std::ios::cur); } /** @@ -1936,7 +1917,7 @@ void Document::SkipToNextDocEntry(DocEntry *newDocEntry) * the parser went Jabberwocky) one can hope improving things by * applying some heuristics. * @param entry entry to check - * @param foundLength fist assumption about length + * @param foundLength first assumption about length */ void Document::FixDocEntryFoundLength(DocEntry *entry, uint32_t foundLength) @@ -1952,17 +1933,14 @@ void Document::FixDocEntryFoundLength(DocEntry *entry, if ( foundLength % 2) { - std::ostringstream s; - s << "Warning : Tag with uneven length " - << foundLength - << " in x(" << std::hex << gr << "," << elem <<")" << std::dec; - Debug::Verbose(0, s.str().c_str()); + gdcmVerboseMacro( "Warning : Tag with uneven length " << foundLength + << " in x(" << std::hex << gr << "," << elem <<")"); } //////// Fix for some naughty General Electric images. // Allthough not recent many such GE corrupted images are still present // on Creatis hard disks. Hence this fix shall remain when such images - // are no longer in user (we are talking a few years, here)... + // are no longer in use (we are talking a few years, here)... // Note: XMedCom probably uses such a trick since it is able to read // those pesky GE images ... if ( foundLength == 13) @@ -2015,7 +1993,7 @@ void Document::FixDocEntryFoundLength(DocEntry *entry, */ bool Document::IsDocEntryAnInteger(DocEntry *entry) { - uint16_t element = entry->GetElement(); + uint16_t elem = entry->GetElement(); uint16_t group = entry->GetGroup(); const std::string &vr = entry->GetVR(); uint32_t length = entry->GetLength(); @@ -2023,7 +2001,7 @@ bool Document::IsDocEntryAnInteger(DocEntry *entry) // 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 ( elem == 0 ) // This is the group length of the group { if ( length == 4 ) { @@ -2038,13 +2016,11 @@ bool Document::IsDocEntryAnInteger(DocEntry *entry) // test is useless (and might even look a bit paranoid), when we // encounter such an ill-formed image, we simply display a warning // message and proceed on parsing (while crossing fingers). - std::ostringstream s; long filePosition = Fp->tellg(); - s << "Erroneous Group Length element length on : (" \ - << std::hex << group << " , " << element + gdcmVerboseMacro( "Erroneous Group Length element length on : (" + << std::hex << group << " , " << elem << ") -before- position x(" << filePosition << ")" - << "lgt : " << length; - Debug::Verbose(0, "Document::IsDocEntryAnInteger", s.str().c_str() ); + << "lgt : " << length ); } } @@ -2080,8 +2056,8 @@ uint32_t Document::FindDocEntryLengthOBOrOW() } catch ( FormatError ) { - throw FormatError("Document::FindDocEntryLengthOBOrOW()", - " group or element not present."); + throw FormatError("Unexpected end of file encountered during ", + "Document::FindDocEntryLengthOBOrOW()"); } // We have to decount the group and element we just read @@ -2089,12 +2065,13 @@ uint32_t Document::FindDocEntryLengthOBOrOW() if ( group != 0xfffe || ( ( elem != 0xe0dd ) && ( elem != 0xe000 ) ) ) { - Debug::Verbose(1, "Document::FindDocEntryLengthOBOrOW: neither an Item " - "tag nor a Sequence delimiter tag."); + long filePosition = Fp->tellg(); + gdcmVerboseMacro( "Neither an Item tag nor a Sequence delimiter tag on :" + << std::hex << group << " , " << elem + << ") -before- position x(" << filePosition << ")" ); + Fp->seekg(positionOnEntry, std::ios::beg); - throw FormatUnexpected("Document::FindDocEntryLengthOBOrOW()", - "Neither an Item tag nor a Sequence " - "delimiter tag."); + throw FormatUnexpected( "Neither an Item tag nor a Sequence delimiter tag."); } if ( elem == 0xe0dd ) @@ -2224,7 +2201,7 @@ bool Document::CheckSwap() char *entCur = deb + 128; if( memcmp(entCur, "DICM", (size_t)4) == 0 ) { - Debug::Verbose(1, "Document::CheckSwap:", "looks like DICOM Version3"); + gdcmVerboseMacro( "Looks like DICOM Version3" ); // Next, determine the value representation (VR). Let's skip to the // first element (0002, 0000) and check there if we find "UL" @@ -2258,27 +2235,23 @@ bool Document::CheckSwap() // instead of just checking for UL, OB and UI !? group 0000 { Filetype = ExplicitVR; - Debug::Verbose(1, "Document::CheckSwap:", - "explicit Value Representation"); + gdcmVerboseMacro( "Explicit Value Representation"); } else { Filetype = ImplicitVR; - Debug::Verbose(1, "Document::CheckSwap:", - "not an explicit Value Representation"); + gdcmVerboseMacro( "Not an explicit Value Representation"); } if ( net2host ) { SwapCode = 4321; - Debug::Verbose(1, "Document::CheckSwap:", - "HostByteOrder != NetworkByteOrder"); + gdcmVerboseMacro( "HostByteOrder != NetworkByteOrder"); } else { SwapCode = 0; - Debug::Verbose(1, "Document::CheckSwap:", - "HostByteOrder = NetworkByteOrder"); + gdcmVerboseMacro( "HostByteOrder = NetworkByteOrder"); } // Position the file position indicator at first tag (i.e. @@ -2291,7 +2264,7 @@ bool Document::CheckSwap() // 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). - Debug::Verbose(1, "Document::CheckSwap:", "not a DICOM Version3 file"); + gdcmVerboseMacro( "Not a DICOM Version3 file"); Fp->seekg(0, std::ios::beg); // Our next best chance would be to be considering a 'clean' ACR/NEMA file. @@ -2366,8 +2339,7 @@ bool Document::CheckSwap() Filetype = ACR; return true; default : - Debug::Verbose(0, "Document::CheckSwap:", - "ACR/NEMA unfound swap info (Really hopeless !)"); + gdcmVerboseMacro( "ACR/NEMA unfound swap info (Really hopeless !)"); Filetype = Unknown; return false; } @@ -2388,8 +2360,7 @@ bool Document::CheckSwap() */ void Document::SwitchSwapToBigEndian() { - Debug::Verbose(1, "Document::SwitchSwapToBigEndian", - "Switching to BigEndian mode."); + gdcmVerboseMacro( "Switching to BigEndian mode."); if ( SwapCode == 0 ) { SwapCode = 4321; @@ -2469,7 +2440,7 @@ void Document::HandleBrokenEndian(uint16_t group, uint16_t elem) group = 0xfffe; elem = 0xe000; } - else if ((group == 0xfffe) && (elem == 0xe00d) && reversedEndian) + else if (group == 0xfffe && elem == 0xe00d && reversedEndian) { // end of reversed endian group reversedEndian--; @@ -2508,18 +2479,18 @@ DocEntry *Document::ReadNextDocEntry() if( vr == GDCM_UNKNOWN) { - DictEntry *dictEntry = GetDictEntryByNumber(group,elem); + DictEntry *dictEntry = GetDictEntry(group,elem); if( dictEntry ) realVR = dictEntry->GetVR(); } DocEntry *newEntry; if( Global::GetVR()->IsVROfSequence(realVR) ) - newEntry = NewSeqEntryByNumber(group, elem); + newEntry = NewSeqEntry(group, elem); else if( Global::GetVR()->IsVROfStringRepresentable(realVR) ) - newEntry = NewValEntryByNumber(group, elem,vr); + newEntry = NewValEntry(group, elem,vr); else - newEntry = NewBinEntryByNumber(group, elem,vr); + newEntry = NewBinEntry(group, elem,vr); if( vr == GDCM_UNKNOWN ) { @@ -2530,7 +2501,7 @@ DocEntry *Document::ReadNextDocEntry() std::string msg; msg = Util::Format("Falsely explicit vr file (%04x,%04x)\n", newEntry->GetGroup(), newEntry->GetElement()); - Debug::Verbose(1, "Document::FindVR: ", msg.c_str()); + gdcmVerboseMacro( msg.c_str() ); } newEntry->SetImplicitVR(); } @@ -2606,15 +2577,13 @@ bool Document::ReadTag(uint16_t testGroup, uint16_t testElement) } if ( itemTagGroup != testGroup || itemTagElement != testElement ) { - std::ostringstream s; - s << " We should have found tag ("; - s << std::hex << testGroup << "," << testElement << ")" << std::endl; - s << " but instead we encountered tag ("; - s << std::hex << itemTagGroup << "," << itemTagElement << ")" - << std::endl; - s << " at address: " << (unsigned)currentPosition << std::endl; - Debug::Verbose(0, "Document::ReadItemTagLength: wrong Item Tag found:"); - Debug::Verbose(0, s.str().c_str()); + gdcmVerboseMacro( "Wrong Item Tag found:" + << " We should have found tag (" + << std::hex << testGroup << "," << testElement << ")" << std::endl + << " but instead we encountered tag (" + << std::hex << itemTagGroup << "," << itemTagElement << ")" + << std::dec + << " at address: " << (unsigned int)currentPosition ); Fp->seekg(positionOnEntry, std::ios::beg); return false; @@ -2650,11 +2619,9 @@ uint32_t Document::ReadTagLength(uint16_t testGroup, uint16_t testElement) long currentPosition = Fp->tellg(); uint32_t itemLength = ReadInt32(); { - std::ostringstream s; - s << "Basic Item Length is: " - << itemLength << std::endl; - s << " at address: " << (unsigned)currentPosition << std::endl; - Debug::Verbose(0, "Document::ReadItemTagLength: ", s.str().c_str()); + gdcmVerboseMacro( "Basic Item Length is: " + << itemLength << std::endl + << " at address: " << (unsigned int)currentPosition); } return itemLength; } @@ -2684,12 +2651,8 @@ void Document::ReadAndSkipEncapsulatedBasicOffsetTable() { uint32_t individualLength = str2num( &basicOffsetTableItemValue[i], uint32_t); - std::ostringstream s; - s << " Read one length: "; - s << std::hex << individualLength << std::endl; - Debug::Verbose(0, - "Document::ReadAndSkipEncapsulatedBasicOffsetTable: ", - s.str().c_str()); + gdcmVerboseMacro( "Read one length: " << + std::hex << individualLength ); } #endif //GDCM_DEBUG @@ -2746,7 +2709,7 @@ void Document::ComputeRLEInfo() if ( nbRleSegments > 16 ) { // There should be at most 15 segments (refer to RLEFrame class) - Debug::Verbose(0, "Document::ComputeRLEInfo: too many segments."); + gdcmVerboseMacro( "Too many segments."); } uint32_t rleSegmentOffsetTable[16]; @@ -2789,8 +2752,7 @@ void Document::ComputeRLEInfo() // Delimiter Item': if ( !ReadTag(0xfffe, 0xe0dd) ) { - Debug::Verbose(0, "Document::ComputeRLEInfo: no sequence delimiter "); - Debug::Verbose(0, " item at end of RLE item sequence"); + gdcmVerboseMacro( "No sequence delimiter item at end of RLE item sequence"); } } @@ -2830,8 +2792,7 @@ void Document::ComputeJPEGFragmentInfo() // Delimiter Item': if ( !ReadTag(0xfffe, 0xe0dd) ) { - Debug::Verbose(0, "Document::ComputeRLEInfo: no sequence delimiter "); - Debug::Verbose(0, " item at end of JPEG item sequence"); + gdcmVerboseMacro( "No sequence delimiter item at end of JPEG item sequence"); } } @@ -2938,8 +2899,8 @@ TagDocEntryHT *Document::BuildFlatHashTable() bool Document::operator<(Document &document) { // Patient Name - std::string s1 = GetEntryByNumber(0x0010,0x0010); - std::string s2 = document.GetEntryByNumber(0x0010,0x0010); + std::string s1 = GetEntry(0x0010,0x0010); + std::string s2 = document.GetEntry(0x0010,0x0010); if(s1 < s2) { return true; @@ -2951,8 +2912,8 @@ bool Document::operator<(Document &document) else { // Patient ID - s1 = GetEntryByNumber(0x0010,0x0020); - s2 = document.GetEntryByNumber(0x0010,0x0020); + s1 = GetEntry(0x0010,0x0020); + s2 = document.GetEntry(0x0010,0x0020); if ( s1 < s2 ) { return true; @@ -2964,8 +2925,8 @@ bool Document::operator<(Document &document) else { // Study Instance UID - s1 = GetEntryByNumber(0x0020,0x000d); - s2 = document.GetEntryByNumber(0x0020,0x000d); + s1 = GetEntry(0x0020,0x000d); + s2 = document.GetEntry(0x0020,0x000d); if ( s1 < s2 ) { return true; @@ -2977,8 +2938,8 @@ bool Document::operator<(Document &document) else { // Serie Instance UID - s1 = GetEntryByNumber(0x0020,0x000e); - s2 = document.GetEntryByNumber(0x0020,0x000e); + s1 = GetEntry(0x0020,0x000e); + s2 = document.GetEntry(0x0020,0x000e); if ( s1 < s2 ) { return true;