Make all post-processing conditional

[Denneisk]

Optimizes post-processing by not having their hooks run when they're never used. Also removes some unnecessary checks or moves them to run only once.

Now post-processing hooks run only when needed.

I tested it and it all works fine, but I would appreciate anyone looking over this to make sure I didn't do any copying errors.

Was waiting for someone else to do this before I realized it'd be faster to do it myself.

[Cheatoid]

I like the idea, but:

• Change this logic: newValue != "0" to check particular convar's boolean value instead (ConVar:GetBool()), this is because any non-zero value is considered true in engine, even -1 and 123.45. Consistency.
• Have you considered what would happen if any of those Lua files were (hot)reloaded (or loaded twice for whatever reason), no? Well, it would add another instance of those change callbacks into the list, and there be double shenanigans.
• In regards to bloom/sunbeams/toytown, you can completely wrap those around the render.SupportsPixelShaders_2_0() check, because GPU isn't going to change while Garry's Mod is running, ay? (Basically no change callback even.)

[Denneisk]

Thanks for the response

Change this logic: newValue != "0"

I thought all nonzero values were true for convars. I thought it would be simpler this way.

(hot)reloaded

Thank you for considering this, that's a very important oversight.

render.SupportsPixelShaders_2_0()

Good idea. Thanks.