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

Reply via email to