[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: KDE/kdelibs/kimgio
From:       Christoph Feck <christoph () maxiom ! de>
Date:       2011-01-18 16:13:20
Message-ID: 201101181713.20585.christoph () maxiom ! de
[Download RAW message or body]

On Tuesday 18 January 2011 15:37:15 Friedrich W. H. Kossebau wrote:
> Hi Christoph,
> 
> Lundi, le 17 janvier 2011, à 23:49, Christoph Feck a écrit:
> > SVN commit 1215179 by cfeck:
> > 
> > Fix writing JPEG 2000 files with non-US locale
> > 
> >  M  +5 -6      jp2.cpp
> > 
> > --- trunk/KDE/kdelibs/kimgio/jp2.cpp #1215178:1215179
> > @@ -128,7 +128,7 @@
> > 
> >                  newpos = io->size() - offset;
> >                  break;
> >          
> >          case SEEK_CUR:
> > -                newpos = io->size() + offset;
> > +                newpos = io->pos() + offset;
> > 
> >                  break;
> 
> That part of the changes seems to not match the commit message, is it an
> intended change or a mistake?

You are right, the commit message for this change is missing. Sorry.

The change is intended, though. I forgot that I fixed that some days ago while 
reviewing the code.

> Cheers
> Friedrich

Christoph Feck

[prev in list] [next in list] [prev in thread] [next in thread]