all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
To: Michal Nazarewicz <mina86@mina86.com>
Cc: Juanma Barranquero <lekktu@gmail.com>, emacs-devel@gnu.org
Subject: Re: [PATCHv3 0/3] Add `notes' function, take three
Date: Fri, 21 Jun 2013 11:21:58 -0400	[thread overview]
Message-ID: <jwv8v23cx4d.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <cover.1371816337.git.mina86@mina86.com> (Michal Nazarewicz's message of "Fri, 21 Jun 2013 14:16:40 +0200")

> Change to kill-buffer allowed an advice to be replaced with a hook,

OK

> and after looking more closely at remember.el, I think notes might be
> a good fit for it.

Indeed.  Why even define a separate notes-file?
And why not use (find-file-noselect notes-file), maybe followed by
a rename-buffer?

> Stefan, are you still not convinced, and prefer to have it in ELPA
> after all?

No, I think remember.el is a good place for it.  It would also be good
to use the "remember-" prefix.

> The second patch is actually unrelated, but something I've noticed
> while playing with notes.

Good point.  I think we should turn get-file-buffer into an alias for
find-buffer-visiting.


        Stefan



  parent reply	other threads:[~2013-06-21 15:21 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-21 12:16 [PATCHv3 0/3] Add `notes' function, take three Michal Nazarewicz
2013-06-21 12:16 ` [PATCH 1/3] `kill-buffer' runs query functions before checking buffer modification Michal Nazarewicz
2013-06-21 12:16 ` [PATCH 2/3] `remember-append-to-file' appending to buffer bug fix Michal Nazarewicz
2013-06-21 12:16 ` [PATCH 3/3] Add `notes' function to store random notes across Emacs restarts Michal Nazarewicz
2013-06-21 15:21 ` Stefan Monnier [this message]
2013-06-21 17:13   ` [PATCHv3 0/3] Add `notes' function, take three Michal Nazarewicz
2013-06-21 19:14     ` Stefan Monnier

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=jwv8v23cx4d.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=emacs-devel@gnu.org \
    --cc=lekktu@gmail.com \
    --cc=mina86@mina86.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.