On Sun, Jun 22, 2014 at 12:18:02AM +0100, Peter Maydell wrote:
> > +    if (!lock_user_struct(VERIFY_READ, target_tz, target_tz_addr, 1))
> > +        return -TARGET_EFAULT;
> 
> Coding style mandates braces even on single-line if()s; checkpatch.pl
> will catch this usually.

I copied that style from the equivalent timeval functions, but yeah I'll
make sure to get into the habit of running checkpatch for QEMU patches.

> Code looks OK otherwise.

Thanks for looking,
    Paul

Attachment: signature.asc
Description: Digital signature

Reply via email to