KonstantinSeurer
/
mesa
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

aubinator: Delete redundant NULL check. 

handle_struct_decode() is just a wrapper around decode_structure()
with a NULL check.  But the only caller already does that NULL check.

So, just use decode_structure() directly.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
This commit is contained in:
Kenneth Graunke 2017-03-15 00:40:44 -07:00
parent 65138ce019
commit f0aa8fd4e4
1 changed files with 3 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/intel/tools/aubinator.c
View File

@ -161,15 +161,6 @@ decode_structure(struct gen_spec *spec, struct gen_group *strct,
}
}
static void
handle_struct_decode(struct gen_spec *spec, char *struct_name, uint32_t *p)
{
if (struct_name == NULL)
return;
struct gen_group *struct_val = gen_spec_find_struct(spec, struct_name);
decode_structure(spec, struct_val, p);
}
static void
dump_binding_table(struct gen_spec *spec, uint32_t offset)
{
@ -802,7 +793,9 @@ parse_commands(struct gen_spec *spec, uint32_t *cmds, int size, int engine)
if (token != NULL) {
printf("0x%08"PRIx64": 0x%08x : Dword %d\n",
offset + 4 * idx, p[idx], idx);
handle_struct_decode(spec,token, &p[idx]);
struct gen_group *struct_val =
gen_spec_find_struct(spec, token);
decode_structure(spec, struct_val, &p[idx]);
token = NULL;
}
}