From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Chong Yidong Newsgroups: gmane.emacs.bugs Subject: bug#8657: 24.0.50; Emacs code that seems to use `*-single-char-property-change' incorrectly Date: Tue, 07 Feb 2012 13:57:45 +0800 Message-ID: <87obtbxk12.fsf@gnu.org> References: <87bopcnjt6.fsf@gnu.org> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: dough.gmane.org 1328594293 25842 80.91.229.3 (7 Feb 2012 05:58:13 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Tue, 7 Feb 2012 05:58:13 +0000 (UTC) Cc: 8657@debbugs.gnu.org To: Katsumi Yamaoka Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Tue Feb 07 06:58:12 2012 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([140.186.70.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Rue4C-00038w-BY for geb-bug-gnu-emacs@m.gmane.org; Tue, 07 Feb 2012 06:58:12 +0100 Original-Received: from localhost ([::1]:58395 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rue4B-0006I2-Ou for geb-bug-gnu-emacs@m.gmane.org; Tue, 07 Feb 2012 00:58:11 -0500 Original-Received: from eggs.gnu.org ([140.186.70.92]:35142) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rue49-0006Hx-2u for bug-gnu-emacs@gnu.org; Tue, 07 Feb 2012 00:58:10 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Rue47-00074q-Vy for bug-gnu-emacs@gnu.org; Tue, 07 Feb 2012 00:58:09 -0500 Original-Received: from debbugs.gnu.org ([140.186.70.43]:54428) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rue47-00074m-UM for bug-gnu-emacs@gnu.org; Tue, 07 Feb 2012 00:58:07 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.72) (envelope-from ) id 1Rue4z-0007y2-ML for bug-gnu-emacs@gnu.org; Tue, 07 Feb 2012 00:59:01 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Chong Yidong Original-Sender: debbugs-submit-bounces@debbugs.gnu.org Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 07 Feb 2012 05:59:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 8657 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: notabug Original-Received: via spool by 8657-submit@debbugs.gnu.org id=B8657.132859433130610 (code B ref 8657); Tue, 07 Feb 2012 05:59:01 +0000 Original-Received: (at 8657) by debbugs.gnu.org; 7 Feb 2012 05:58:51 +0000 Original-Received: from localhost ([127.0.0.1]:58051 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1Rue4o-0007xe-GV for submit@debbugs.gnu.org; Tue, 07 Feb 2012 00:58:51 -0500 Original-Received: from fencepost.gnu.org ([140.186.70.10]:60980 ident=Debian-exim) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1Rue4m-0007xY-KA for 8657@debbugs.gnu.org; Tue, 07 Feb 2012 00:58:49 -0500 Original-Received: from [155.69.16.237] (port=57768 helo=furball) by fencepost.gnu.org with esmtpsa (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1Rue3r-0000b4-T6; Tue, 07 Feb 2012 00:57:52 -0500 In-Reply-To: (Katsumi Yamaoka's message of "Tue, 07 Feb 2012 07:56:54 +0900") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.93 (gnu/linux) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.13 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 2) X-Received-From: 140.186.70.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:56625 Archived-At: Katsumi Yamaoka writes: > But in XEmacs next-single-char-property-change will return nil > if no property change is found. Ah OK, thanks for checking. > So I've modified the gnus-sum.el code into: > > (if (featurep 'xemacs) > (or (next-single-char-property-change end 'invisible) > (point-max)) > (next-single-char-property-change end 'invisible)) I suggest just restoring the old code, and adding a comment about why the `or' is there.