[Mesa-dev] [Bug 84566] Unify the format conversion code

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=84566

--- Comment #3 from Iago Toral <itoral at igalia.com> ---
(In reply to Jason Ekstrand from comment #0)
(...)
>  5) Make a masater conversion function that takes a void*, format, width,
> height, and stride for both source and destination and just does the
> conversion.  If the above mentioned array format enum is distinct from the
> mesa_format enum, the function could be written to take a uint32_t type and
> accept either mesa_format or an array format in the same parameter.
>  6) Use the above master conversion function for TexSubimage, TexImage,
> GetTexImage, DrawPixels, and ReadPixels.  We still have to deal with pixel
> conversion, but it should vastly simplify all of them.

As far as I can see, functions in textstore do not receive a  mesa_format for
their src argument, instead they receive the GL format and data type, so in
order to use the master function here we need to map this combination to a
mesa_format or mesa_array_format first. It does not look like this conversion
is available anywhere though, so I think we need to create a function that maps
(GLformat, GLDataType)-> mesa_format. Something similar to what 
_mesa_format_to_type_and_comps does, but the other way around. Jason, does this
make sense to you?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20141006/f6cc63fa/attachment-0001.html>