unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
To: Antoine Levitt <antoine.levitt@gmail.com>
Cc: emacs-devel@gnu.org
Subject: Re: [PATCH] Use correct indentation in customize
Date: Thu, 12 Aug 2010 11:01:26 +0200	[thread overview]
Message-ID: <jwvvd7gciq7.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <8762zi2j47.fsf@gmail.com> (Antoine Levitt's message of "Tue, 10 Aug 2010 23:33:28 +0100")

> I noticed that when customize writes its stuff to .emacs (or
> custom-file), the indentation of the accompanying comment is off by a
> column. This is a trivial issue, but I like to indent obsessively my
> files. Since I keep .emacs under version control, this generates useless
> diffs that have been bugging me for a while. So here's the patch to fix
> it, which should be a no-brainer to merge.

Thanks, installed,


        Stefan


> === modified file 'lisp/cus-edit.el'                                           
> --- lisp/cus-edit.el |  2010-04-20 18:52:07 +0000                              
> +++ lisp/cus-edit.el	2010-08-10 22:21:20 +0000
> @@ -4404,10 +4404,10 @@
>        (unless (bolp)
>  	(princ "\n"))
>        (princ "(custom-set-variables
> -  ;; custom-set-variables was added by Custom.
> -  ;; If you edit it by hand, you could mess it up, so be careful.
> -  ;; Your init file should contain only one such instance.
> -  ;; If there is more than one, they won't work right.\n")
> + ;; custom-set-variables was added by Custom.
> + ;; If you edit it by hand, you could mess it up, so be careful.
> + ;; Your init file should contain only one such instance.
> + ;; If there is more than one, they won't work right.\n")
>        (dolist (symbol saved-list)
>  	(let ((spec (car-safe (get symbol 'theme-value)))
>  	      (value (get symbol 'saved-value))
> @@ -4480,10 +4480,10 @@
>        (unless (bolp)
>  	(princ "\n"))
>        (princ "(custom-set-faces
> -  ;; custom-set-faces was added by Custom.
> -  ;; If you edit it by hand, you could mess it up, so be careful.
> -  ;; Your init file should contain only one such instance.
> -  ;; If there is more than one, they won't work right.\n")
> + ;; custom-set-faces was added by Custom.
> + ;; If you edit it by hand, you could mess it up, so be careful.
> + ;; Your init file should contain only one such instance.
> + ;; If there is more than one, they won't work right.\n")
>        (dolist (symbol saved-list)
>  	(let ((spec (car-safe (get symbol 'theme-face)))
>  	      (value (get symbol 'saved-face))




      reply	other threads:[~2010-08-12  9:01 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-10 22:33 [PATCH] Use correct indentation in customize Antoine Levitt
2010-08-12  9:01 ` Stefan Monnier [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=jwvvd7gciq7.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=antoine.levitt@gmail.com \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).