On Sun, Apr 10, 2011 at 09:37:12PM +1000, Brad Hards wrote: > On Sun, 10 Apr 2011 08:26:06 pm Alon Levy wrote: > > - if (d->mode == QXL_MODE_NATIVE || d->mode == QXL_MODE_COMPAT) > > + if (d->mode == QXL_MODE_NATIVE || d->mode == QXL_MODE_COMPAT > > + || (io_port == QXL_IO_LOG)) > I think this might look better with consistent use of brackets. > > Low priority, for your consideration. > Thanks, I'll fix before sending final version (unless this is the final version - but that never happens).
> Brad