From: Stefan Monnier <monnier@iro.umontreal.ca>
To: Leo <sdl.web@gmail.com>
Cc: 5937@debbugs.gnu.org
Subject: bug#5937: 23.1.95; why saving empty abbrev tables
Date: Mon, 12 Apr 2010 14:32:07 -0400 [thread overview]
Message-ID: <jwvtyrgsf21.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <m1mxx8lmnz.fsf@cam.ac.uk> (Leo's message of "Mon, 12 Apr 2010 16:23:44 +0100")
> Why empty abbrev tables are saved to file?
Don't know.
> It seems to make it more difficult for editing (edit-abbrevs) because
> the buffer is full of empty abbrev. I wonder if saving only non-empty
> tables is better and user friendlier.
It does sound like a good idea. Any objection?
> (defun abbrev-table-empty-p (table)
> "Return nil if there are no abbrev symbols in abbrev table object TABLE."
> (unless (abbrev-table-p table)
> (error "Non abbrev table object"))
> (not (catch 'some
> (mapatoms (lambda (sym)
> (when (abbrev-symbol (symbol-name sym) table)
> (throw 'some t)))
> table))))
> (defadvice write-abbrev-file (around nonempty-abbrev-tables activate)
> "Ignore empty abbrev tables when writing to FILE."
> (let ((file (ad-get-arg 0))
> (coding-system-for-write 'emacs-mule)
> (tables (loop for table in abbrev-table-name-list
> unless (abbrev-table-empty-p (symbol-value table))
> collect table)))
> (or (and file (> (length file) 0)) (setq file abbrev-file-name))
> (with-temp-file file
> (insert ";;-*-coding: emacs-mule;-*-\n")
> (dolist (table (sort tables
> (lambda (s1 s2)
> (string< (symbol-name s1)
> (symbol-name s2)))))
> (insert-abbrev-table-description table nil)))))
Obviously, defadvice wouldn't be the right approach for us.
And rather than (abbrev-symbol (symbol-name sym) table), we can use
(symbol-value sym).
We should also eliminate abbrev tables that only contain "system
abbrevs".
Stefan
next prev parent reply other threads:[~2010-04-12 18:32 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-12 15:23 bug#5937: 23.1.95; why saving empty abbrev tables Leo
2010-04-12 18:32 ` Stefan Monnier [this message]
2010-04-15 10:26 ` Leo
2010-04-15 12:44 ` Stefan Monnier
2010-04-16 10:36 ` Leo
2010-04-27 3:49 ` Stefan Monnier
2010-04-27 8:46 ` Leo
2010-04-27 10:12 ` Leo
2010-04-27 10:32 ` Leo
2011-03-27 20:40 ` Stefan Monnier
2011-03-28 4:45 ` Leo
2011-03-28 13:58 ` Stefan Monnier
2011-03-28 14:26 ` Leo
2011-03-28 15:09 ` Stefan Monnier
2011-03-29 0:35 ` Leo
2011-03-29 3:31 ` Stefan Monnier
2011-03-29 4:41 ` Leo
2011-03-29 5:16 ` Leo
2011-03-29 13:49 ` Stefan Monnier
2011-03-29 15:42 ` Leo
2011-03-29 20:54 ` Stefan Monnier
2011-03-30 1:08 ` Leo
2011-03-27 5:09 ` Leo
2011-03-27 17:34 ` Andreas Röhler
2011-03-28 3:38 ` Leo
2011-03-27 20:35 ` Stefan Monnier
2011-03-28 4:03 ` Leo
2011-03-28 14:02 ` Stefan Monnier
2011-03-28 14:40 ` Leo
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=jwvtyrgsf21.fsf-monnier+emacs@gnu.org \
--to=monnier@iro.umontreal.ca \
--cc=5937@debbugs.gnu.org \
--cc=sdl.web@gmail.com \
/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 external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.