On 08/18/2017 09:37 AM, Dr. David Alan Gilbert wrote: > cc'ing in Gerd > > * René Rebe ([email protected]) 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 <[email protected]> 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
