From mboxrd@z Thu Jan 1 00:00:00 1970 Path: main.gmane.org!not-for-mail From: "Stefan Monnier" Newsgroups: gmane.emacs.cc-mode.general,gmane.emacs.devel Subject: Re: regression: filling comments in C++ code (today's CVS) Date: Sat, 25 Jan 2003 20:48:16 -0500 Sender: cc-mode-help-admin@lists.sourceforge.net Message-ID: <200301260148.h0Q1mGX18192@rum.cs.yale.edu> References: <200301171719.h0HHJOP18943@rum.cs.yale.edu> <5b65sl53l3.fsf@lister.roxen.com> <200301201654.h0KGsDJ10072@rum.cs.yale.edu> <5by95b1d0m.fsf@lister.roxen.com> NNTP-Posting-Host: main.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: main.gmane.org 1043545685 16105 80.91.224.249 (26 Jan 2003 01:48:05 GMT) X-Complaints-To: usenet@main.gmane.org NNTP-Posting-Date: Sun, 26 Jan 2003 01:48:05 +0000 (UTC) Cc: "Stefan Monnier" , "Robert Anderson" , emacs-devel@gnu.org, bug-cc-mode@gnu.org Return-path: Original-Received: from lists.sourceforge.net ([66.35.250.206] helo=sc8-sf-list2.sourceforge.net) by main.gmane.org with esmtp (Exim 3.35 #1 (Debian)) id 18cbtz-0004BS-00 for ; Sun, 26 Jan 2003 02:48:03 +0100 Original-Received: from sc8-sf-list1-b.sourceforge.net ([10.3.1.13] helo=sc8-sf-list1.sourceforge.net) by sc8-sf-list2.sourceforge.net with esmtp (Exim 3.31-VA-mm2 #1 (Debian)) id 18cbv0-0007hQ-00; Sat, 25 Jan 2003 17:49:06 -0800 Original-Received: from fencepost.gnu.org ([199.232.76.164]) by sc8-sf-list1.sourceforge.net with esmtp (Cipher TLSv1:DES-CBC3-SHA:168) (Exim 3.31-VA-mm2 #1 (Debian)) id 18cbun-00041Z-00 for ; Sat, 25 Jan 2003 17:48:53 -0800 Original-Received: from monty-python.gnu.org ([199.232.76.173]) by fencepost.gnu.org with esmtp (Exim 4.10) id 18cbul-0008JG-00 for bug-cc-mode@gnu.org; Sat, 25 Jan 2003 20:48:51 -0500 Original-Received: from mail by monty-python.gnu.org with spam-scanned (Exim 4.10.13) id 18cbuh-0003mq-00 for bug-cc-mode@gnu.org; Sat, 25 Jan 2003 20:48:48 -0500 Original-Received: from rum.cs.yale.edu ([128.36.229.169]) by monty-python.gnu.org with esmtp (Exim 4.10.13) id 18cbuF-0002pq-00; Sat, 25 Jan 2003 20:48:20 -0500 Original-Received: (from monnier@localhost) by rum.cs.yale.edu (8.11.6/8.11.6) id h0Q1mGX18192; Sat, 25 Jan 2003 20:48:16 -0500 X-Mailer: exmh version 2.4 06/23/2000 with nmh-1.0.4 Original-To: Martin Stjernholm Errors-To: cc-mode-help-admin@lists.sourceforge.net X-BeenThere: cc-mode-help@lists.sourceforge.net X-Mailman-Version: 2.0.9-sf.net Precedence: bulk List-Help: List-Post: List-Subscribe: , List-Id: Bug reports, feature requests, and general talk about CC Mode. List-Unsubscribe: , List-Archive: X-Original-Date: Sat, 25 Jan 2003 20:48:16 -0500 Xref: main.gmane.org gmane.emacs.cc-mode.general:346 gmane.emacs.devel:11071 X-Report-Spam: http://spam.gmane.org/gmane.emacs.devel:11071 > > Ah, so it's because filladapt does not faithfully implement the > > fill-paragraph interface. > > Well, it's not that Kyle E Jones just was being stupid when he wrote > filladapt. He can't do it any better way since there's no hook to > replace fill-context-prefix which implements adaptive-fill-mode. I don't understand. Why can't filladapt check (and call) fill-paragraph-function before doing anything else ? > It'd be nice if fill.el provided a function that only did the pure > text filling between two positions, preferably also with the option to Isn't that what fill-region-as-paragraph does ? > keep a fill postfix or to fill inside a rectangle. This part is indeed missing, obviously. > > For the sake of correctness in cases other than filladapt, how about > > setting fill-paragraph-function (while still keeping the rebinding > > of M-q) so that calling fill-paragraph DTRT. > > That wouldn't work when filladapt is used, but it's perhaps good That would help in the "normal" case, so the question is not "would it work with filladapt" but "would it hurt with filladapt". AFAIK it can't hurt, but I don't know enough about filladapt (or about cc-mode's filling) to be sure. Stefan ------------------------------------------------------- This SF.NET email is sponsored by: SourceForge Enterprise Edition + IBM + LinuxWorld = Something 2 See! http://www.vasoftware.com