This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] printcmd.c (ui_printf): make internalvar string can be printf and eval when inferior cannot alloc memory


On Fri, Aug 19, 2011 at 22:14, Tom Tromey <tromey@redhat.com> wrote:
>>>>>> ">" == Hui Zhu <teawater@gmail.com> writes:
>
> Tom> This test seems odd to me. ?Why does the lvalue-ness of the value
> Tom> matter?
>
>>> I don't understand your means, could you do some explain?
>
> I just don't understand why a check of the VALUE_LVAL is necessary.
> Does something go wrong in the other cases? ?It doesn't seem to me that
> it should.
>
> Tom
>

Got it.

I make a new patch remove the line:
>> +            if ((VALUE_LVAL (val_args[i]) == lval_internalvar
>> +             || VALUE_LVAL (val_args[i]) == not_lval)

Please help me review it.

Thanks,
Hui

2011-09-06  Hui Zhu  <teawater@gmail.com>

	* printcmd.c (ui_printf): Add a handler for TYPE_CODE_ARRAY.

Attachment: fix-v-string.txt
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]