Templated version of sourcehook.h

[mixern6]

Templated version of sourcehook.h

[dvander]

If you'd like to submit PR with a working prototype I'd be happy to review it.

The macros cannot be completely obsoleted but I think there's a strong chance the easy cases could be more modern.

[mixern6]

It shows when I type "meta" and even after changing the level. If I revert my changes, metamod loads and works without errors.

UPD: I've just checked the wrapper "HookManagerPubFunc" If I change it back to function pointer, metamod works. Don't see what is wrong with the wrapper though

[mixern6]

The implementation is half-ready. I had to get rid of all the static stuff and now faced a problem that the
HookManagerPubFunc is a member function, but RemoveHookManager, AddHook and RemoveHook only accept pointers to static functions (HookManagerPubFunc). Have no idea how it can be bypassed as HookManagerPubFunc can't be turned back into a static function because I can't generate random names for different hooks like macros do.

[dvander]

Very cool, thanks for looking into this. Feel free to share patches and I can take a look.

Note that we have to stay ABI and API compatible but new additions should be no problem, the interfacs are versioned.

One solution to HookManagerPubFunc would be to add a new interface method to take in a virtual object, with a DeleteThis method if SH will own the pointer. Unfortunately due to linkage restrictions we can't malloc/free or share stl structures across library boundaries so std::function won't work.

[mixern6]

I have some progress on the task:
mixern6@dc8d870
One moment is confusing to me. I almost got rid of the MFHCls structure, moved out all the functions.
Now I have reinterpret_cast<void*>(this) across the code which point to the CVirtualHookHandler instance, should it be the same in the last argument of SetInfo (line 5267)?
Like:
reinterpret_cast<void**>(reinterpret_cast<char*>(this) + mfi.vtbloffs)[mfi.vtblindex]

The structure is initially empty, no data members. Is this a sort of hack or am I missing something?

[mixern6]

@dvander Could you please have a look at the current version. It compiles without errors, but the metamod doesn't start and shows this error: "You must change the map to activate Metamod:Source."
master...mixern6:metamod-source:templated_vhooks
The hook wrapper code is not used, I believe the problem is related to the function wrapper "HookManagerPubFunc"

[psychonic]

@dvander Could you please have a look at the current version. It compiles without errors, but the metamod doesn't start and shows this error: "You must change the map to activate Metamod:Source." master...mixern6:metamod-source:templated_vhooks The hook wrapper code is not used, I believe the problem is related to the function wrapper "HookManagerPubFunc"

Is that error when using meta before or after issuing map <mapname>? It would be normal for that message to be printed beforehand.