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
signature.asc
Description: Digital signature