* DEVELOPPER spread out in Doc/Website/Developpers.html, CodingStyle.html,
    DeveloppersPython.html
  * INSTALL nows refers to Doc/Website/Installation.html

Cosmetic modifs to be more Coding style compliant

See 2004-09-24 entry.

Print() displays old key instead of new in order to make PrintHeader human readable
again

  * Added Doc/Website/CodingStyle.html, Developpers.html,
    DeveloppersPython.html, GdcmDataCvs.html and
    DownloadVersion0_1.html, DownloadVersion0_3.html.
  * Some Doc/*.txt Doxygen files (which do not really concern the
    documentation itself, but the website) moved to html and
    placed in directory Doc/Website:
    - Doc/DoxyDevelInstal.txt moved to Doc/Website/Developpers.html
    - Doc/DoxyInstallation.txt moved to Doc/Website/Installation.html
    - Doc/DoxyIntroduction.txt included in Doc/Website/Main.html
  * Doc/DoxyfileDeveloppers, DoxyfileUsers, Makefile.am oldies removed.
  * CMakeLists.txt changed accordingly.

FIX: Revert back to previous version, I don't believe this was a really safe code anyway. Plus this solve my issue on the Mac using gcc version 3.3 20030304 (Apple Computer, Inc. build 1666). The code should also more readable.

ENH: Remove function instead of putting stoopid comment

ENH: Could not compile using g++ -pedantic

ENH: update dvpt version

ENH: What the hell! There is no way it compiles on Win32

FIX: Python wrapping was broken due to renaming

ENH: Initial pass at playing with regex in cmake, to parse dicomV3.dic to stick in gdcm

ENH: license issue

ENH: Solve some licensing issue

ENH: Minor tweak so that user find documentation easily

STYLE: No DOS file, please

ENH: Fix some compilation warnings

 * Added Doc/Website directory, that contains a first raw version
    of gdcm Web site.

  * class gdcmObject is now called gdcmDicomDirObject (less confusing name)

  * VoidArea is now called BinArea (less confusing name),
    and all the methods called ...VoidArea... are now ... BinArea...

2004-09-23 Jean-Pierre Roux
  * FIX In order not to be poluted any longer by casting problems,
    the member VoidArea of gdcmBinEntry is now uint8_t* (instead of void *)
 we can now delete[] it safely

 Reverting to previous version temporarily. Sorry !  Frog

  * src/gdcmDocument.cxx: gdcmDocument::~gdcmDocument() doesn't clear (nor
    clear) TagHT, which is an inherited member of gdcmElementSet. It is
    up to the destructor of gdcmElementSet to clean up TagHt and it's
    pointed content.

GetImageData is now done *before* copying the docEntries

add a v(ideo inv) option to TestWrite -Special Hector Duque-

Code identation

Code indentation

string "gdcm::Loaded (BinEntry)" replaced by GDCM_BINLOADED

ENH: Sorry OT-PAL-8-face.dcm is still seg faulting on gcc3.3 and gcc3.4

ENH: Unused var

ENH: Some unused vars

FIX: Some unused var, just (void)'ing them

* src/gdcmCommon.h, gdcmDocument.cxx: doxygen typos

  * Test/CMakeLists.txt: update to follow gdcmData renamings.

Example/TestReadWriteReadCompare may now be called with command line parameters
(I said : EXAMPLE/Test... )

uncomment no longer ctest breaker images

Last modif before gdcmPixelData class introduction :
   High Bit and Bits Allocated are now save and restored
Color Palettes are now droped out the H Table (and no longer destroyed)

  * ENH: added some utility method that builds a flat dictionnary
    holding all the Dicom entries contained in the recursive structure
    of a gdcmElementSet. Refer to add FlatHashTablePrint.cxx for
    an example of usage.
    - src/gdcmDocument.[h|cxx] added BuildFlatHashTableRecurse() and
      BuildFlatHashTable() that build a flat dictionary.
    - src/gdcmElementSet.h: added a new private GetTag() accessor.
      gdcmDocument is now a friend of gdcmElementSet.
    - src/gdcmElementSet.cxx: clean up.
    - Example/FlatHashTablePrint.cxx added.
    - Example/CmakeLists.txt changed accordingly

uncomment ShowDicom

inclusion of code for jpeg compression
(user intended methods not yet commited)

  * gdcmDocEntrySet::SQDepthLevel and gdcmDocEntrySet::BaseTagKey attributes
    moved away from gdcmDocEntrySet (since this class is an abstract class
    acting like an interface). SQDepthLevel and BaseTagKey are now
    in class
    - src/gdcmDocEntrySet.[h|cxx] removal of SQDepthLevel and BaseTagKey
      and associated accessors. Doxygenation of the class.
    - src/gdcmSQItem.[h|cxx] SQDepthLevel and BaseTagKey and associated
      accessors added.
    - src/gdcmSeqEntry.[h|cxx]: constructor doesn't handle depth anymore.
      Use SQDepthLevel accessor instead. ::Print() adapted.
    - src/gdcmElementSet.cxx changed according to changes in gdcmSeqEntry.
    - src/gdcmDocument.cxx changed accordingly.

gdcmDocument::ReplaceOrCreateByNumber has now an extra parameter,
 std::string VR (default value : "unkn"), that makes
 TestCopyDicom code much lighter .

Very cautiously put gdcmDocentry destr as virtual, to prevent mem leak...well in theory since I had to comment inherited destructor. Sorry this was necessary

ENH: Warnings about shadow vars, + unused var

# As of 14/09 this image creates a crash:
#Program received signal SIGSEGV, Segmentation fault.
#0x4032bc4b in gdcmHeader::GetLUTRGBA (this=0x8149228) at /home/malaterre/Creatis/gdcm/src/gdcmHeader.cxx:1170
#1170          *a = lutR[i*mult+1];

Now, TestCopyDicom deals with private Entries

  * Preparation of writing a gdcmHeader iterator: generalisation of gdcmTagKey
    - The following is the doxygen comment of the typedef declaration
      of gdcmagKey in src/gdcmCommon.h:
           gdcmTagKey is made to old an "universal" (as in URL, Universal
           Ressource Locator)  key to a gdcmDocEntry i.e. a dicom tag.
           A dicom tag allways has a group and an element, but a set of tags
           embeded in various (optionally nested) sequences and sharing
           the same group and element all share the same (group, element)
           "identifier". Hence the (group, element) cannot be used as an
           identifier (in gdcm we shall refer to a "TagKey") of a tag.
           In order to construct a proper tag identifier (i.e. a key) we
           consider the following definition of a TagKey:
           - let Group, Element be the string representation of the
             group and element dicom tag members,
           - let ItemNumber be the string representation of the integer
             index of the considered item number of a sequence,
           Let the key of a tag embeded in a sequence, noted SeqTag, be
           the form:
              /ItemNumber#Group|Element
           where "/", "#" and "|" are characters acting as separators.
           Then the general form of a gdcmTagKey is given by:
              Group|Element<SeqTag>
           where <SeqTag> means NO or many instances of SeqTag.
           Hence the gdcmTagKey of a tag not "leaving" in a sequence is the
           string e.g.
               0028|1201
           but the gdcmTagKey of a tag "embeded" is the first item of
           a sequence, itself nested in the third item of a sequence is the
           string e.g.
               0004|1220/2#0008|0082/0#0008|0090
    - src/gdcmDocEntry.h: added a new Key (of type gdcmTagKey) member, in
      order to hold the new sequence compatible key. Previously, the
      GetKey() method would look in the underlying gdcmDictEntry.
    - src/gdcmDocEntry.cxx:
      -- constructor now copies the underlying DictEntry key, in the local
         Key member.
      -- ::Print: displays the member Key, instead of the (group, element).
    - src/gdcmCommon.h: added some comments on typedef gdcmTagKey.
    - src/gdcmDocEntrySet.h:xi
      -- ::ParseDES() now setups the gdcmTagKey of the sequence it is parsing.
      -- now has a new BaseTagKey member.
      -- STYLE.
  * src/gdcmValEntry.[h|cxx], src/gdcmBinEntry.[h|cxx]: the member VoidArea,
    previously a member of gdcmValEntry, moved to gdcmBinEntry were is
    truly belongs.
    This poses the problem with the semantics of the following lines
       LoadEntryVoidArea(0x0028,0x1201);  // R    LUT
       LoadEntryVoidArea(0x0028,0x1202);  // G    LUT
       LoadEntryVoidArea(0x0028,0x1203);  // B    LUT
    in gdcmDocument::gdcmDocument(std::string const & ). Please refer
    to the long FIXME note for what the problem is. Nevertheless in
    order to get things working the dicom dictionary was altered !
    Please fix things urgently...
  * Dicts/dicomV3.dic WRONGLY altered (this means we introduced a uncorrect
    information), see above note on moving the member VoidArea. Nevertheless
    the following entries previously correctly set as US are now inproperly
    set to OW:
      0028 1201 OW IMG Red Palette Color Lookup Table Data
      0028 1202 OW IMG Green Palette Color Lookup Table Data
      0028 1203 OW IMG Blue Palette Color Lookup Table Data
  * src/gdcmDocEntry.[h|cxx], src/gdcmSeqEntry.h: SQDepthLevel member
    of gdcmDocEntry moved to gdcmSeqEntry.
  * src/gdcmSeqEntry.cxx: STYLE.

new version of TestCopyDicom

dealing with 'zero length' integers

Update of Dicom Dictionary

Disabls TestChangeHeader, this is not a test right now

FIX: put back everything since I broke tests, warnings should be discarded -apparently-

FIX: Some warnings

FIX: icc could not build otherwise

Cosmetic modif
(Still don't solve Mathieu's pb on TestCopyDicom :-(

Doxygenation

zilch.

Adding some warning messages

 WARNING :  This was not designed as a 'Test' program, but as a utility
            provided to 'transform' an image 'Siemens MRI New version'
            into an image 'Siemens MRI old version'
            If user tries to 'merge' two mismatching images
            e.g. a LUT image and a RGB image
            or a compressed and an uncompressed one
            or a single frame and a multiframe,
            or a '12 Bits Allocated' image and anything else, etc,
            Probabely TestChangeHeader will fail !

Some fixes to help dealing with UnLoaded values
(ForceLoad no yet commited)

ENH: Adding a test I know that segfault

ENH: Applied JPR comments, but still doesn't work on 012345.002.050.dcm

FIX: warning

ADD : gdcmGlobal::CountSubstring method
ADD  : Example/TestChangeHeader.cxx (Mathieu's one)
FIX : now gdcmDocument::SetEntryByNumber deals properly with 'multivaluated'
integer entries (US, SS, etc )
(e.g. : Aquisition Matrix)

FIX: minor inconsistency

FIX: Pb on recent linux machine

CheckEntryIfExistByNumber returns a bool (no longer an int, because we use now a map, not a multimap)

Offset set to 0 in the constructor to prevent any further missprinting

FIX :
    gdcmElementSet::RemoveEntry() now removes the entry from the H Table
ADD :
   gdcmElementSet::RemoveEntryNoDestroy() removes the entry, without deletion

Temporary (?) uncomment of method gdcmElementSet::GetEntry()
to allow Example/TestCopyDicom.cxx to compile
(she shall have to decide if we consider the modif as permanent, or if we remove TestCopyDicom.cxx)

ENH: bad hack to be able to write dicom the way I want, sorry

STYLE: No need to use c string

dded a modified version of Mathieu's TestCopyDicom.cxx
(put here not to polute gdcm/Test)

ENH: minor tweaks

New test for testing the copying for creating dicom file from scratch is ok

Some improvements for Write problems

remove commented out line after fix of  gdcmFile::SetImageData()
ctest doesn't break.
Is it enought to say my fix is OK ?

Doxygenation

Doxygenation

some more DICOMDIR fixes

ENH: Refactor some code

FIX: Unused vars

light modif around DICOMDIR stuff

Eric Boix

return 0 replaces continue in step 3a

minor updates

- now, DicomDir stuff stakes into account the 'new' structure
- gdcmObject is no longer composed of 2 pointers in a chained list, but is an SQItem
- useless gdcmDicomDir::WriteEntries method is removed
- useless gdcmDicomDir::AddObjectToEnd method is removed
- WARNING : BuidUpDicomDir not yet debugged, but I prefer to commit right now.
-> See you tomorow

Memo for gcc flags added.

FIX: Compilation on Mac OSX is fine now + remove tabs

ENH: Fix a shadow variable

ENH: forgot this easy one

FIX: There were lot of shadowed variables, could have been a hell to debug + remove tabs

  * Test/TestAllEntryVerify.cxx minor fix and added comments.

  * gdcmPython/testSuite.py: all the test suite in python is now moved
    to it's C++ version (see gdcmData/TestAllEntryVerifyReference.txt)
  * Test/CMakeLists.txt adapted to renaming of files in gdcmData
  * gdcm/TODO and src/gdcmDictSet.h cleaned up frow the "TODO Swig" oldies

  * gdcmPython/testSuite.py checks on CR-MONO1-10-chest.dcm moved to
    gdcmData/TestAllEntryVerifyReference.txt
  * Test/TestAllEntryVerify.cxx is now effective (used allways return true)
  * src/gdcmDocument.[cxx|h]: constructors no longer use the bool
    exception_on_error parameter.
    - src/gdcmFile.[cxx|h], src/gdcmHeader.[cxx|h] changed accordingly,
    - vtk/vtkGdcmReader.cxx changed accordingly,
    - Example/*.cxx and Test/*.cxx changed accordingly.

  * gdcmPython/CMakeLists.txt: SWIG_FLAGS doesn't declare includeall
    to avoid inclusion recursion until STL is reached.
  * src/gdcmDocument.[h|cxx]: exceptions substituted to errno C-style
    mecanism. errno.h is not included in gdcm anymore.
  * src/gdcmException.h: introduced new gdcmFormatUnexpected class
    (gdcmFormatError now inherits from gdcmFormatUnexpected).
  * TODO updated

ENH: Who would have bet ! Some cosmetic cleanups

ENH: Guess what ! Still some cosmetic cleanup

ENH: Minor patch, cosmetic -again-. Remove all this->. I might have discover the reason of misterious leaks

ENH: Minor patch, mostly cosmetic clean up

ENH: Test now support passing argument, thus instead of iterating over all images, you can pass the input dicom images and the reference image. This is pretty usefull in debug time