klip

Regarding naming: Only enumerations that actually represent flags should be named `Flags`. The naming usually implies that you can do bitwise operations on them. Enumerations that represent "types" or similar...

The performance should be the least of your concern when deciding `stock` vs `native`, especially when it's something as insignificant as this which you won't (and can't) execute too much...

`menu_additem` shouldn't get deprecated. It's not broken, it's just that the new API will make for simpler and cleaner code. In `menu_additem2` third param shouldn't strictly be `DataPack:`, but `any:`...

> Other: > Add a reset score plugin with some command aliases > AFK manager with some cvars (like afk time before do anything, transfer to spec or kick) >...

This can probably just be in `amxmisc.inc`, we don't need a new file. I haven't checked thoroughly, but because this is in "core" it shouldn't work only for CS teams,...

There isn't such a list, that's the thing. That's why I think the ability to target teams shouldn't be in the core.

I like the second suggestion the most and provided that no errors are produced when matching team names, it seems like the best solution which can stay in core.

All plugins have to be recompiled to make this applicable. If `cmd_target` was a native, and it sadly isn't, it would've been fine. I'd say this is a problem, because...

Repeating numbers aren't a sign of a bad algorithm, that's not how random number generators are tested.

At first I thought there may be an engine limitation, but it looks like the fake command API was just badly designed. It should be rather easy to change ```sourcepawn...