Doxygenation

Doxygen

In order to avoid confusing user with 'public' methods he is not allowed to use,
- move *all* the methods of PixelReadConvert, PixelWriteConvert to 'private'.
- declare as friend class FileHelper

Doxygenation

Doxgenation :
+ remove header for Deprecated methods, hoping  Doxygen stop yelling

Doxygen

Doxygenation

Add TestInline

Add some comments for better understanding the gdcm::LD_NOSHADOW,
 gdcm::LD_NOSEQ options.

Add 2 stupif inline functions (to be removed later) , to be able to run
TestInline on all the platforms.

Add a test to 'visualize' the difference betwwen inline functions and regular
ones.
(Actually, on my Linux box, I don't see *any* difference !
I'dd like to see what happens on other platforms)

Uncomment Private GE transger syntax part.
(The trouble came from Write in the Mandatory elems processing
-free twice the same one :-( -

Speed up -a little bit- the Loading, avoiding to parse any Sequence, not only
shadow ones.
(any way *all* the sequences are skipped a prpcessing time.)

Comments

Fix GetZOrigin().
Thx to Marco Feuerstein for reporting bug.

BUG: 1-n VM are also allowed to be empty

BUG: Fix VM for 60xx,0050. it is 2 not 1

ENH: Handle the case where the element is empty, previous GetValueCount was reported erronurously as 1

   * Fix compilation errors in the Python part
   -- BeNours

ENH: oh my god, none of the old ACR-NAME entry was correct, MR-MONO2-12-an2.acr is now passing.

ENH: How in the world could it be 1

BUG: (0008,0041) is LO and not US

BUG: 1-n VM was not handle properly

ENH: Rearchitecture the test to test one file at a time

BUG: ooops

ENH: You can now do os << TagKey .... rahhhhh finally

BUG: TestAllPrint actually is doing something now. Update to compile TestAllVM

ENH: Adding a test for VM and validity... should allow us to check our own dict :-P

BUG: I believe 0020,0030 has a VM of 3 an not 1

COMP: Fix comp, and use IsValueCountValid in an assert just for kick

ENH: Adding IsValueCountValid. Should save a lot of time for dev, since we you really want to do is compare against the VM to find out if the operation is valid or not.

   * Reactivate TestVR because the problem was already solved
   -- BeNours

Doxygen

Doxygen disagrees with comments on deprecated methods

// -->
// --> Can't understand *why* TestWriteSimple fails since the commit
// --> for dealing with GE Private Transfer Syntax.
// --> commented out, to be sure the troubles started from here.
// --> JP

   * Fix the unix installation for the python part
   -- BeNours

Shorten up the code, and avoid memory leaks.
Thx to Benoit for reporting trouble.

   * Fix compilation error for the Python part
   -- BeNours

   * Remove obvious Example that don't pass on Windows
   -- BeNours

   * Fix bug... sorry, too quick commit when correcting memory leaks
   -- BeNours

   * Fix compilation errors due to the 'virtual' key-word
   -- BeNours

   * Fix memory leaks
   -- BeNours

Comment out TestVR.
(probabely useless -except for testcoverage ?- and polutes the dashbord.
Uncomment it when the mess is fixed.

Doxygen disagrees with commented 'deprecated' methods

Add exInline.cxx, to 'visualize' the difference betwwen inline an not inline
functions.
Well...
We see there is no difference ?!?

Comments

Normalization

add comments

Add some prints

Use SetLoadMode(gdcm::LD_ALL), better than 0

   * Fix bug when reading the DicomVR.dic file
   -- BeNours

   * bug fix in the Python demos, due to the recent changes
   -- BeNours

Comment out, using C style, C++ style comments, to avoid warnings.

   * Remove compilation warnings
   -- BeNours

Doxygenation

Seems some compiler disagree with trailing semi-column at end of
 gdcmTypeMacro( ...)

COMP: Fix comp on VS6... not sure what it was supposed to do

   * Add a RefCounter object that is deleted only when it's reference count is
     to 0.
   * DicomEntry (thus, DictEntry) inherits from RefCounter... next stage to do,
     all classes inherit from RefCounter
   * Remove the VirtualDictEntry use, now useless due to the reference
     counting. So, there will be no memory leaks.
   -- BeNours

COMP: I don't think this is needed

ENH: Add a very bizarre case

Remove deprecated method names 'GetFirstCoherentFileList' 'GetNextCoherentFileList'
replace by GetFistSingleSerieUIDFileSet, GetNextSingleSerieUIDFileSet

ENH: Fix TestVR on VS6, while preserving inline optimization on working compiler

ENH: Previous code did not work on VS6

ENH: Saving a comment for a better approach in cmake 2.2 (later)

COMP: Fix bcc55

ENH: Saving some stuff from an explicit image...if I remember correctly

STYLE: Some minor comment

   * Remove missed FASTTAGKEY (due to the integration of the
     gdcm::TagKey class).
     Thx to Mathieu

Last (?) stage of Private GE Syntax Transer name normalization

   * Fix some compilation warnings
   * Amelioration of Test's output
   -- BeNours

   * Coding style
   -- BeNours

Oops.
Forgot to commit this one.
Thx, Benoit!

-> Fix the pb of 'Private GE Transfer Syntax' for 16 bits images.
-> The expressions related to 'ImplicitVRLittleEndianDLXGE' are changed to talk
about 'Implicit VR Big Endian Transfer Syntax', that's the term commonly used in
the literature.
--> Merge ITK comments (much more detailled than mines) in the gdcm code.

Coding style

Try to make a more kosher Dicom Header (improve various UID consistebcy)

In order to get UID without padding to even-length string, add a '.' before
random number.

Fix 'delimitors' Print()

ENH: Tons of change now support creation of basic table offset to please dicom reader like dicomworks.

STYLE: Imager Pixel Spacing

ENH: Still cleaning up stuff

ENH: properly return 0 when everything was ok, 1 otherwise

STYLE: Fix comment

COMP: That was simply a typo

COMP: remove suspend comp warn

ENH: Reorganize the code to prepare multiple fragments

COMP: Please VS6

   * Remove the Key information in Entry
   * Create a DicomEntry parent of DictEntry
   * Remap correctly the TagKey
   -- BeNours

   * Remove memory leaks
   -- BeNours

   * Fix compilation errors for the Python part
   * Fix errors in the Python tests
   -- BeNours

   * Amelioration of error messages
   -- BeNours

   * Error message amelioration
   -- BeNours

*** empty log message ***

   * Fix some bugs
   -- BeNours

STYLE: A Debug statement is not a warning, a warning is not an error.. you get the picture...

ENH: Hoorray the first gdcm-jpeg image written from scratch :)

ENH: Getting real close to having JPEG support in gdcm... jpeg offset table still missing

ENH: Several change, fix compilation warnings, and update to newer API...no comment

COMP: Fix other people warnings

ENH: Adding *very* experimental support for writting DICOM file as JPEG

ENH: Adding example on how to write a dicom file as jpeg