From mboxrd@z Thu Jan  1 00:00:00 1970
From: Kevin Buettner <kevinb@cygnus.com>
To: Fernando Nasser <fnasser@redhat.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFA] More completion improvements
Date: Sun, 18 Feb 2001 10:39:00 -0000
Message-id: <1010218183948.ZM21394@ocotillo.lan>
References: <Pine.SUN.3.91.1010218092227.4804N-100000@is> <1010218164030.ZM21263@ocotillo.lan> <3A900C1D.BB086BEF@redhat.com> <fnasser@redhat.com>
X-SW-Source: 2001-02/msg00340.html

On Feb 18, 12:53pm, Fernando Nasser wrote:

> This style for the add_show_from_set() call is everywhere. As we need to
> revise the creation of CLI commands anyway, this will not last long
> enough for us to consider a major cleanup.  It will just go away
> naturally...

Fernando,

I took another look, and I don't see the style that I was objecting to
used *anywhere* with add_show_from_set().  Perhaps, I was not clear
enough in my original message.  What I find (mildly) objectionable is
changing code which looks like this:

    add_show_from_set (add_set_cmd (...), &showlist);

to:

    add_show_from_set ((c = add_set_cmd (...)), &showlist);

Note the addition of an embedded assignment expression.  Instead, if
it becomes necessary to add an assignment, it should be added as a
separate statment, thusly:

    c = add_set_cmd (...);
    add_show_from_set (c, &showlist);

Kevin