Return-Path: <tiff-bounces@lists.maptools.org>
Received: from inas028.isp.belgacom.be (localhost [127.0.0.1])
        by inav022.isp.belgacom.be (8.12.11/8.12.11/Skynet-IN-AV-3.00) with ESMTP id j9IE2wOI027556
        for <joris.at.lebbeke@skynet.be>; Tue, 18 Oct 2005 16:02:58 +0200
        (envelope-from <tiff-bounces@lists.maptools.org>)
Received: from inmx001.isp.belgacom.be (inmx001.isp.belgacom.be [195.238.5.1])
        by inas028.isp.belgacom.be (8.12.11/8.12.11/Skynet-IN-AS-2.03) with ESMTP id j9IE2qY2023363
        for <joris.at.lebbeke@skynet.be>; Tue, 18 Oct 2005 16:02:54 +0200
        (envelope-from <tiff-bounces@lists.maptools.org>)
Received: from duke.maptools.org (r153.dmsolutions.ca [209.217.116.153] (may be forged))
        by inmx001.isp.belgacom.be (8.12.11/8.12.11/Skynet-IN-2.32) with ESMTP id j9IE2ia1029042
        for <joris.at.lebbeke@skynet.be>; Tue, 18 Oct 2005 16:02:44 +0200
        (envelope-from <tiff-bounces@lists.maptools.org>)
Received: from duke.maptools.org (duke.maptools.org [127.0.0.1])
	by duke.maptools.org (8.12.11/8.12.10) with ESMTP id j9IDwOic012504;
	Tue, 18 Oct 2005 09:58:34 -0400
Received: from qproxy.gmail.com (qproxy.gmail.com [72.14.204.197])
	by duke.maptools.org (8.12.11/8.12.10) with ESMTP id j9IDwNq6012501
	for <tiff@lists.maptools.org>; Tue, 18 Oct 2005 09:58:23 -0400
Received: by qproxy.gmail.com with SMTP id p30so1308376qba
	for <tiff@lists.maptools.org>; Tue, 18 Oct 2005 06:58:22 -0700 (PDT)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com;
	h=received:message-id:date:from:sender:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
	b=MReu5EwAXZjIZ2OvjNaYafW0gAzB3nHTeUJqmxxaBnHdqQrjuZUcfYQ4LdisC/XTj2RxKNRn+sUlHTAkugPTdAT0P7EwNOFCwIhAx6ih0KUzTO4r4Ixd+jr9iBmOi4Mgvl1N3PblhO5U9nBRWInQcfPtx9EGZB/RPtkXld28FSI=
Received: by 10.65.112.7 with SMTP id p7mr3061101qbm;
	Tue, 18 Oct 2005 06:58:22 -0700 (PDT)
Received: by 10.65.155.9 with HTTP; Tue, 18 Oct 2005 06:58:22 -0700 (PDT)
Message-ID: <931f8ea90510180658n245d9e8bxb243487e7cb35464@mail.gmail.com>
Date: Tue, 18 Oct 2005 09:58:22 -0400
From: Frank Warmerdam <warmerdam@pobox.com>
To: Joris <joris.at.lebbeke@skynet.be>
Subject: Re: [Tiff] Small bug report, tif_fax3.c and tif_row
In-Reply-To: <001001c5d3e5$f0c94160$8aa4f651@dfgrrxsknr4e>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Disposition: inline
References: <000e01c5d3d7$78b3b600$8aa4f651@dfgrrxsknr4e>
	<001001c5d3e5$f0c94160$8aa4f651@dfgrrxsknr4e>
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by duke.maptools.org id
	j9IDwNq6012501
Cc: TIFF mailing list <tiff@lists.maptools.org>
X-BeenThere: tiff@lists.maptools.org
X-Mailman-Version: 2.1.4
Precedence: list
List-Id: TIFF and libtiff discussion <tiff.lists.maptools.org>
List-Unsubscribe: <http://lists.maptools.org/mailman/listinfo/tiff>,
	<mailto:tiff-request@lists.maptools.org?subject=unsubscribe>
List-Archive: <http://lists.maptools.org/mailman/private/tiff>
List-Post: <mailto:tiff@lists.maptools.org>
List-Help: <mailto:tiff-request@lists.maptools.org?subject=help>
List-Subscribe: <http://lists.maptools.org/mailman/listinfo/tiff>,
	<mailto:tiff-request@lists.maptools.org?subject=subscribe>
Sender: tiff-bounces@lists.maptools.org
Errors-To: tiff-bounces@lists.maptools.org
X-UIDL: 000000000023ceae.00000d47
X-MSP-STORE: Tue, 18 Oct 2005 16:03:02 +0200
Status:   

On 10/18/05, Joris <joris.at.lebbeke@skynet.be> wrote:
> Joris wrote:
> > Can anyone shed some light on this? Where, in the LibTiff design, is
> > tif->tif_row actually meant to be incremented? How is this supposed to
> > work inside strip/tile decoders?
>
> Upon investigation, it seems that the same problem is there for most
> decoders. I'm very confused, for these reasons:

Joris,

I was vaguely aware there some significant confusion in the handling
of the tif_row.  I agree that something needs to be done to clarify the
roll of different variables, and cleanup their use in libtiff.   Your proposal
sounds reasonable, unless Andrey has a concern.

> How do people feel about this? If I do this on some decoders, will it
> end up in CVS?

Better yet, how about we provide you with CVS access so you can
commit it yourself?

Best regards,
--
---------------------------------------+--------------------------------------
I set the clouds in motion - turn up   | Frank Warmerdam, warmerdam@pobox.com
light and sound - activate the windows | http://pobox.com/~warmerdam
and watch the world go round - Rush    | Geospatial Programmer for Rent

_______________________________________________
Tiff mailing list: Tiff@lists.maptools.org
http://lists.maptools.org/mailman/listinfo/tiff
http://www.remotesensing.org/libtiff/
