[Top][All Lists]

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

Re: [PATCH 1/2] scm/define-markup-commands.scm: remove some unnecessary

From: Nicolas Sceaux
Subject: Re: [PATCH 1/2] scm/define-markup-commands.scm: remove some unnecessary lookups
Date: Sun, 22 Nov 2009 10:47:20 +0100

Le 22 nov. 2009 à 00:00, David Kastrup a écrit :
>> I'd be insterested to see an implementation of a single
>> `define-markup-command' for builtin and user defined markups, where
>> user defined commands do not pollute the (lily) module, and still are
>> available across file includes.
> There is no real necessity: you can perfectly well define different
> define-markup-command versions for builtin and user defined markups and
> put them in different namespaces.

Oh, two different macros, doing different things, but with the same name,
right, now that design seems much better indeed. No possible confusion.
We're not going in the right direction.

If you can come up with a single implementation, then it's excellent.
If there are two different implementations, then there will be two
different names. No confusion.

If you want to test your unified implementation, consider the following
test case:




must give exactly the same result, even if redefines a builtin

> It is not a "little inconvenience" if internals are defined in a
> different, undocumented way that apparently only a single developer
> understands and realizes.

This comment would have been valid a few hours before, but it's not


Maybe the way to make a user defined command, starting from a bultin one,
should be detailed in the manuals. Granted.

Any other /valid/ objections?

You seem to to able to spend a lot of time in endless discussions. You're
lucky. That's not my case. Now I'll be discussing patches. Only.


reply via email to

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