On 14 February 2016 at 21:33, Stefan Dösinger <stefandoesinger(a)gmx.at> wrote:
> + if (gl_info->supported[EXT_FRAMEBUFFER_MULTISAMPLE])
> + gl_info->fbo_ops.glGetRenderbufferParameteriv(GL_RENDERBUFFER, GL_RENDERBUFFER_SAMPLES, &samples);
> + else
> + samples = 1;
I don't think this will do the right thing in core contexts, or if
only ARB_framebuffer_object is supported.
> + gl_info->fbo_ops.glGetRenderbufferParameteriv(GL_RENDERBUFFER, GL_RENDERBUFFER_INTERNAL_FORMAT, &fmt);
> + FIXME("\t%s: renderbuffer %d, %dx%d, %d samples, format %x.\n",
> + debug_fboattachment(attachment), name, width, height, samples, fmt);
The '\t' is from the original code of course, but please just replace
it with spaces.
> + FIXME("\t%s: %s texture %d, %dx%d, format %x.\n", debug_fboattachment(attachment),
> + tex_type_str, name, width, height, fmt);
%#x for hexadecimal, please.
> +#define FBOATTACHMENT_TO_STR(u) case u: return #u
The existing functions are a bit of a mess in this regard, but I'd
prefer "WINED3D_TO_STR" for new functions.
> + FIXME("Unrecognied FBO status 0x%08x\n", attachment);
> + return "unrecognized";
The FIXME has room for improvement. You could also consider doing
something like "return wine_dbg_sprintf("unknown attachment %#x",
attachment);"