diff options
author | Paulo Zanoni <paulo.r.zanoni@intel.com> | 2015-03-24 17:35:10 -0300 |
---|---|---|
committer | Paulo Zanoni <paulo.r.zanoni@intel.com> | 2015-05-05 17:29:14 -0300 |
commit | c0ed7d344af4d60d2c9663041233c3e0fbc2dc38 (patch) | |
tree | a32d8c4eb66deb373451279ee93581de5f3aff86 /lib/intel_batchbuffer.h | |
parent | 235e87de450714df9982fb86fae2dd5a9e8e2980 (diff) |
tests/kms_fbc_crc: remove redundant information from data_t
While it is nice to have shorter names for the most-accessed
variables, it makes the code more difficult to read since it's not
clear to the code reader whether that "gem_handle" is from some FB or
something else. The reader also has to audit the code to see if, for
example, the value of data->handle[0] stays consistent with
data->fb[0].gem_handle all the tame or if at some point the value is
replaced with something else. So remove the redundant information,
making it explicit that we're using the gem handles and FB IDs of the
framebuffers all the time.
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Diffstat (limited to 'lib/intel_batchbuffer.h')
0 files changed, 0 insertions, 0 deletions