Program: gdcm
Module: $RCSfile: gdcmDocument.cxx,v $
Language: C++
- Date: $Date: 2005/09/20 15:11:19 $
- Version: $Revision: 1.281 $
+ Date: $Date: 2005/10/03 16:08:07 $
+ Version: $Revision: 1.285 $
Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
l'Image). All rights reserved. See Doc/License.txt or
}
/**
- * \brief Writes in a file all the Header Entries (Dicom Elements)
+ * \brief Writes in a file all the Entries (Dicom Elements)
* @param fp file pointer on an already open file (actually: Output File Stream)
* @param filetype Type of the File to be written
* (ACR-NEMA, ExplicitVR, ImplicitVR)
- * @return Always true.
*/
void Document::WriteContent(std::ofstream *fp, FileType filetype)
{
bool used; // will be set to false when something wrong happens to an Entry.
// (Entry will then be deleted)
bool delim_mode_intern = delim_mode;
- bool first = true;
+ bool first = true;
+ gdcmWarningMacro( "Enter in ParseDES, delim-mode " << delim_mode
+ << " at offset " << std::hex << offset );
while (true)
{
if ( !delim_mode && ((long)(Fp->tellg())-offset) >= l_max)
newDocEntry = ReadNextDocEntry( );
+ // FIXME :
+ // Private tag, in IMplicit VR are defaulted as a BinEntry,
+ // Very often they are only composed of Printable characters,
+ // and could be defaulted as a ValEntry.
+ // It's too late to do the Job
+ // (we should check the value, but we know it after LoadDocEntry ...)
+ // --> in next gdcm major release let's unify ValEntry and BinEntry !
+
// Uncoment this printf line to be able to 'follow' the DocEntries
// when something *very* strange happens
break;
}
- // an Item Starter found elsewhere but the first postition
+ // an Item Starter found elsewhere but the first position
// of a SeqEntry means previous entry was a Sequence
// but we didn't get it (private Sequence + Implicit VR)
// we have to backtrack.
{
lgrGroup = atoi(strLgrGroup.c_str());
Fp->seekg(lgrGroup, std::ios::cur);
- used = false;
- RemoveEntry( newDocEntry );
+ //used = false; // never used
+ RemoveEntry( newDocEntry ); // Remove and delete
// bcc 5.5 is right "assigned a value that's never used"
// newDocEntry = 0;
continue;
if ( newDocEntry->GetGroup()%2 != 0 )
{
Fp->seekg( l, std::ios::cur);
- used = false;
+ RemoveEntry( newDocEntry ); // Remove and delete
+ //used = false; // never used
continue;
}
}
{
// User asked to skip *any* SeQuence
Fp->seekg( l, std::ios::cur);
- used = false;
+ //used = false; // never used
+ RemoveEntry( newDocEntry ); // Remove and delete
continue;
}
// delay the dynamic cast as late as possible
if ( l != 0 )
{ // Don't try to parse zero-length sequences
+
+ gdcmWarningMacro( "Entry in ParseSQ, delim " << delim_mode_intern
+ << " at offset " << std::hex
+ << newDocEntry->GetOffset() );
+
ParseSQ( newSeqEntry,
newDocEntry->GetOffset(),
l, delim_mode_intern);
+
+ gdcmWarningMacro( "Exit from ParseSQ, delim " << delim_mode_intern);
+
}
if ( !set->AddEntry( newSeqEntry ) )
{
used = false;
}
- if ( !delim_mode && ((long)(Fp->tellg())-offset) >= l_max)
+ if ( !delim_mode && ((long)(Fp->tellg())-offset) >= l_max)
{
if ( !used )
delete newDocEntry;
}
first = false;
} // end While
+ gdcmWarningMacro( "Exit from ParseDES, delim-mode " << delim_mode );
}
/**
}
else if (group%2 == 1 && (elem >= 0x0010 && elem <=0x00ff ))
{
- //DICOM PS 3-5 7.8.1 a) states that those
- //(gggg-0010->00FF where gggg is odd) attributes have to be LO
- realVR= "LO";
+ // DICOM PS 3-5 7.8.1 a) states that those
+ // (gggg-0010->00FF where gggg is odd) attributes have to be LO
+ realVR = "LO";
}
else
{
if ( Global::GetVR()->IsVROfSequence(realVR) )
newEntry = NewSeqEntry(group, elem);
else if ( Global::GetVR()->IsVROfStringRepresentable(realVR) )
- newEntry = NewValEntry(group, elem, vr);
+ newEntry = NewValEntry(group, elem, realVR);
else
- newEntry = NewBinEntry(group, elem, vr);
+ newEntry = NewBinEntry(group, elem, realVR);
if ( vr == GDCM_UNKNOWN )
{
reversedEndian--;
SwitchByteSwapCode();
}
+ else if (group == 0xfeff && elem == 0xdde0)
+ {
+ // reversed Sequence Terminator found
+ // probabely a bug in the header !
+ // Do what you want, it breaks !
+ //reversedEndian--;
+ //SwitchByteSwapCode();
+ gdcmWarningMacro( "Should never get here! reversed Sequence Terminator!" );
+ // fix the tag
+ group = 0xfffe;
+ elem = 0xe0dd;
+ }
+ else if (group == 0xfffe && elem == 0xe0dd)
+ {
+ gdcmWarningMacro( "Straight Sequence Terminator." );
+ }
}
/**