[PATCH 8/8] wined3d: Skip non-textureable formats in check_fbo_compat().
Matteo Bruni
matteo.mystral at gmail.com
Wed Apr 10 10:53:48 CDT 2013
2013/4/10 Henri Verbeet <hverbeet at gmail.com>:
> On 10 April 2013 01:35, Matteo Bruni <mbruni at codeweavers.com> wrote:
>> ---
>> dlls/wined3d/utils.c | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/dlls/wined3d/utils.c b/dlls/wined3d/utils.c
>> index da8fc7b..5010676 100644
>> --- a/dlls/wined3d/utils.c
>> +++ b/dlls/wined3d/utils.c
>> @@ -1038,6 +1038,9 @@ static void check_fbo_compat(const struct wined3d_gl_info *gl_info, struct wined
>> GLenum status;
>> GLuint tex;
>>
>> + if (!(format->flags & WINED3DFMT_FLAG_TEXTURE))
>> + return;
>> +
>
> I'm not sure about this one, what exactly should this achieve? In
> either case, this probably belongs in init_format_fbo_compat_info(),
> together with the other similar checks.
>
The idea is that, since check_fbo_compat() creates a texture of the
specified format to attach it to a test FBO and verify that it works
correctly, it doesn't make sense to even try if the format is
non-texturable.
The check can surely go to init_format_fbo_compat_info() instead.
More information about the wine-devel
mailing list