cginternals / gloperate

C++ library for defining and controlling modern GPU rendering/processing operations.
MIT License
43 stars 26 forks source link

Revise ScreenAlignedTriangle and ScreenAlignedQuad (again) #402

Open j-o opened 7 years ago

j-o commented 7 years ago

Typically, a screen-aligned quad/triangle is used to do post-processing using shaders; however, the current implementations of ScreenAlignedTriangle and ScreenAlignedQuad do not allow this: they simply display an existing texture. This should be reflected in the class name to avoid confusion.

Further, since their functionality is reduced to displaying an existing texture, they are pretty much identical, so what is the point of having both?

I would suggest to replace the current ScreenAlignedTriangle and ScreenAlignedQuad classes with:

The latter might not be required if a screen-aligned triangle is considered state-of-the-art, making the screen-aligned quad obsolete.