emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed

From: Carsten Dominik <dominik@science.uva.nl>
To: "Tom Breton (Tehom)" <tehom@panix.com>
Cc: emacs-orgmode@gnu.org
Subject: Re: Org-choose
Date: Fri, 13 Feb 2009 22:34:20 +0100	[thread overview]
Message-ID: <CAB66967-2785-4461-88A2-93882A3FD2D3@uva.nl> (raw)
In-Reply-To: <1036.24.63.20.74.1234553649.squirrel@mail.panix.com>

Hi Tom,

thanks, this patch seems to fix at least the problem I was having.

I see that you now require 'cl also during runtime, which is
probably (?) because you are also calling some functions,
not only macros?

If we decide to move org-choose into the Org core at some point,
then we would have to change this, because Emacs rules require
that cl is only required during compilation.

I think that is a pity because it is *so* useful, but
there you go.

So, maybe at some point you might want to replace the cl functions
with other code.


On Feb 13, 2009, at 8:34 PM, Tom Breton (Tehom) wrote:
>
> That just might be it!  I have changed it to org-split-string.  A
> patch is attached.
>
> I can't detect any difference - all tests pass either way - so I hope
> you will tell me whether it works now.

It does!  And I like it, I am already using it.  Also, because it opens
the possibility to think of more nice things to do with TODO keywords.
>
> Thank you.  And thank you for your patience with this.

Of course!  Thank you for your contributions!

- Carsten

next prev parent reply	other threads:[~2009-02-13 21:34 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <273688C0-9A36-47D5-B747-A1371C2BD05A@uva.nl>
     [not found] ` <1782.65.96.63.84.1234471662.squirrel@mail.panix.com>
     [not found]   ` <5E0C3EFF-06FA-4446-B869-F514B6748C5E@uva.nl>
2009-02-13 19:34     ` Org-choose Tom Breton (Tehom)
2009-02-13 21:34       ` Carsten Dominik [this message]
2009-02-13 22:29         ` Org-choose Tom Breton (Tehom)

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.orgmode.org/

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

  git send-email \
    --in-reply-to=CAB66967-2785-4461-88A2-93882A3FD2D3@uva.nl \
    --to=dominik@science.uva.nl \
    --cc=emacs-orgmode@gnu.org \
    --cc=tehom@panix.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

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.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).