On Mon, Jan 28, 2013 at 2:09 PM, Ian Romanick <idr at freedesktop.org> wrote: > These changes seem innocuous enough. > > Reviewed-by: Ian Romanick <ian.d.romanick at intel.com> They are... but 2/3 and 3/3 are adding unreachable return statements after switches with a default case that asserts. Kind of bogus.