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 1/2] Remove parameter valaddr from c print functions

From: Yao Qi <qiyaoltc at gmail dot com>
To: Doug Evans <dje at google dot com>
Cc: gdb-patches <gdb-patches at sourceware dot org>
Date: Fri, 11 Nov 2016 08:41:28 +0000
Subject: Re: [PATCH 1/2] Remove parameter valaddr from c print functions
Authentication-results: sourceware.org; auth=none
References: <1478789679-30957-1-git-send-email-yao.qi@linaro.org> <CADPb22QnOOamZA2sQCbUjzkREoEE5djFhsdUV_-qG06HdxPL9g@mail.gmail.com>

On Thu, Nov 10, 2016 at 5:53 PM, Doug Evans <dje@google.com> wrote:
> On Thu, Nov 10, 2016 at 6:54 AM, Yao Qi <qiyaoltc@gmail.com> wrote:
>> This patch removes parameter valaddr from some c print functions.
>>
>> Regression tested on x86_64-linux.
>>
>> gdb:
>>
>> 2016-11-10  Yao Qi  <yao.qi@linaro.org>
>>
>>         * c-lang.h (cp_print_value_fields): Update declaration.
>>         * cp-valprint.c (cp_print_value): Update declaration.
>>         (cp_print_value_fields): Remove parameter valaddr.  Callers
>>         updated.
>>         (cp_print_value): Likewise.
>
> LGTM

Thanks for the review, patch is pushed in.

-- 
Yao (齐尧)

References:
- [PATCH 1/2] Remove parameter valaddr from c print functions
  - From: Yao Qi
- Re: [PATCH 1/2] Remove parameter valaddr from c print functions
  - From: Doug Evans

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