Skip to content

Fix template deduction for windows compilation - #449

Merged
Vauff merged 2 commits into
Source2ZE:khookfrom
Kenzzer:khook
Jun 8, 2026
Merged

Vauff merged 2 commits into
Source2ZE:khookfrom
Kenzzer:khook

Conversation

@Kenzzer

@Kenzzer Kenzzer commented Jun 4, 2026

Copy link
Copy Markdown

No description provided.

@Vauff
Vauff merged commit ebe7aeb into Source2ZE:khook Jun 8, 2026
3 checks passed
Vauff added a commit that referenced this pull request Sep 9, 2026
* Switch to Metamod KHook development branch

* fix includes, some definitely wrong messing around with api

* first khook theoretically working

* Finish migration of main virtual hooks

Still needs testing

* fix building docker container

* fix build errors

* fix more build errors

* update khook include path

* Fix VTable hooks

* Migrate EntWatch virtual hooks

* various fixes & remove/add some todo's

* more fixes

* "fix" another calloriginal

* Fix EntWatch crashes

* Re-enable GoToIntermission hook

Seems to have stopped crashing?

* Fix PostEventAbstract crash

* Migrate detour system to KHook

* fix TakeDamage crash

* Fix template deduction for windows compilation (#449)

* Fix template deduction

* alternative workaround: explicitly define template arguments

---------

Co-authored-by: Kenzzer <kenzzer@users.noreply.github.com>
Co-authored-by: Vauff <mctehkitti@gmail.com>

* fix bad copy paste

* Replace our signature lookup with KHook's implementation

This allows for signature lookups to always work even on signatures modified by another plugins KHook detour

* switch back to metamod master branch

* Remove wildcard converter now that KHook supports singles

* Address automatic review findings

---------

Co-authored-by: Benoist <14257866+Kenzzer@users.noreply.github.com>
Co-authored-by: Kenzzer <kenzzer@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants