TeamSpen210 / HammerAddons

Useful tweaks and content for Source Engine Games
123 stars 36 forks source link

feat: Ability to disable transforms #275

Closed ENDERZOMBI102 closed 3 months ago

ENDERZOMBI102 commented 3 months ago

Adds the ability to disable transforms via a comma-separated list option (transform_disable)

We're trying to be less intrusive on the edits to the postcompiler. As we support a few of the transforms natively, by using a config we can still be compatible while also avoiding the postcompiler trying to do what was already done.

TeamSpen210 commented 3 months ago

Seems useful to have in general. For the names used, configs do use the name specified in @trans(), the function name is internal and doesn't need to be unique. Instead we should probably ensure , is not permitted in a name, then use that.

For the Strata case, what would be better is to rename the pack_tags config option, then use that in the transforms to selectively disable. That way we don't have to tell users which of the many configs to disable, and other transforms can use the info to improve their implementation. Maybe I could make vactubes use RunScriptCode instead of packing scripts, for instance.