[PATCH] PR/12691 Add the inferior to Python exited event
Kevin Pouget
kevin.pouget@gmail.com
Thu Sep 15 15:27:00 GMT 2011
On Thu, Sep 15, 2011 at 3:57 PM, <Paul_Koning@dell.com> wrote:
>> An integer representing the exit code, if available, which the
>>> inferior has returned. (The exit code could be unavailable if, for
>>> example, @value{GDBN} detaches from the inferior.)
>>>
>>
>> right, I've updated the patch, is it fine for you ?
>
> The old text is more explicit about what you see if the exit code is not available. It would be good to keep that -- i.e., the attribute isn't there if the exit code is not available. The reason for being explicit is that a different way of handling that situation -- and arguably a more "Pythonic" one -- is for the attribute to exist in all cases but have the value None if not applicable. So you might make it:
>
> +An integer representing the exit code, if available, which the inferior
> +has returned. (The exit code could be unavailable if, for example,
> +@value{GDBN} detaches from the inferior.) If the exit code is
> +unavailable, the attribute does not exist.
I'm okay with this modification; I'll let Eli accept it before
applying this patch
Cordially,
Kevin
More information about the Gdb-patches
mailing list