On Friday, 2019-01-11 14:20:38 +0000, Emil Velikov wrote:
> On Wed, 2 Jan 2019 at 13:48, Eric Engestrom <eric.engest...@intel.com> wrote:
> >
> > On Thursday, 2018-12-27 20:41:47 +0000, Alexander von Gluck IV wrote:
> > > ---
> > >  src/egl/drivers/haiku/egl_haiku.cpp | 3 ++-
> > >  1 file changed, 2 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/src/egl/drivers/haiku/egl_haiku.cpp 
> > > b/src/egl/drivers/haiku/egl_haiku.cpp
> > > index a9c5cf8d29..d4b046c79b 100644
> > > --- a/src/egl/drivers/haiku/egl_haiku.cpp
> > > +++ b/src/egl/drivers/haiku/egl_haiku.cpp
> > > @@ -29,6 +29,7 @@
> > >
> > >  #include "eglconfig.h"
> > >  #include "eglcontext.h"
> > > +#include "egldevice.h"
> > >  #include "egldisplay.h"
> > >  #include "egldriver.h"
> > >  #include "eglcurrent.h"
> > > @@ -215,7 +216,7 @@ init_haiku(_EGLDriver *drv, _EGLDisplay *dpy)
> > >               _eglError(EGL_NOT_INITIALIZED, "DRI2: failed to find 
> > > EGLDevice");
> > >               return EGL_FALSE;
> > >       }
> > > -     disp->Device = dev;
> > > +     dpy->Device = dev;
> > >
> > >       TRACE("Add configs\n");
> > >       if (!haiku_add_configs_for_visuals(dpy))
> > > --
> > > 2.14.5
> >
> > Thanks!
> >
> > Pushed with these tags added:
> > Reviewed-by: Eric Engestrom <eric.engest...@intel.com>
> > Fixes: 00992700c9a812a54563 "egl: set the EGLDevice when creating a display"
> 
> Thanks guys. I think we should follow-up with s/dpy/disp/ to align the
> haiku code with the rest.
> Otherwise we might end up doing such mistake again :-\

Ack on the idea, but a quick grep shows that both dri2 and haiku have
a mix of both `dpy` and `disp`, so make sure you fix them all ;)
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to