On 29 August 2018 at 12:47, Eric Engestrom <eric.engest...@intel.com> wrote:
>> +{ >> + _EGLDevice *dev; >> + >> + mtx_lock(_eglGlobal.Mutex); >> + dev = _eglGlobal.DeviceList; >> + /* The first device is always software */ >> + if (!dev) { >> + _eglGlobal.DeviceList = &software_device; >> + dev = _eglGlobal.DeviceList; >> + } > > You could avoid that (duplicated) runtime logic by just putting that > line in the initialisation of `_eglGlobal` > Ack. Can init upon declaration [in eglglobal.c]. Will fix that and use your name suggestions with v2. -Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev