all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Andreas Röhler" <andreas.roehler@online.de>
To: Tim Cross <theophilusx@gmail.com>
Cc: Daniel Colascione <dan.colascione@gmail.com>,
	Drew Adams <drew.adams@oracle.com>,
	emacs-devel@gnu.org
Subject: Re: more than one prefix argument
Date: Wed, 27 Jul 2011 12:21:32 +0200	[thread overview]
Message-ID: <4E2FE6AC.8070706@online.de> (raw)
In-Reply-To: <CAC=50j_yDhsX1ci_tM7+hTYBmFLr2py=nXw2Bx27aR-XTNNemQ@mail.gmail.com>

Am 27.07.2011 11:48, schrieb Tim Cross:
[ ... ]
>> Don't suggest additional code but a better use of "P" - no parallel
>> implementations. Having "P" as a branch-flag not affected by "p"
>>
>
> Sorry, just not quite following your argument.
>
> If we were to get rid of one, it would have to be the numeric version
> (i.e. p) rather than P as sometimes you need to be able to distinguish
> between nil and 1.

Could you give a case for that? IMHO we have to distinguish between nil 
and t at these occasions. Looks like wrongly reffered c-style 
limitation, which we gladly don't have business with.

> Essentially, we would need the raw form.


why?

We need a number or a boolean.
"p" should send a number, 1 as default.
"P" a boolean, nil as default.


This means
> that whenever you need to process the prefix argument as a number you
> will need to test and convert it to distinguish nil and extract the
> value from the list - this would be additional code ini your function.
>
> I don't understand what you mean by having a handy code for branching.
> Can you give an example of how this code would work and how a command
> using this code would be called?
>

How do you switch an alternative now?
Quite often that is done with an negative or positive numeric value.
(abbrev-mode -1)
(abbrev-mode 1)

Bad style IMHO, C-like obfuscating the code.

much better would be

(abbrev-mode nil)
(abbrev-mode t)

Cheers,

Andreas

> tim
>




  reply	other threads:[~2011-07-27 10:21 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-26 19:59 more than one prefix argument Andreas Röhler
2011-07-26 20:10 ` Daniel Colascione
2011-07-26 20:36   ` Andreas Röhler
2011-07-26 20:57     ` Drew Adams
2011-07-26 20:36   ` Drew Adams
2011-07-27  6:25     ` Andreas Röhler
2011-07-27  9:25       ` Tim Cross
2011-07-27  9:37         ` Andreas Röhler
2011-07-27  9:48           ` Tim Cross
2011-07-27 10:21             ` Andreas Röhler [this message]
2011-07-27 11:21               ` Andreas Schwab
2011-07-27 11:46                 ` Andreas Röhler
2011-07-27 12:39                   ` Andreas Schwab
2011-07-27 12:51                   ` Tim Cross
2011-07-27 12:46               ` Tim Cross
2011-07-27 15:09               ` Drew Adams
2011-07-27  9:38         ` Tim Cross

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=4E2FE6AC.8070706@online.de \
    --to=andreas.roehler@online.de \
    --cc=dan.colascione@gmail.com \
    --cc=drew.adams@oracle.com \
    --cc=emacs-devel@gnu.org \
    --cc=theophilusx@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.