[PATCH 2/2] Tweak the handling of $GDBHISTSIZE edge cases [PR gdb/16999]
Patrick Palka
patrick@parcs.ath.cx
Wed Jun 17 19:18:00 GMT 2015
On Fri, May 29, 2015 at 5:32 AM, Pedro Alves <palves@redhat.com> wrote:
> On 05/22/2015 12:23 PM, Patrick Palka wrote:
>
>> + while (isspace (*tmpenv))
>> + tmpenv++;
>
> ...
>
>> +
>> + while (isspace (*endptr))
>> + endptr++;
>
> Use skip_spaces/skip_spaces_const for these.
>
> Otherwise looks good to me.
>
> (I suspect that testing the interaction between setting
> the history both from a .gdbinit and $GDBHISTSIZE would
> call for merging the gdbhistsize-history.exp / gdbinit-history.exp
> to a single file.)
Committed.
The test files would not necessarily have to be merged to test the
interaction between $GDBHISTSIZE and .gdbinit. One may worry about
code duplication by not merging them, but I think there would be code
duplication either way unless you really refactor the tests. I think
it would be easier to test the interaction inside gdbinit-history.exp
since adding environment variable manipulation to gdbinit-history.exp
is easier than adding .gdbinit file manipulation in
gdbhistsize-history.exp. In fact, gdbinit-history.exp already has to
manipulate the environment to unset GDBHISTSIZE. So adding an
interaction test would only require a few lines of code. I'll do it.
>
> Thanks,
> Pedro Alves
>
More information about the Gdb-patches
mailing list