On Mon, 2018-03-12 at 17:11 +0000, Daniel Stone wrote:
> On 12 March 2018 at 16:57, Mario Kleiner <mario.kleiner...@gmail.com> wrote:
> > The check for "no modifier specified" in drmmode_is_format_supported()
> > should check for DRM_FORMAT_MOD_INVALID, not for zero, as zero actually
> > means DRM_FORMAT_MOD_LINEAR.
> 
> If I had a time machine, that would probably be the first thing I
> would change about the modifiers API.
> 
> Reviewed-by: Daniel Stone <dani...@collabora.com>

Programming isn't about math, it's about numerology. Merged, thanks:

remote: I: patch #209954 updated using rev 
757e0ee1884f2a34a287d632462e3bc689eccb3d.
remote: I: 1 patch(es) updated to state Accepted.
To ssh://git.freedesktop.org/git/xorg/xserver
   136b45e0c0..757e0ee188  master -> master

- ajax
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to