From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Lawrence Mitchell Newsgroups: gmane.emacs.bugs Subject: bug#8150: 23.2; cperl-uncomment-region is NOT an inverse of cperl-comment-region Date: Fri, 17 Feb 2012 12:47:10 +0000 Message-ID: <87d39dsk35.fsf@ed.ac.uk> References: <878vwxgb07.fsf@cds.caltech.edu> <0aipw177lj.fsf@fencepost.gnu.org> <20110302203816.7e8e2fa1@smart> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: dough.gmane.org 1329482902 29093 80.91.229.3 (17 Feb 2012 12:48:22 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Fri, 17 Feb 2012 12:48:22 +0000 (UTC) To: 8150@debbugs.gnu.org Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Fri Feb 17 13:48:18 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 1RyNEX-0002KZ-Ck for geb-bug-gnu-emacs@m.gmane.org; Fri, 17 Feb 2012 13:48:17 +0100 Original-Received: from localhost ([::1]:46183 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RyNEW-0001fX-Em for geb-bug-gnu-emacs@m.gmane.org; Fri, 17 Feb 2012 07:48:16 -0500 Original-Received: from eggs.gnu.org ([140.186.70.92]:60866) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RyNET-0001fS-46 for bug-gnu-emacs@gnu.org; Fri, 17 Feb 2012 07:48:14 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RyNEP-0004zt-9Z for bug-gnu-emacs@gnu.org; Fri, 17 Feb 2012 07:48:13 -0500 Original-Received: from debbugs.gnu.org ([140.186.70.43]:39812) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RyNEP-0004zp-3N for bug-gnu-emacs@gnu.org; Fri, 17 Feb 2012 07:48:09 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.72) (envelope-from ) id 1RyNGD-0002Ft-OG for bug-gnu-emacs@gnu.org; Fri, 17 Feb 2012 07:50:01 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Lawrence Mitchell Original-Sender: debbugs-submit-bounces@debbugs.gnu.org Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Fri, 17 Feb 2012 12:50:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 8150 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 8150-submit@debbugs.gnu.org id=B8150.13294829668610 (code B ref 8150); Fri, 17 Feb 2012 12:50:01 +0000 Original-Received: (at 8150) by debbugs.gnu.org; 17 Feb 2012 12:49:26 +0000 Original-Received: from localhost ([127.0.0.1]:43435 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1RyNFa-0002Ek-Fc for submit@debbugs.gnu.org; Fri, 17 Feb 2012 07:49:24 -0500 Original-Received: from treacle.ucs.ed.ac.uk ([129.215.16.102]:57365) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1RyNFQ-0002ER-Ll for 8150@debbugs.gnu.org; Fri, 17 Feb 2012 07:49:20 -0500 Original-Received: from lmtp1.ucs.ed.ac.uk (lmtp1.ucs.ed.ac.uk [129.215.149.64]) by treacle.ucs.ed.ac.uk (8.13.8/8.13.4) with ESMTP id q1HCl3j6013877 for <8150@debbugs.gnu.org>; Fri, 17 Feb 2012 12:47:08 GMT Original-Received: from e4300lm (e4300lm.epcc.ed.ac.uk [129.215.63.156]) (authenticated user=lmitche4 mech=PLAIN bits=0) by lmtp1.ucs.ed.ac.uk (8.13.8/8.13.7) with ESMTP id q1HCkl3H006519 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT) for <8150@debbugs.gnu.org>; Fri, 17 Feb 2012 12:47:02 GMT In-Reply-To: (Lawrence Mitchell's message of "Thu, 03 Mar 2011 10:06:53 +0000") User-Agent: Gnus/5.110018 (No Gnus v0.18) Emacs/24.0.92 (gnu/linux) X-Edinburgh-Scanned: at treacle.ucs.ed.ac.uk with MIMEDefang 2.60, Sophie, Sophos Anti-Virus, Clam AntiVirus X-Scanned-By: MIMEDefang 2.60 on 129.215.16.102 X-Scanned-By: MIMEDefang 2.52 on 129.215.149.64 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:56962 Archived-At: I'm chasing up open bugs I've participated in, probably too late for 24.1, but just to see if anything's doing. Is there any objection to applying the patch in this bug report? Lawrence Mitchell wrote: [...] > Note that the problem is that (uncomment-region A B 1) is not the > inverse of (comment-region A B 1). Irrespective of the major > mode in effect: > emacs -Q > C-x h > C-1 M-x comment-region RET > C-x h > C-1 M-x uncomment-region RET > Note how there is now a space at the beginning of every line in > the *scratch* buffer. > This is due to the following code in uncomment-region-default: > | (if (null arg) (delete-region (point-min) (point)) > | (skip-syntax-backward " ") > | (delete-char (- numarg)) > # foo bar > ^ point is here when the above code is called. > If an argument is supplied, arg is non-nil and so to delete the > comment we first move backwards over whitespace and then delete > as many comment characters as specified by the argument (1 in > this case).