diff options
author | Eric Anholt <[email protected]> | 2013-06-26 15:22:13 -0700 |
---|---|---|
committer | Eric Anholt <[email protected]> | 2013-08-17 11:43:19 +0200 |
commit | bef423bee62f7b74858f1b7f74be21405ca75eef (patch) | |
tree | 1652593dcdcb566e62637347ffe99ed55dc8e7a0 /src/mesa/drivers/dri/i915 | |
parent | 703a2f4219e6a95b7445138082e24cd762b1fa7e (diff) |
dri: Choose a decent global driNConfigOptions.
Previously, we were asserting that each driver specified an NConfigOptions
exactly equal to the number of options they supplied, leading to frequent
bugs when people would forget to adjust the value when adjusting driver
options. Instead, just overallocate the table by a bit and leave sanity
checking to the assert in findOption().
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri/i915')
-rw-r--r-- | src/mesa/drivers/dri/i915/intel_screen.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/src/mesa/drivers/dri/i915/intel_screen.c b/src/mesa/drivers/dri/i915/intel_screen.c index 30a867e029e..f8b95f44a9d 100644 --- a/src/mesa/drivers/dri/i915/intel_screen.c +++ b/src/mesa/drivers/dri/i915/intel_screen.c @@ -77,8 +77,6 @@ PUBLIC const char __driConfigOptions[] = DRI_CONF_SECTION_END DRI_CONF_END; -const GLuint __driNConfigOptions = 12; - #include "intel_batchbuffer.h" #include "intel_buffers.h" #include "intel_bufmgr.h" @@ -1124,8 +1122,7 @@ __DRIconfig **intelInitScreen2(__DRIscreen *psp) return false; } /* parse information in __driConfigOptions */ - driParseOptionInfo(&intelScreen->optionCache, - __driConfigOptions, __driNConfigOptions); + driParseOptionInfo(&intelScreen->optionCache, __driConfigOptions); intelScreen->driScrnPriv = psp; psp->driverPrivate = (void *) intelScreen; |