[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: silly face function names

From: Kim F. Storm
Subject: Re: silly face function names
Date: 20 Dec 2002 12:28:56 +0100
User-agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.3.50

Miles Bader <address@hidden> writes:

> address@hidden (Kim F. Storm) writes:
> > The names are badly chosen (having both 'set-' and '-p' is confusing),
> > but if you read the names like this, it does make sense:
> > 
> >         set-face - FOO-p
> > 
> > I would argue that 'set-' takes precedence over '-p' here...
> I'd argue that you're wrong -- `FOO-p' is not a thing which you can set.
> Faces have an `inverse-video' property, and `face-inverse-video-p' is
> merely the name of the predicate which tests that property.
> Having bogus function names around in emacs-proper just serves to
> confuse people and cause the _next_ generation of bogus function names.

Ok, but since we already have N ways to do this:
        (make-face-FOO face [F])
        (make-face-unFOO face [F])
        (set-face-attribute face F :FOO bool)
        (set-face-FOO-p face bool [F])

So I really don't see _any_ reason to add an N+1'th way to do the same:
        (set-face-FOO face bool [F])

Instead, I'd suggest simply to mark the set-face-FOO-p obsolete, and
change the few existing uses of those functions to use one of the other
alternatives above.

Kim F. Storm <address@hidden> http://www.cua.dk

reply via email to

[Prev in Thread] Current Thread [Next in Thread]