[Mesa-dev] RFC Remove _mesa_problem() from paths where execution path is not altered

Timothy Arceri tarceri at itsqueeze.com
Mon May 15 01:26:01 UTC 2017


Ping!

On 09/05/17 13:04, Timothy Arceri wrote:
> There is still more that can be removed I just though I'd get comments before
> going too far.
> 
> This is a follow on from [1][2].
> 
> Besides patch 1 we are just replacing it with an unreachable() when the
> path doesn't try to work around the problem i.e return early.
> 
> _mesa_problem() is still useful in some places such as is if a backend
> compile fails, but for the majority of cases we should be able to
> remove it. I did a search and we haven't really added _mesa_problem()
> much in recent years, most of these have existed for a really long
> time.
> 
> OpenGL test suites are becoming very mature, we should place more
> trust in debug builds picking up missed cases.
> 
> 
> [1] https://patchwork.freedesktop.org/patch/154888/
> [2] https://patchwork.freedesktop.org/patch/155094/
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 


More information about the mesa-dev mailing list