Why? I think it's probably fine but changing signedness of a variable is always a bit tricky.

On January 19, 2018 08:04:24 Emil Velikov <emil.l.veli...@gmail.com> wrote:

From: Emil Velikov <emil.veli...@collabora.com>

Already used as such through the codebase.

Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
---
 src/mesa/drivers/dri/i965/brw_context.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/brw_context.h b/src/mesa/drivers/dri/i965/brw_context.h
index 79e9f49a389..574a0d9d860 100644
--- a/src/mesa/drivers/dri/i965/brw_context.h
+++ b/src/mesa/drivers/dri/i965/brw_context.h
@@ -977,7 +977,7 @@ struct brw_context
     * Number of samples in ctx->DrawBuffer, updated by BRW_NEW_NUM_SAMPLES so
     * that we don't have to reemit that state every time we change FBOs.
     */
-   int num_samples;
+   unsigned int num_samples;

    /* BRW_NEW_URB_ALLOCATIONS:
     */
--
2.15.0

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to