BTW, the subject prefix thing goes for all your patches. Have a look at other patches that touched these files for inspiration.
On Thu, Dec 8, 2016 at 8:27 PM, Ilia Mirkin <[email protected]> wrote: > Usually the subject prefix here would just be "mesa". Sometimes people > might put "mesa/main". I don't think the "a: b:" thing is frequently > done in mesa. With that fixed up, > > Reviewed-by: Ilia Mirkin <[email protected]> > > On Thu, Dec 8, 2016 at 6:00 AM, Lionel Landwerlin > <[email protected]> wrote: >> Signed-off-by: Lionel Landwerlin <[email protected]> >> Cc: Ilia Mirkin <[email protected]> >> --- >> src/mesa/main/enable.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/src/mesa/main/enable.c b/src/mesa/main/enable.c >> index 8440c62..2267e31 100644 >> --- a/src/mesa/main/enable.c >> +++ b/src/mesa/main/enable.c >> @@ -440,7 +440,7 @@ _mesa_set_enable(struct gl_context *ctx, GLenum cap, >> GLboolean state) >> ctx->Color.IndexLogicOpEnabled = state; >> break; >> case GL_CONSERVATIVE_RASTERIZATION_INTEL: >> - if (!_mesa_is_desktop_gl(ctx) && ctx->API != API_OPENGLES) >> + if (!_mesa_has_INTEL_conservative_rasterization(ctx)) >> goto invalid_enum_error; >> if (ctx->IntelConservativeRasterization == state) >> return; >> -- >> 2.10.2 >> _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
