Skip to content

Conversation

@itsneufox
Copy link
Contributor

@itsneufox itsneufox commented Oct 29, 2025

title

solves warning 213: tag mismatch: expected tag "t_WEAPON", but found none ("_")

@NexiusTailer
Copy link
Contributor

NexiusTailer commented Oct 30, 2025

I also noticed there's still a_npc.inc with such a stub:
#error open.mp doesn't yet support NPC scripts

Maybe somehow change this to be a wrapper over the new npc include file?

UPD: Oh, seems like we really don't support specifically NPC scripts like it is for vanilla SA-MP NPCs (laying in npcmodes folder and using a_npc), or do we? Does FCNPC support this as well?

@AmyrAhmady
Copy link
Member

no we don't support them, nor does FCNPC, because they run in another isolated environment in samp-npc (samp-npc has its own pawn vm and runs the script in there)

@NexiusTailer
Copy link
Contributor

no we don't support them, nor does FCNPC, because they run in another isolated environment in samp-npc (samp-npc has its own pawn vm and runs the script in there)

Oh that's really interesting. So maybe deprecate them instead of the current message like "it's not supported yet"

@itsneufox itsneufox changed the title Change NPC_Kill reason from 255 to REASON_SUICIDE in omp_npc,inc Change NPC_Kill reason from 255 to REASON_SUICIDE in omp_npc.inc Oct 30, 2025
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