From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Reiner Steib Newsgroups: gmane.emacs.devel Subject: Additional safe-local-variables (was: turn off safe-local-variable-values checking) Date: Thu, 06 Apr 2006 17:40:23 +0200 Message-ID: References: <9508491d0603300327i6da294bcp553f2f48df85d65f@mail.gmail.com> <8764lue3jv.fsf-monnier+emacs@gnu.org> <9508491d0604021712r78136dco5e33b05a3d39109b@mail.gmail.com> <87y7ynjt1y.fsf-monnier+emacs@gnu.org> <87odzhhxxo.fsf-monnier+emacs@gnu.org> <87slosgov3.fsf-monnier+emacs@gnu.org> Reply-To: Reiner Steib NNTP-Posting-Host: main.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: sea.gmane.org 1144338070 6714 80.91.229.2 (6 Apr 2006 15:41:10 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Thu, 6 Apr 2006 15:41:10 +0000 (UTC) Cc: Stefan Monnier , emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Thu Apr 06 17:41:05 2006 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by ciao.gmane.org with esmtp (Exim 4.43) id 1FRWbI-0006VD-PJ for ged-emacs-devel@m.gmane.org; Thu, 06 Apr 2006 17:40:49 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1FRWbH-0001J3-Qy for ged-emacs-devel@m.gmane.org; Thu, 06 Apr 2006 11:40:47 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1FRWb5-0001If-C1 for emacs-devel@gnu.org; Thu, 06 Apr 2006 11:40:35 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1FRWb2-0001G6-NV for emacs-devel@gnu.org; Thu, 06 Apr 2006 11:40:35 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1FRWb2-0001Fl-EA for emacs-devel@gnu.org; Thu, 06 Apr 2006 11:40:32 -0400 Original-Received: from [134.60.1.1] (helo=mail.uni-ulm.de) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA:32) (Exim 4.52) id 1FRWej-000441-E7; Thu, 06 Apr 2006 11:44:22 -0400 Original-Received: from bridgekeeper.physik.uni-ulm.de (bridgekeeper.physik.uni-ulm.de [134.60.10.123]) by mail.uni-ulm.de (8.13.6/8.13.6) with ESMTP id k36FeN1p021089; Thu, 6 Apr 2006 17:40:24 +0200 (MEST) Original-Received: by bridgekeeper.physik.uni-ulm.de (Postfix, from userid 170) id ABF1F11668; Thu, 6 Apr 2006 17:40:23 +0200 (CEST) Original-To: Richard Stallman Mail-Followup-To: rms@gnu.org, Stefan Monnier , emacs-devel@gnu.org X-Face: :6KQZ[nyoS_edmB.%gw-=)BYth^|2+Y+^cu%I$FSx!&>-'om>3H7A|M&n(V7fIo3P.; yo.b yq4$p; ZaBtkv)\}biaiBQe"mD}iib1AA@99-fZ7i*bLhNRVC&0Wkxg9)SH?oWc@{ In-Reply-To: (Richard Stallman's message of "Wed, 05 Apr 2006 21:11:39 -0400") User-Agent: Gnus/5.110004 (No Gnus v0.4) Emacs/22.0.50 (gnu/linux) X-DCC-INFN-TO-Metrics: gemini 1233; Body=3 Fuz1=3 Fuz2=3 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:52477 Archived-At: On Thu, Apr 06 2006, Richard Stallman wrote: > Interesting. I'd have to disagree: put it/them in the minor mode's file, > but with an ;;;###autoload cookie. > > That seems like a good method to achieve the goal > (which is that the property should always be there). > Would you please edit the comment? > > I think it's important to put those safe-local-variable properties at the > same spot as where the variable is declared. > > I agree, that will be better for maintenance, in case a change in the meaning > of the variable calls for a change in the property. I've done that for the new variable I've suggested and for some other variable (I didn't have time to process all of the list), e.g.: ;;;###autoload(put 'paragraph-separate 'safe-local-variable 'stringp) `page-delimiter' and `paragraph-start' were marked with string-or-null whereas the custom type doesn't allow nil. I used stringp then. Feel free to adjust the autoloads or the defcustoms. Maybe it would make sense to introduce `string-or-null-p' to avoid using (lambda (a) (or (stringp a) (null a))) in the autoloads. BTW, wouldn't it be possible to integrate the safe-local-variable properties into defcustom? Bye, Reiner. -- ,,, (o o) ---ooO-(_)-Ooo--- | PGP key available | http://rsteib.home.pages.de/