Re: [PATCH] c_value_print: Revert 'val' to a reference for TYPE_CODE_STRUCT

[Pedro Alves <palves@redhat.com>]

On 04/27/2016 02:21 PM, Martin Galvan wrote:
> On Wed, Apr 27, 2016 at 7:01 AM, Pedro Alves <palves@redhat.com> wrote:
>> What's the motivation behind this?  Does it change anything user visible?
> 
> AFAIK not directly, but I'm going to need it for the synthetic reference bug
> fix.

I see. 

> Since this is an isolated change I thought I could send it for
> review now.

Since you didn't mention whether the change had any user-visible
impact, I was left wondering if we could add a testcase to
the testsuite that exposes the need for the change.  From the original
log it kind of sounded like it would be possible.

It's better to be explicit in such cases, and say something like,
"this has no effect currently, so can be seen as a small code
cleanup, but once we do X, we'll print the wrong thing", or some such,
and mention that this causes no testsuite regressions, in the 
email/commit log.

The code change is OK.

Thanks,
Pedro Alves