On Wed, 11 Apr 2012 10:46:51 +0200, Olivier Galibert <galib...@pobox.com> wrote:
> No hookup with GL yet.  Planned to be used to simplify profiles.

This is pretty cool.  The payoff in patch 2 is huge.  I'd just been
thinking about how to get that sort of win, with just having a common
file for each version along with the per-target file.  Implementing
#include seems like a way better way to go.

My worry: what happens if a GLSL consumer in the real compiler enables
the extension and #includes something?  It looks like you allow the
extension to be enabled regardless, and just call the NULL get_string
method (also, using 0 in preprocess() instead of NULL?  ew.).

Also, get_string is a super generic name.  Let's
s/get_string/include_get_string/ and be consistent in that name.

I'm hoping cworth and idr can take a look at the cpp and parser bits.

Attachment: pgpjAJ0RwjYiQ.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to