On 07Feb2009 20:51, Dominik Vogt <[email protected]> wrote:
| [...]  I have attached a much smaller version of the
| patch that removes all code that replaces other styles and fixes a
| couple of small problems:
| 
|   * Don't look up the style in events.c again but pass the string
|     back from AddWindow().  As a side effect, the command is only
|     executed when the window is mapped initially, not when it's
|     mapped because of deiconification.  Therefore I renamed the
|     style to InitialMapCommand (which may or may not be a good
|     idea).

I'm +1 on the changed name - I like the distinction, unless someone likes with:

  MapCommand [Initial|Every|<condition>] command...

I.e. have an optional leading condition to MapCommand.

Just my 2c,
-- 
Cameron Simpson <[email protected]> DoD#743

Reply via email to