diff options
| author | Eric Anholt <eric@anholt.net> | 2015-06-09 12:16:19 -0700 |
|---|---|---|
| committer | Eric Anholt <eric@anholt.net> | 2015-06-15 10:32:23 -0700 |
| commit | 932d1613d1e15ec22555e5ec09105c49eb850e36 (patch) | |
| tree | 8b08f4d126b40d8d1e3d8e3df06771be61d7996d | |
| parent | bcd8a64f32f6387cbd8ed8d0bda0f49bd7dd4251 (diff) | |
egl: Drop check for driver != NULL.
Back in 2013, a patch was added (with 2 reviewers!) at the end of the
block to early exit the loop in this case, without noticing that the loop
already did. I added another early exit case, again without noticing, but
Rob caught me. Just drop the loop condition that apparently surprises
most of us, instead of leaving the end of the loop conspicuously not
exiting on success.
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Rob Clark <robdclark@gmail.com>
| -rw-r--r-- | src/egl/drivers/dri2/egl_dri2.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c index 44a6c96ae9..dceb9a011d 100644 --- a/src/egl/drivers/dri2/egl_dri2.c +++ b/src/egl/drivers/dri2/egl_dri2.c @@ -397,7 +397,7 @@ dri2_open_driver(_EGLDisplay *disp) dri2_dpy->driver = NULL; end = search_paths + strlen(search_paths); - for (p = search_paths; p < end && dri2_dpy->driver == NULL; p = next + 1) { + for (p = search_paths; p < end; p = next + 1) { int len; next = strchr(p, ':'); if (next == NULL) |