Re: [Qemu-devel] [PATCH] QXL: always show default cursor, even for unsupported chunks

[Eric Blake]

On 08/18/2017 09:37 AM, Dr. David Alan Gilbert wrote:
> cc'ing in Gerd
> 
> * René Rebe (address@hidden) wrote:
>> Hi,
>>
>> I was testing a Windows 10 guest with HiDPI scaling which
>>
>> actually made the cursor disappear and only multiple chunks
>>
>> warnings being printed.
>>
>> When the cursor is using currently unsupported multiple chunks, the
>> builtin-left cursor is better than none.

This part is okay, but might be better written in an imperative sense
(the best commit messages are ones where you add an implicit "Apply this
patch to ..." prior to the first word after the blank line separating
the subject).

>>
>> PS: if TB damages the white-space just let me know, only recently started
>> using it, ...

This part belongs after a --- separator, as it is useful to reviewers
but adds nothing to git.

By the way, while your patch did not appear to be whitespace damaged on
a cursory glance, the fact that you used thunderbird to send it is
risky; thunderbird HAS been known to mess up patches that are pasted
inline, and sending patches as attachments is harder to read than direct
patches.  Using 'git send-email' excels at making sure the patch will be
in a format easy to apply.

>>
>>   - René Rebe <address@hidden>

And this is not a proper Signed-off-by: tag; we'll need one of those
before your patch can be considered.  More hints at:
http://wiki.qemu.org/Contribute/SubmitAPatch

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

signature.asc
Description: OpenPGP digital signature