[Debian-med-packaging] Bug#816607: Patch is good !

[Mathieu Malaterre]

Control: tags -1 patch fixed-upstream


On Wed, Apr 27, 2016 at 2:43 PM, Mathieu Malaterre <malat at debian.org> wrote:
> Hi Victor,
>
> On Wed, Mar 30, 2016 at 11:53 PM, Victor Derks <vbaderks at gmail.com> wrote:
>> Hi Sjors,
>>
>> I pushed the fix to the master and 1.x-master branch on github.
>>
>> The sample file can now be encoded to JPEG-LS and back to the .pnm format.
>> If I decode it back with CharLS the image is however not 100% identical. If
>> I decode it back with the ITU LOCO exe, the original and the encoded\decoded
>> files are identical.
>> So the encoding to JPEG-LS seems to be ok, but there seems another issue in
>> the charls decoding process (maybe the fact the width is odd of this sample
>> image?).
>>
>> Will keep you posted
>
> Hum your patch make the attached DICOM file works. So indeed it does
> solve some issues.

Well your patch is actually correct. The issue is in fact within DCMTK
and handling of Big Endian Transfer Syntax. This does not affect GDCM.

Thanks very much for fixing it !