Add support for sending gameevent to specific client.

[psychonic]

I also initially planned on adding support for blocking existing events, per client on-the-fly, but API was getting complex and potentially inefficient. (This can still be added at some point). The net effect of that can still be achieved by blocking existing events with a prehook, saving event data, and then resending to the desired clients (not ideal, but doable).

Only tested on TF2 in Windows

[WildCard65]

So is this blank line needed?

[psychonic]

...

[psychonic]

Thinking about it more, it shouldn't be necessary to add any further hooking. Even copying the event data in the "workaround" can be avoided, since you can use Event.FireToClient in an event prehook.

Pattern would be something like:

public Action Ev_SomeEventPre(Event event, const char[] name, bool dontBroadcast)
{
    for (int i = 1; i <= MaxClients; ++i)
    {
        if (IsClientBlah(client))
            event.FireToClient(i);
    }

    return Plugin_Handled;
}

[Kenzzer]

So event.BroadcastDisabled = true; is no longer required?
If we do event.FireToClient(i); before, or return Plugin_Handled does it? I'm thinking about death message in tf2.

[psychonic]

I don't fully understand the question, but this is a better example of the above, if you don't want to actually block the event serverside.

public Action Ev_SomeEventPre(Event event, const char[] name, bool dontBroadcast)
{
    for (int i = 1; i <= MaxClients; ++i)
    {
        if (IsClientBlah(client))
            event.FireToClient(i);
    }

    event.BroadcastDisabled = true;

    return Plugin_Continue;
}

When you only block broadcast, it stops the event from going to clients, but the server will still do any processing that it may have for the event (such as the log line for player_death). If you block the event itself, that gets blocked, too. I hope that clarifies things.

[KyleSanderson]

There are super rare cases where a client can be connected but still have an invalid net channel (KAC/SMAC used to cause crashes in convar queries). Despite this, everything looks fine.

[psychonic]

Pics or it didn't happen.

[KyleSanderson]

Ironically it was part of the code you refactored 😄

https://hg.alliedmods.net/releases/sourcemod-1.3/rev/82cfdef347cc