Skip to content

Commit

Permalink
mesa: Remove redundant gl_config::sampleBuffers
Browse files Browse the repository at this point in the history
This is just !!samples.

Reviewed-by: Eric Anholt <eric@anholt.net>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9817>
  • Loading branch information
nwnk authored and Marge Bot committed Mar 30, 2021
1 parent 4fbe1cb commit 4daef7f
Show file tree
Hide file tree
Showing 6 changed files with 4 additions and 7 deletions.
2 changes: 1 addition & 1 deletion src/gallium/frontends/dri/dri_screen.c
Expand Up @@ -425,7 +425,7 @@ dri_fill_st_visual(struct st_visual *stvis,
return;
}

if (mode->sampleBuffers) {
if (mode->samples > 0) {
stvis->samples = mode->samples;
}

Expand Down
5 changes: 3 additions & 2 deletions src/mesa/drivers/dri/common/utils.c
Expand Up @@ -359,7 +359,6 @@ driCreateConfigs(mesa_format format,
}

modes->samples = msaa_samples[h];
modes->sampleBuffers = modes->samples ? 1 : 0;

modes->sRGBCapable = is_srgb;
modes->mutableRenderBuffer = mutable_render_buffer;
Expand Down Expand Up @@ -437,7 +436,9 @@ driGetConfigAttribIndex(const __DRIconfig *config,
__ATTRIB(__DRI_ATTRIB_ACCUM_GREEN_SIZE, accumGreenBits);
__ATTRIB(__DRI_ATTRIB_ACCUM_BLUE_SIZE, accumBlueBits);
__ATTRIB(__DRI_ATTRIB_ACCUM_ALPHA_SIZE, accumAlphaBits);
__ATTRIB(__DRI_ATTRIB_SAMPLE_BUFFERS, sampleBuffers);
case __DRI_ATTRIB_SAMPLE_BUFFERS:
*value = !!config->modes.samples;
break;
__ATTRIB(__DRI_ATTRIB_SAMPLES, samples);
case __DRI_ATTRIB_RENDER_TYPE:
/* no support for color index mode */
Expand Down
1 change: 0 additions & 1 deletion src/mesa/main/context.c
Expand Up @@ -305,7 +305,6 @@ _mesa_initialize_visual( struct gl_config *vis,
vis->accumBlueBits = accumBlueBits;
vis->accumAlphaBits = accumAlphaBits;

vis->sampleBuffers = numSamples > 0 ? 1 : 0;
vis->samples = numSamples;

return GL_TRUE;
Expand Down
1 change: 0 additions & 1 deletion src/mesa/main/framebuffer.c
Expand Up @@ -450,7 +450,6 @@ _mesa_update_framebuffer_visual(struct gl_context *ctx,
* attachments).
*/
fb->Visual.samples = rb->NumSamples;
fb->Visual.sampleBuffers = rb->NumSamples > 0 ? 1 : 0;

if (_mesa_is_legal_color_format(ctx, baseFormat)) {
fb->Visual.redBits = _mesa_get_format_bits(fmt, GL_RED_BITS);
Expand Down
1 change: 0 additions & 1 deletion src/mesa/main/mtypes.h
Expand Up @@ -171,7 +171,6 @@ struct gl_config
GLint stencilBits;

/* ARB_multisample / SGIS_multisample */
GLint sampleBuffers;
GLuint samples;

/* OML_swap_method */
Expand Down
1 change: 0 additions & 1 deletion src/mesa/state_tracker/st_manager.c
Expand Up @@ -424,7 +424,6 @@ st_visual_to_context_mode(const struct st_visual *visual,
}

if (visual->samples > 1) {
mode->sampleBuffers = 1;
mode->samples = visual->samples;
}
}
Expand Down

0 comments on commit 4daef7f

Please sign in to comment.