Indentation

Debug vs Warning

Forget to commit this indentation fix

Try to track 'FIXME' VM problem

Comments

Try to track the "FIXME" problem for VM

revert to old printf syntax (better warning on some compilers than error on some
others!)

STYLE: Remove personal comments...make patch more difficult

Fix mouse slipping (derapage de mulot, in french language)

ENH: update comment to reflect API change

ENH: Adding Dart support in openjpeg

ENH: Ok dashboard looks clean enough turning MPEG on

ENH: Saving demo code

ENH: A user does not know (and should not) what we internally use for representing a dicom tag, let him use TagKey instead

Some identation

Some identation, to make the code reader friendly

Avoid some warnings

Try to track BinEndian troubles

Apply Laurent Guigues' advice to avoid icc warnings.

Try to track BigEndian troubles

Avoid warning

Avoid warnings on C/C++ syle comments

WTF is cvs doing?
(Lost some lines in my source -> Seg fault on continuous buid.
I comment out the broken piece of code)

Misstyping.

Avoid poluting the output, right now

Avoid warning

Avoid warning

Misstyping

Avoid warning

Avoid warning

Avoid C/C++ comment style

Avoid warning

To avoid warnings

Use fh->SetKeepMediaStorageSOPClassUID(true);
( Since we just Anonymized/ReWrited the file, we know no modification
  was performed on the pixels.
  We don't want this image appears as a 'Secondary Captured image'
 )

Comments.

- Add Method void SetKeepMediaStorageSOPClassUID (bool v).

 /// \brief if user knows he didn't modify the pixels (e.g. he just anonymized
 ///        or rewrite the file), he is allowed to ask to keep the original
 ///        'Media Storage SOP Class UID' and 'Image Type'

- Enforce CheckMandatoryElements()

Add Method ComputeFullLength(), to allow further computaion of element 0x0000
(Group length) of each 'zero level' group.

BUG: Report if getcwd did not succeed

PrintFile uses debug and warning flags

Comments

Debug vs Warning

Adding private variables CurrentGroup and CurrentElem allows inner methods to
know the Entry there are working on.

Flags Debug and Warning may be used separately.
--> Setting on Debug sets also Warning
--> Setting off Warning is applied only if Debug is off.
Note :
Debug On :   Developer wants to track a missbehaviour in gdcm
Warning on : User wants to be warned about oddities in the files.

Fix bug in DicomDir MetaElements.
Thx to Chen Chen for providing data.

Enforce TestValidate, and run it.
(Just to have an overwiew of oddities in the files.
Never breaks the test suite)

ENH: Cleanup the test

ENH: Adding TRY_RUN

ENH: Apparently some gcc version are seriously broken

Avoid warnings

Avoid bcc warnings

COMP: Fix warning

ENH: Rework the 1000 limit problem, the code is better but not perfect, at least we will get an error

Comments

Remove useless commented out code

Dealing first with the most frequently used cases should speed up the process.

COMP: Attempt to please icc

Do not prefix filenames with /, that hangs a lot od clinical softwares

COMP: Fix warning about unused var... cant remember what I was trying to do, maybe a faster memcpy?

COMP: Fix warning

COMP: Fix warning about unused var

ENH: Apply patch from Antonin

To avoid C / C++ style warnings

Avoid bool / int warning

Use CopyBinArea() better than SetEntryBinArea(), that doesn't make the job (?!?)

Move fmiv update to WriteContent() method
(Same place than other updaters)

Fix Media Storage SOP Instance UID' Tag

New() vs new()

Modify TestValidate to compile with New() vs new

Avoid warnings

Comments

Don't warn user with useless messages.

- Set a decent value to 'Media Storage SOP Instance UID' and
  'Implementation Class UID' for DICOMDIR

- Add DicomDirMeta::ComputeGroup0002Length() to fix (now variable)
  group 0002 length

Comments

Add TestValidate

Add TestValidate, to be used as stand alone exec.

Avoid warning

Avoid some warnings

Oops.
Fornutalely we never had to use FD values !

(try to) affect a legal to 'File Meta Information Version'

'File Meta Information Version' has a mandatory length of 2.

Private GE Syntax is not only DLX

Comments

in CheckMandatoryElements :
- Patient Orientation, if missing, is computed from Image Orientation (Patient).
- 0x0008,0x0050 is kept as is (defaulted as "").

Avoid propagating uneven lengthes.

Oops.
I forgot to re-introduce 16 bits image writting on Big Endian processors.

Don't write 'Zero length' Sequences.

Enforce Validator:
 - to check uneven lengthes
 - to stop yelling on Implicit VR Private Tags

Comments

Add some verbosity when something wrong occurs.

Identation.

Use arguments manager.

exXCoherentFileSet can be used to check splitting on a given Tag value.

Looping 10000000 times should be enought

Checking the endianness at Pixels Reading time should help the test to work on
Big Endian Processors.

COMP: Fix comp problem on Win32 (encoder does not compile yet...)

Fix Debug vs Warning

Comments

>   // Don't waste time checking tags where VM is OB and OW, since we know
>   // it's allways 1, whatever the actual length (found on disc)

Don't check VM for OB and OW tags : we know it's allways 1, whatever the actual
length (on disc) is.

Diving by 2 image sizes should make tests 4 times quicker ;-)