Hi Emil, On 4 May 2017 at 15:14, Emil Velikov <[email protected]> wrote: > On 4 May 2017 at 14:43, Daniel Stone <[email protected]> wrote: >> IOW, there is no change as far as I can see, but perhaps for the >> meantime, we could use an unreachable() at the bottom of >> modifier_to_tiling(). Would that help? >> > Yes the default statement and return I915_TILING_NONE in > modifier_to_tiling() seem unreachable. > > unreachable or assert - I am leaning slightly towards the latter since > it will catch fallouts in the future. > I'll leave the decision to you and the Intel devs. > > In either case: > Reviewed-by: Emil Velikov <[email protected]>
Cool, I've pushed these both now with an assert - thanks! Cheers, Daniel _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
