On Fri, Aug 26, 2016 at 6:48 AM, Rob Clark <robdcl...@gmail.com> wrote: > On Thu, Aug 25, 2016 at 8:00 PM, Brian Paul <bri...@vmware.com> wrote: >> If someone is looking for a simple task in gallium... >> >> In commit 0135bd44 Marek introduced some new enum types in p_defines.h >> >> In p_context.h we should replace several instances of "unsigned shader" with >> "enum pipe_shader_type shader" and propagate that change into the driver >> functions which correspond to those interface functions. > > yes please.. and probably not just 'enum pipe_shader_type (although > that would be a good place to start).. imho the extra type information > from use of enum types helps make the gallium interface easier to > follow > > (I wonder if it would be worthwhile to have a mesa trello page for > todos like these) > Yeah it would be a really good idea, the todos could be simple code cleanups for the beginners to get started.
Cheers, Nayan. > BR, > -R > >> -Brian >> _______________________________________________ >> mesa-dev mailing list >> mesa-dev@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/mesa-dev > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
