On Fri, Aug 14, 2015 at 4:09 PM, Chad Versace <chad.vers...@intel.com> wrote:
> On Tue 11 Aug 2015, Nanley Chery wrote: > > From: Nanley Chery <nanley.g.ch...@intel.com> > > > > Combine the adjacent cases which have the same GL type in the switch > statemnt. > > > > Signed-off-by: Nanley Chery <nanley.g.ch...@intel.com> > > --- > > src/mesa/main/formats.c | 152 > ++++++------------------------------------------ > > 1 file changed, 17 insertions(+), 135 deletions(-) > > I checked every line in this patch and found no errors. > > Patch 4/4 is > Reviewed-by: Chad Versace <chad.vers...@intel.com> > > As an aside... Do you think the format's datatype be stored directly in > the Mesa's format info table? It seems strange that Mesa maintains that > bit of information outside the info table. > Thanks. I do think this is something that should be added to the format info table. The only downsides I can see are that the csv would increase in width and that the format info table would take up more memory, but I don't think that'd be a real problem.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
