[Mesa-dev] [PATCH v2 26/27] i965/blorp: brw_blorp_blit.cpp -> blorp_blit.c

Jason Ekstrand jason at jlekstrand.net
Wed Jul 27 02:19:29 UTC 2016


On Jul 26, 2016 6:49 PM, "Matt Turner" <mattst88 at gmail.com> wrote:
>
> On Tue, Jul 26, 2016 at 3:11 PM, Jason Ekstrand <jason at jlekstrand.net>
wrote:
> > ---
> >  src/mesa/drivers/dri/i965/Makefile.sources   |    2 +-
> >  src/mesa/drivers/dri/i965/blorp_blit.c       | 1662
++++++++++++++++++++++++++
> >  src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 1662
--------------------------
> >  3 files changed, 1663 insertions(+), 1663 deletions(-)
> >  create mode 100644 src/mesa/drivers/dri/i965/blorp_blit.c
> >  delete mode 100644 src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
>
> Same comment about the other rename patches: what's the point of
> dropping the "brw_"?

Eventually, these are going to move to src/intel/blorp.  When they do, the
blorp prefix will be sufficient.  At the moment, it makes it clear that
they're is little to nothing brw about them (hence the distinction between
brw_blorp.c and blorp.c in patch 25) and avoids the annoying dependency
issues of renaming a fine from .c to .cpp.

--Jason
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160726/d37e5ef0/attachment.html>


More information about the mesa-dev mailing list