main: Major refactor of get_texture_for_framebuffer.
authorLaura Ekstrand <laura@jlekstrand.net>
Tue, 3 Mar 2015 00:48:59 +0000 (16:48 -0800)
committerFredrik Höglund <fredrik@kde.org>
Thu, 14 May 2015 13:48:13 +0000 (15:48 +0200)
commit085c67dc77ab3c00d3f68ba52f8343f8ff0cab53
tree39f3e55c960530d1788d7e684e008c37aac7c537
parentd78c831a147e8af6f6fc1a610f4c2e490e75fad1
main: Major refactor of get_texture_for_framebuffer.

This splits off the (still) rather large chunk that is
get_texture_for_framebuffer into lots of smaller functions specialized to
service the wide variety of unique needs of *FramebufferTexture* entry points.
The result is much cleaner because, rather than having a pile of branches and
confusing conditions (like the boolean layered), the uniqueness is baked into
the entry points. The entry points know whether or not they are layered or use
a textarget.

[Fredrik: - Mention the value of <textarget> in the error message.
          - Rename check_zoffset to check_layer, and zoffset to layer.
            The zoffset parameter was renamed to layer in
            ARB_framebuffer_object.
          - Make layered a GLboolean since the value is visible to the API.
          - Remove EXT suffixes in refactored code.
          - Whitespace fixes.]

Reviewed-by: Fredrik Höglund <fredrik@kde.org>
Signed-off-by: Fredrik Höglund <fredrik@kde.org>
src/mapi/glapi/gen/ARB_framebuffer_object.xml
src/mesa/main/fbobject.c
src/mesa/main/fbobject.h