[Gdcm-hackers] [ gdcm-Bugs-3349280 ] TestSystem fails on MSVC 2010

[SourceForge.net <no...@so...>]

Bugs item #3349280, was opened at 2011-07-02 01:40
Message generated for change (Settings changed) made by malat
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=739587&aid=3349280&group_id=137895

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: General (default)
Group: 2.2.0
Status: Closed
Resolution: Fixed
Priority: 5
Private: No
Submitted By: Niels Dekker (niels_dekker)
Assigned to: Mathieu Malaterre (malat)
>Summary: TestSystem fails on MSVC 2010

Initial Comment:
TestSystem fails on the latest release, GDCM 2.0.18, when compiled using MSVC 2010 (32 bits) Does this mean that the system I'm using is incompatible with GDCM?

Anyway, the error message is "size_t is diff from uintmax_t: 4 8". This seems inappropriate, because TestSystem does not compare size_t to uintmax_t. It explicitly ignores sizeof(uintmax_t):

  unsigned long size1 = sizeof(off_t);
  unsigned long size2 = sizeof(size_t);
  unsigned long size3 = sizeof(uintmax_t);
  (void)size3;
  unsigned long size4 = sizeof(std::streamsize);
  ...
  if( size2 != size4 )
    {
    std::cerr << "size_t is diff from uintmax_t: " << size2 << " " << size4 << std::endl;
    return 1;
    }


----------------------------------------------------------------------

Comment By: Mathieu Malaterre (malat)
Date: 2011-12-30 10:29

Message:
This looks like this has been fixed by
6b60b7ac423c336e0d3c189336197b0b55acb9fe

----------------------------------------------------------------------

Comment By: Mathieu Malaterre (malat)
Date: 2011-07-13 12:21

Message:
There is also the same issue with the proper type for accumulator in
SequenceOfFragments::ComputeByteLength()

----------------------------------------------------------------------

Comment By: Mathieu Malaterre (malat)
Date: 2011-07-13 12:18

Message:
There is a design issue in gdcm::System::FileSize
Originally size_t was used, but apparently for this system size_t !=
uintmax_t

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=739587&aid=3349280&group_id=137895