Skip to content

feat: add a way to add CVAR descriptions in cgame - #83

Merged
ec- merged 1 commit into
ec-:masterfrom
WofWca:add-cvar-description
Jul 23, 2026
Merged

ec- merged 1 commit into
ec-:masterfrom
WofWca:add-cvar-description

Conversation

@WofWca

@WofWca WofWca commented Jul 22, 2026 •

Copy link
Copy Markdown
Contributor

It's a nice feature for new mods, which might introduce new vars.

This syscall extension is supported by at least the Quake3e engine:
ec-/Quake3e@ada9f42
On vanilla and unsupporting engines this is a no-op.

Yes, this doesn't add this to game/ for now.
That can be done in a follow up, and is trivial
(basically just copy and paste).

Also see the relevant feature request in ioquake3:
ioquake/ioq3#884.

I have tested this with a VM build (not DLL though), by adding a description to a var.

@WofWca
WofWca force-pushed the add-cvar-description branch from 2abbe1a to 0d8992b Compare July 22, 2026 15:40
@WofWca WofWca changed the title feat: add a way to add CVAR descriptions to cgame feat: add a way to add CVAR descriptions in cgame Jul 22, 2026
@ensiform

Copy link
Copy Markdown

Description type in the struct should definitely be const char * IMHO

@WofWca

WofWca commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

Ah, I didn't pay much attention to that part. Will change it now.

It's a nice feature for new mods, which might introduce new vars.

This syscall extension is supported by at least the Quake3e engine:
ec-/Quake3e@ada9f42
On vanilla and unsupporting engines this is a no-op.

Yes, this doesn't add this to `game/` for now.
That can be done in a follow up, and is trivial
(basically just copy and paste).

Also see the relevant feature request in ioquake3:
ioquake/ioq3#884.
@WofWca
WofWca force-pushed the add-cvar-description branch from 0d8992b to a6a90c7 Compare July 23, 2026 07:48
@ec-
ec- merged commit a6fd05f into ec-:master Jul 23, 2026
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.

3 participants