Re: [PATCH] load_cursor_argb is supposed to return a Bool, not void

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 14, 2014 at 11:22:03 -0700, Keith Packard wrote:

> By mis-declaring this function, we ended up using software cursors
> because the value seen by the caller was 0.
> 
> Signed-off-by: Keith Packard <keithp@xxxxxxxxxx>
> ---
>  src/sna/sna_display.c      | 8 ++++++--
>  src/sna/sna_display_fake.c | 3 ++-
>  src/uxa/intel_display.c    | 7 +++++--
>  3 files changed, 13 insertions(+), 5 deletions(-)
> 
Only since
http://cgit.freedesktop.org/xorg/xserver/commit/?id=901fbfbbbd71c0d82080957f8ba09eebbc786f2b

Which could probably have used a different name to avoid silent
breakage.

Cheers,
Julien
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux