Global Namespaces

[garrynewman]

A feature of net6 that we pushed yesterday is global namespaces.

We're taking advantage of this. By default every addon will include this statically in their global namespace.

(this is akin to adding global using static Sandbox.Internal.Game.GlobalNamespace; in one of your addon's .cs files)

This means that DebugOverlay and Global aren't static classes anymore - which means they can be extended with your own functions. But usage stays exactly the same.

We'll probably move more things like this over in time. I'm considering whether to move IsServer, IsClient etc to this class too - so that checking then is just if ( IsServer ) instead of if ( Host.IsServer ) etc.. let me know what you think of that possibility.

[MuffinTastic]

Very neat stuff.

I may very well be alone in this but there always has to be that one guy ー I've personally never really understood the appeal of having it just be IsServer. I actually rather like the slight verbosity of Host.IsServer, to me it makes it feel ever so slightly less like "dark magic", and it draws your attention to it just a bit more. I think whether or not something is server- or clientside code, or shared code, is worthy of one's attention.

[garrynewman]

I agree with this

[xezno]

Could we get the same treatment for Log? I've had a few cases where I've wanted to extend that and haven't been able to (e.g. making an extension function that prefixes the output with client/server scope)

[garrynewman]

Yeah!