diff options
author | Timothy Arceri <[email protected]> | 2017-04-21 13:29:46 +1000 |
---|---|---|
committer | Timothy Arceri <[email protected]> | 2017-04-22 10:01:15 +1000 |
commit | 622a68ed3e36a6b56db35df62c5913d2d54d5ed6 (patch) | |
tree | 974998286efcf4ed0528766666b54013e8e07c62 /src/mesa/main/pipelineobj.c | |
parent | 0cc8c81902dffdd0c2e1f74e7828a6132cb70b9f (diff) |
mesa: remove fallback RefCount == 0 pattern
We should never get here if this is 0 unless there is a
bug. Replace the check with an assert.
Reviewed-by: Nicolai Hähnle <[email protected]>
Reviewed-by: Samuel Pitoiset <[email protected]>
Diffstat (limited to 'src/mesa/main/pipelineobj.c')
-rw-r--r-- | src/mesa/main/pipelineobj.c | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/src/mesa/main/pipelineobj.c b/src/mesa/main/pipelineobj.c index c1dd8d75c76..2988c97455e 100644 --- a/src/mesa/main/pipelineobj.c +++ b/src/mesa/main/pipelineobj.c @@ -206,16 +206,10 @@ _mesa_reference_pipeline_object_(struct gl_context *ctx, if (obj) { /* reference new pipeline object */ mtx_lock(&obj->Mutex); - if (obj->RefCount == 0) { - /* this pipeline's being deleted (look just above) */ - /* Not sure this can ever really happen. Warn if it does. */ - _mesa_problem(NULL, "referencing deleted pipeline object"); - *ptr = NULL; - } - else { - obj->RefCount++; - *ptr = obj; - } + assert(obj->RefCount > 0); + + obj->RefCount++; + *ptr = obj; mtx_unlock(&obj->Mutex); } } |