[python] [patch] PR python/13345

Phil Muldoon pmuldoon@redhat.com
Tue Nov 1 14:15:00 GMT 2011


Meador Inge <meadori@codesourcery.com> writes:

> On 10/31/2011 10:51 AM, Phil Muldoon wrote:
>
>> This patch fixes a case where the tilde (~) command was being passed to
>> Python via the "source" command.  Python does not understand what to do
>> with a tilde, so we have to expand it first.
>
> While I can't give an OK, I did review this and was able to reproduce
> the stated problem and the patch fixes it.  So, LGTM.  What about
> a test case, though?  You could construct a relative path to a
> test directory from '~/'.

The only addition in the patch was tilde_expand, and an additional error
check.  tilde_expand is a readline function.  So we would be testing
that, more or less.  I do normally write regression tests, but I felt
for this one it was not necessary as the patch is somewhat trivial.
Plus I am not sure how constructing a path with a ~ in it would work on
mingw builds?  If so, and we really do want one, I can attempt to write
one.

Cheers

Phil



More information about the Gdb-patches mailing list