Skip to content

refactor: use SourceMod 1.12 methodmaps - #52

Merged
Rushaway merged 1 commit into
masterfrom
feat/methodmaps
Oct 21, 2026
Merged

Rushaway merged 1 commit into
masterfrom
feat/methodmaps

Conversation

@Rushaway

Copy link
Copy Markdown
Member

Summary

  • Replace legacy natives with their SourceMod 1.12 methodmap equivalents:
    • Database / DBResultSet / Transaction
    • Event (event.GetInt(), ...)
  • Retype the affected handles and callback parameters (Handle -> ConVar, Event, Menu, KeyValues, ...)
  • Bump version: PlayerManager.sp: 2.3.5 -> 2.3.6

Why

Methodmaps are the API SourceMod documents and maintains; the procedural natives only remain for backward compatibility. The typed handles also let the compiler catch handle misuse. No behaviour change is intended.

Test plan

  • Compiles with spcomp 1.12.0.7255 using the same dependencies as CI (no errors, no new warnings)
  • CI build passes
  • Smoke-test the plugin on a test server

🤖 Generated with Claude Code

- Replace legacy natives with their SourceMod 1.12 methodmap equivalents:
  - `Database` / `DBResultSet` / `Transaction`
  - `Event` (`event.GetInt()`, ...)
- Retype the affected handles and callback parameters (`Handle` -> `ConVar`, `Event`, `Menu`, `KeyValues`, ...)
- Bump version: `PlayerManager.sp`: 2.3.5 -> 2.3.6

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Rushaway
Rushaway merged commit 0dbd30a into master Oct 21, 2026
7 checks passed
@Rushaway
Rushaway deleted the feat/methodmaps branch October 21, 2026 12:47
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.

1 participant