Skip to content

New pistol/smg/rifle/shotgun weapon classifications#1995

Open
sunzenshen wants to merge 1 commit into
NeotokyoRebuild:masterfrom
sunzenshen:wep-classifications-pistol-smg-shotgun-rifle
Open

New pistol/smg/rifle/shotgun weapon classifications#1995
sunzenshen wants to merge 1 commit into
NeotokyoRebuild:masterfrom
sunzenshen:wep-classifications-pistol-smg-shotgun-rifle

Conversation

@sunzenshen

Copy link
Copy Markdown
Contributor

Description

New weapon flags for pistols/smgs/shotguns/rifles, to simplify bot weapon flag checks.

Toolchain

  • Windows MSVC VS2022

@sunzenshen sunzenshen requested a review from a team June 18, 2026 07:18
@sunzenshen

Copy link
Copy Markdown
Contributor Author

In reference to: #1805 (comment)

@sunzenshen sunzenshen added the Bots Related to bot players label Jun 18, 2026
@AdamTadeusz AdamTadeusz requested a review from a team June 19, 2026 08:41
// SUPPRESSED - Suppressed weapons
// EXPLOSIVE - Generic for weapons that count as explosive kills on killfeed.
// SHOTGUN - Weapons with shotgun spread
// BATTLERIFLE - Semi-auto rifles

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should add ASSAULTRIFLE too while we're at it?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought about it, but I wanted to keep changes limited to gameplay significant changes.

Otherwise then it brings up philosophical questions of what bits should we set the BALC, PBK, and PZ to. Is it easier to classify them as Assault Rifles? Is the PBK an SMG? etc.

Alternatively maybe it makes more sense to brand things along the lines of SCOPEDWEAPON? e.g. instead of BATTLERIFLE, specify something instead like AUTOWEAPON vs. SEMIWEAPON, but I'm not sure about the wording of the latter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bots Related to bot players

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants