intel: Silence several "warning: unused parameter"
authorIan Romanick <ian.d.romanick@intel.com>
Fri, 26 Aug 2011 17:22:49 +0000 (10:22 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Fri, 9 Sep 2011 19:01:51 +0000 (12:01 -0700)
commit4d5c820b4f1b5c8f182be98ae2f3cdc993025388
treefa66a14ad8a4e6f84abe77b64d4839e3aa89299b
parent6f23d9b637602d9997896c6ca5cba0dfe36a092e
intel: Silence several "warning: unused parameter"

The internalFormat, format, and type parameters were not used by
either try_pbo_upload or try_pbo_zcopy, so remove them.  The width
parameter was also not used by try_pbo_zcopy (because it doesn't
actually copy anything), so remove it too.

Eric Anholt notes:

    The current structure of this code is so hateful I can't bring
    myself to say anything about whether changing the current code is
    good or bad.

    I have a dream that one call would try to make a surface
    (miptree/region) out of the PBO, then we'd see about whether it
    matches up nicely and zero-copy/blit using that.  That would be
    reusable for texsubimage, which is currently awful in this
    respect.

At some point we should revisit this code with pitchforks and torches.
src/mesa/drivers/dri/intel/intel_tex_image.c