Minor npc_zombie improvements

[Wikot235]

-Added a keyvalue that changes zombie melee attack range
-Added a spawnflag that disables spawning a headcrab when a zombie dies

---

PR Checklist

• My PR follows all guidelines in the CONTRIBUTING.md file
• My PR targets a develop branch OR targets another branch with a specific goal in mind

[Blixibon]

I don't understand why the original #define definitions are being removed when the numbers from them are still being used to initialize the custom values. It seems unnecessary to delete them and it may add more risk for regressions or conflicts if other mods make changes to or around them, and it also removes the reference for where these numbers came from.

I would advise leaving them in and just doing this when initializing the custom values in the constructor:

#ifdef MAPBASE
m_iMeleeReach = ZOMBIE_MELEE_REACH;
m_iMaxPlayerDistToSwat = ZOMBIE_PLAYER_MAX_SWAT_DIST;
m_iMaxObjWeightToSwat = ZOMBIE_MAX_PHYSOBJ_MASS;
#endif

I would use GitHub's suggestions feature to make it easier to address this, although I don't know how that works with re-adding lines and I don't trust myself to make those suggestions accurately 🥴

This is less important, but I also think MaxObjWeightToSwat should be renamed to MaxObjMassToSwat since 'mass' is more accurate and weight isn't a commonly used metric in Source.

[Wikot235]

-Added a keyvalue that changes zombie melee attack range
-Added a spawnflag that disables spawning a headcrab when a zombie dies

---

PR Checklist

• My PR follows all guidelines in the CONTRIBUTING.md file
• My PR targets a develop branch OR targets another branch with a specific goal in mind

[Blixibon]

I don't understand why the original #define definitions are being removed when the numbers from them are still being used to initialize the custom values. It seems unnecessary to delete them and it may add more risk for regressions or conflicts if other mods make changes to or around them, and it also removes the reference for where these numbers came from.

I would advise leaving them in and just doing this when initializing the custom values in the constructor:

#ifdef MAPBASE
m_iMeleeReach = ZOMBIE_MELEE_REACH;
m_iMaxPlayerDistToSwat = ZOMBIE_PLAYER_MAX_SWAT_DIST;
m_iMaxObjWeightToSwat = ZOMBIE_MAX_PHYSOBJ_MASS;
#endif

I would use GitHub's suggestions feature to make it easier to address this, although I don't know how that works with re-adding lines and I don't trust myself to make those suggestions accurately 🥴

This is less important, but I also think MaxObjWeightToSwat should be renamed to MaxObjMassToSwat since 'mass' is more accurate and weight isn't a commonly used metric in Source.

[Wikot235]

-Added a keyvalue that changes zombie melee attack range
-Added a spawnflag that disables spawning a headcrab when a zombie dies

---

PR Checklist

• My PR follows all guidelines in the CONTRIBUTING.md file
• My PR targets a develop branch OR targets another branch with a specific goal in mind

[Blixibon]

I don't understand why the original #define definitions are being removed when the numbers from them are still being used to initialize the custom values. It seems unnecessary to delete them and it may add more risk for regressions or conflicts if other mods make changes to or around them, and it also removes the reference for where these numbers came from.

I would advise leaving them in and just doing this when initializing the custom values in the constructor:

#ifdef MAPBASE
m_iMeleeReach = ZOMBIE_MELEE_REACH;
m_iMaxPlayerDistToSwat = ZOMBIE_PLAYER_MAX_SWAT_DIST;
m_iMaxObjWeightToSwat = ZOMBIE_MAX_PHYSOBJ_MASS;
#endif

I would use GitHub's suggestions feature to make it easier to address this, although I don't know how that works with re-adding lines and I don't trust myself to make those suggestions accurately 🥴

This is less important, but I also think MaxObjWeightToSwat should be renamed to MaxObjMassToSwat since 'mass' is more accurate and weight isn't a commonly used metric in Source.

[Wikot235]

-Added a keyvalue that changes zombie melee attack range
-Added a spawnflag that disables spawning a headcrab when a zombie dies

---

PR Checklist

• My PR follows all guidelines in the CONTRIBUTING.md file
• My PR targets a develop branch OR targets another branch with a specific goal in mind

[Blixibon]

I don't understand why the original #define definitions are being removed when the numbers from them are still being used to initialize the custom values. It seems unnecessary to delete them and it may add more risk for regressions or conflicts if other mods make changes to or around them, and it also removes the reference for where these numbers came from.

I would advise leaving them in and just doing this when initializing the custom values in the constructor:

#ifdef MAPBASE
m_iMeleeReach = ZOMBIE_MELEE_REACH;
m_iMaxPlayerDistToSwat = ZOMBIE_PLAYER_MAX_SWAT_DIST;
m_iMaxObjWeightToSwat = ZOMBIE_MAX_PHYSOBJ_MASS;
#endif

I would use GitHub's suggestions feature to make it easier to address this, although I don't know how that works with re-adding lines and I don't trust myself to make those suggestions accurately 🥴

This is less important, but I also think MaxObjWeightToSwat should be renamed to MaxObjMassToSwat since 'mass' is more accurate and weight isn't a commonly used metric in Source.

[Wikot235]

-Added a keyvalue that changes zombie melee attack range
-Added a spawnflag that disables spawning a headcrab when a zombie dies

---

PR Checklist

• My PR follows all guidelines in the CONTRIBUTING.md file
• My PR targets a develop branch OR targets another branch with a specific goal in mind

[Blixibon]

I don't understand why the original #define definitions are being removed when the numbers from them are still being used to initialize the custom values. It seems unnecessary to delete them and it may add more risk for regressions or conflicts if other mods make changes to or around them, and it also removes the reference for where these numbers came from.

I would advise leaving them in and just doing this when initializing the custom values in the constructor:

#ifdef MAPBASE
m_iMeleeReach = ZOMBIE_MELEE_REACH;
m_iMaxPlayerDistToSwat = ZOMBIE_PLAYER_MAX_SWAT_DIST;
m_iMaxObjWeightToSwat = ZOMBIE_MAX_PHYSOBJ_MASS;
#endif

I would use GitHub's suggestions feature to make it easier to address this, although I don't know how that works with re-adding lines and I don't trust myself to make those suggestions accurately 🥴

This is less important, but I also think MaxObjWeightToSwat should be renamed to MaxObjMassToSwat since 'mass' is more accurate and weight isn't a commonly used metric in Source.

[Wikot235]

-Added a keyvalue that changes zombie melee attack range
-Added a spawnflag that disables spawning a headcrab when a zombie dies

---

PR Checklist

• My PR follows all guidelines in the CONTRIBUTING.md file
• My PR targets a develop branch OR targets another branch with a specific goal in mind

[Blixibon]

I don't understand why the original #define definitions are being removed when the numbers from them are still being used to initialize the custom values. It seems unnecessary to delete them and it may add more risk for regressions or conflicts if other mods make changes to or around them, and it also removes the reference for where these numbers came from.

I would advise leaving them in and just doing this when initializing the custom values in the constructor:

#ifdef MAPBASE
m_iMeleeReach = ZOMBIE_MELEE_REACH;
m_iMaxPlayerDistToSwat = ZOMBIE_PLAYER_MAX_SWAT_DIST;
m_iMaxObjWeightToSwat = ZOMBIE_MAX_PHYSOBJ_MASS;
#endif

I would use GitHub's suggestions feature to make it easier to address this, although I don't know how that works with re-adding lines and I don't trust myself to make those suggestions accurately 🥴

This is less important, but I also think MaxObjWeightToSwat should be renamed to MaxObjMassToSwat since 'mass' is more accurate and weight isn't a commonly used metric in Source.

[Wikot235]

-Added a keyvalue that changes zombie melee attack range
-Added a spawnflag that disables spawning a headcrab when a zombie dies

---

PR Checklist

• My PR follows all guidelines in the CONTRIBUTING.md file
• My PR targets a develop branch OR targets another branch with a specific goal in mind

[Blixibon]

I don't understand why the original #define definitions are being removed when the numbers from them are still being used to initialize the custom values. It seems unnecessary to delete them and it may add more risk for regressions or conflicts if other mods make changes to or around them, and it also removes the reference for where these numbers came from.

I would advise leaving them in and just doing this when initializing the custom values in the constructor:

#ifdef MAPBASE
m_iMeleeReach = ZOMBIE_MELEE_REACH;
m_iMaxPlayerDistToSwat = ZOMBIE_PLAYER_MAX_SWAT_DIST;
m_iMaxObjWeightToSwat = ZOMBIE_MAX_PHYSOBJ_MASS;
#endif

I would use GitHub's suggestions feature to make it easier to address this, although I don't know how that works with re-adding lines and I don't trust myself to make those suggestions accurately 🥴

This is less important, but I also think MaxObjWeightToSwat should be renamed to MaxObjMassToSwat since 'mass' is more accurate and weight isn't a commonly used metric in Source.

[Wikot235]

-Added a keyvalue that changes zombie melee attack range
-Added a spawnflag that disables spawning a headcrab when a zombie dies

---

PR Checklist

• My PR follows all guidelines in the CONTRIBUTING.md file
• My PR targets a develop branch OR targets another branch with a specific goal in mind

[Blixibon]

I don't understand why the original #define definitions are being removed when the numbers from them are still being used to initialize the custom values. It seems unnecessary to delete them and it may add more risk for regressions or conflicts if other mods make changes to or around them, and it also removes the reference for where these numbers came from.

I would advise leaving them in and just doing this when initializing the custom values in the constructor:

#ifdef MAPBASE
m_iMeleeReach = ZOMBIE_MELEE_REACH;
m_iMaxPlayerDistToSwat = ZOMBIE_PLAYER_MAX_SWAT_DIST;
m_iMaxObjWeightToSwat = ZOMBIE_MAX_PHYSOBJ_MASS;
#endif

I would use GitHub's suggestions feature to make it easier to address this, although I don't know how that works with re-adding lines and I don't trust myself to make those suggestions accurately 🥴

This is less important, but I also think MaxObjWeightToSwat should be renamed to MaxObjMassToSwat since 'mass' is more accurate and weight isn't a commonly used metric in Source.