mesa: remove _mesa_init_get_hash()
The actual code of the function print_table_stats() is guarded by a ifdef GET_DEBUG, which was not been defined in years. The last fix in 2013 (7db6b5aa91
) indicates that it's rarely used/tested. Since the issue has gone unnoticed for a whole year (broken with2ad4a47547
). Let's remove it for now. We can always revive it at a later stage. Signed-off-by: Emil Velikov <emil.velikov@collabora.com> Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
This commit is contained in:
parent
b81685eb32
commit
23a7fca6aa
|
@ -407,8 +407,6 @@ one_time_init( struct gl_context *ctx )
|
||||||
|
|
||||||
/* per-API one-time init */
|
/* per-API one-time init */
|
||||||
if (!(api_init_mask & (1 << ctx->API))) {
|
if (!(api_init_mask & (1 << ctx->API))) {
|
||||||
_mesa_init_get_hash(ctx);
|
|
||||||
|
|
||||||
_mesa_init_remap_table();
|
_mesa_init_remap_table();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -148,9 +148,6 @@ _mesa_get_current_context(void);
|
||||||
extern void
|
extern void
|
||||||
_mesa_init_constants(struct gl_constants *consts, gl_api api);
|
_mesa_init_constants(struct gl_constants *consts, gl_api api);
|
||||||
|
|
||||||
extern void
|
|
||||||
_mesa_init_get_hash(struct gl_context *ctx);
|
|
||||||
|
|
||||||
extern void
|
extern void
|
||||||
_mesa_notifySwapBuffers(struct gl_context *gc);
|
_mesa_notifySwapBuffers(struct gl_context *gc);
|
||||||
|
|
||||||
|
|
|
@ -553,71 +553,6 @@ static const int extra_core_ARB_color_buffer_float_and_new_buffers[] = {
|
||||||
* collisions for any enum (typical numbers). And the code is very
|
* collisions for any enum (typical numbers). And the code is very
|
||||||
* simple, even though it feels a little magic. */
|
* simple, even though it feels a little magic. */
|
||||||
|
|
||||||
#ifdef GET_DEBUG
|
|
||||||
static void
|
|
||||||
print_table_stats(int api)
|
|
||||||
{
|
|
||||||
int i, j, collisions[11], count, hash, mask;
|
|
||||||
const struct value_desc *d;
|
|
||||||
const char *api_names[] = {
|
|
||||||
[API_OPENGL_COMPAT] = "GL",
|
|
||||||
[API_OPENGL_CORE] = "GL_CORE",
|
|
||||||
[API_OPENGLES] = "GLES",
|
|
||||||
[API_OPENGLES2] = "GLES2",
|
|
||||||
};
|
|
||||||
const char *api_name;
|
|
||||||
|
|
||||||
api_name = api < ARRAY_SIZE(api_names) ? api_names[api] : "N/A";
|
|
||||||
count = 0;
|
|
||||||
mask = ARRAY_SIZE(table(api)) - 1;
|
|
||||||
memset(collisions, 0, sizeof collisions);
|
|
||||||
|
|
||||||
for (i = 0; i < ARRAY_SIZE(table(api)); i++) {
|
|
||||||
if (!table(api)[i])
|
|
||||||
continue;
|
|
||||||
count++;
|
|
||||||
d = &values[table(api)[i]];
|
|
||||||
hash = (d->pname * prime_factor);
|
|
||||||
j = 0;
|
|
||||||
while (1) {
|
|
||||||
if (values[table(api)[hash & mask]].pname == d->pname)
|
|
||||||
break;
|
|
||||||
hash += prime_step;
|
|
||||||
j++;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (j < 10)
|
|
||||||
collisions[j]++;
|
|
||||||
else
|
|
||||||
collisions[10]++;
|
|
||||||
}
|
|
||||||
|
|
||||||
printf("number of enums for %s: %d (total %ld)\n",
|
|
||||||
api_name, count, ARRAY_SIZE(values));
|
|
||||||
for (i = 0; i < ARRAY_SIZE(collisions) - 1; i++)
|
|
||||||
if (collisions[i] > 0)
|
|
||||||
printf(" %d enums with %d %scollisions\n",
|
|
||||||
collisions[i], i, i == 10 ? "or more " : "");
|
|
||||||
}
|
|
||||||
#endif
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Initialize the enum hash for a given API
|
|
||||||
*
|
|
||||||
* This is called from one_time_init() to insert the enum values that
|
|
||||||
* are valid for the API in question into the enum hash table.
|
|
||||||
*
|
|
||||||
* \param the current context, for determining the API in question
|
|
||||||
*/
|
|
||||||
void _mesa_init_get_hash(struct gl_context *ctx)
|
|
||||||
{
|
|
||||||
#ifdef GET_DEBUG
|
|
||||||
print_table_stats(ctx->API);
|
|
||||||
#else
|
|
||||||
(void) ctx;
|
|
||||||
#endif
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Handle irregular enums
|
* Handle irregular enums
|
||||||
*
|
*
|
||||||
|
|
Loading…
Reference in New Issue