X-Spam-Check-By: sourceware.org
Date: Wed, 20 Sep 2006 20:50:04 -0700
From: Christopher Layne <clayne@anodized.com>
To: cygwin@cygwin.com
Subject: Re: bash-3.1-7$B!!(BBUG
Message-ID: <20060921035004.GD6790@ns1.anodized.com>
References: <ee9oa9$sj$1@sea.gmane.org> <ee9q53$74g$1@sea.gmane.org> <loom.20060913T234039-426@post.gmane.org> <ee9vg0$qeb$1@sea.gmane.org> <45089854.8010705@scytek.de> <20060914001902.GB24899@trixie.casa.cgf.cx> <4508ABAF.5090408@scytek.de> <20060914020737.GC24899@trixie.casa.cgf.cx> <45093972.7080606@byu.net> <45098A3B.4060105@scytek.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <45098A3B.4060105@scytek.de>
User-Agent: Mutt/1.5.11
X-Assp-Spam-Prob: 0.00000
X-Assp-Whitelisted: Yes
X-Assp-Envelope-From: clayne@ns1.anodized.com
X-Assp-Intended-For: cygwin@cygwin.com
X-IsSubscribed: yes
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Precedence: bulk
List-Unsubscribe: <mailto:cygwin-unsubscribe-archive-cygwin=delorie.com@cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.com

On Thu, Sep 14, 2006 at 12:58:36PM -0400, Volker Quetschke wrote:
> > +#ifdef __CYGWIN__
> > +  /* lseek'ing on text files is problematic; lseek reports the true
> > +     file offset, but read collapses \r\n and returns a character
> > +     count.  We cannot reliably seek backwards if nr is smaller than
> > +     the seek offset encountered during the read, and must instead
> > +     treat the stream as unbuffered.  */
> > +  if ((bp->b_flag & (B_TEXT | B_UNBUFF)) == B_TEXT)
> ------------------------^^^^^^^^^^^^^^^^^      ^^^^^^
> part of the patch looks suspicious to me. You probably just want to test
> if the LHS expression is true.
> 
>   Volker

It's called a mask.

-cl

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

