It´s a minor thing, and first time I share an idea on Github, but about Hellfire Satyr Lord...

[Drerhu]

Could be fixed its enemy type? Probaby was an oversight from the devs, but they are categorized as "animals" instead as Diablo 1 vanilla Goatmen, which are all "demon" type. Thank you so much for how the source port works!

[StephenCWills]

Comparing Diablo and Hellfire in the decompilation, it seem that Satyr Lord takes the place of Lord Sayter in vanilla Diablo's monster data table. This strongly suggests that Hellfire developers explicitly changed the monster type from MC_DEMON to MC_ANIMAL. I have no idea why they would do this, but I don't see any evidence that it was a mistake.

https://github.com/diasurgical/devilution/blob/9f01757243412bb154d02d9af65c804159e6e0ea/Source/monstdat.cpp#L49-L53

#ifdef HELLFIRE
	{    160,    800, "Monsters\\GoatLord\\GoatL%c.CL2",  FALSE,       "Monsters\\newsfx\\Satyr%c%i.WAV",   FALSE,       FALSE,     NULL,                              { 13, 13, 14,  9, 16,  0 }, { 2, 0, 0, 0, 0, 0 }, "Satyr Lord",              40,       43,     28,    160,    200, AI_SKELSD,                                     MFLAG_SEARCH                      ,    3,   90,      8,         20,         30,     0,       0,           0,           0,          70, MC_ANIMAL,                  RESIST_FIRE | RESIST_LIGHTNING                , RESIST_MAGIC | IMMUNE_FIRE  | IMMUNE_LIGHTNING                 ,         0,        3,  2800 },
#else
	{    160,   2000, "Monsters\\GoatLord\\GoatL%c.CL2",  FALSE,       "Monsters\\GoatLord\\Goatl%c%i.WAV", FALSE,       FALSE,     NULL,                              { 13, 13, 14,  9, 16,  0 }, { 2, 0, 0, 0, 0, 0 }, "Lord Sayter",             13,       13,     12,    351,    351, AI_SKELSD,                                     MFLAG_SEARCH                      ,    3,   80,      8,         14,         24,     0,       0,           0,           0,          60, MC_DEMON,    RESIST_MAGIC | RESIST_FIRE |                    IMMUNE_NULL_40, RESIST_MAGIC | RESIST_FIRE |                    IMMUNE_NULL_40,         0,        3,  1500 },
#endif

Maybe the Diablo devs are the ones who changed it from MC_ANIMAL to MC_DEMON between 1.04 and 1.09, but I don't know why they would when Lord Sayter doesn't even appear in any version of the game.

[StephenCWills]

So maybe is a balance thing since mc_animal has different resistance to weapons?

Seems to me like it would have been more balanced if they left them as demon, since they already have Devil Kin Brute, Tomb Rat, and Firebat in the same range of dungeon levels. Meanwhile, Crypt 1 has no demons. Instead, if I were really reaching for an argument to justify their decision, perhaps they intentionally avoided introducing demons on Crypt 1 to give it a form of miniature monster type progression. After all, there are no animals on Crypt 4.

Just for curiosity (and thanks again for your time) what´s your opinion about these changes? (just for curiosity)

Thematically, demon seems like the more logical choice. It's more in line with Christian depictions of satyrs, and it's also more in line with the goatman-type monsters. However, I also think that changing the data for Lord Sayter is a clear expression of intent. That's not to say they couldn't have made a mistake when updating that data, but there is enough uncertainty that I am inclined to simply leave it alone.

[yuripourre]

My 2 cents,

From the Greek mythology Satyrs (Simia Satyrus) are "a kind of ape".

Which probably make them closer to an animal than a demon.

A page from "The Book of Beasts" of T. H. White:

[StephenCWills]

I´m sorry since I´m a bit lost about the version changes in vanilla Diablo 1 and Hellfire

I thought a table might help clarify.

	1.04	1.09	Hellfire
Lord Sayter/Satyr Lord	MC_DEMON	MC_DEMON	MC_ANIMAL
Hell Spawn	MC_ANIMAL	MC_DEMON	MC_ANIMAL

As you can see, when it comes to Lord Sayter/Satyr Lord, 1.04 matches 1.09 so Diablo devs didn't change it, but 1.04 doesn't match Hellfire so Hellfire devs did change it. As for Hell Spawn, 1.04 does not match 1.09 so Diablo devs changed it, but 1.04 does match Hellfire so Hellfire devs didn't change it.

[Drerhu]

I´m sorry since I´m a bit lost about the version changes in vanilla Diablo 1 and Hellfire

I thought a table might help clarify.
1.04 1.09 Hellfire
Lord Sayter/Satyr Lord MC_DEMON MC_DEMON MC_ANIMAL
Hell Spawn MC_ANIMAL MC_DEMON MC_ANIMAL

As you can see, when it comes to Lord Sayter/Satyr Lord, 1.04 matches 1.09 so Diablo devs didn't change it, but 1.04 doesn't match Hellfire so Hellfire devs did change it. As for Hell Spawn, 1.04 does not match 1.09 so Diablo devs changed it, but 1.04 does match Hellfire so Hellfire devs didn't change it.

Thank you so much. This reminded me something that happened with the Battle of Middle Earth 2 strategy games. The expansion (Rise of the Witch King) was developed with an earlier build/source code from vanilla game, and the patched expansion is less updated and bugfixed than the last vanilla patch. I usually see expansion packs are the most updated version, but not in that case. Since Blizzard didn´t want at all Synergistic doing the expansion, I totally believe they made a less updated version and Blizzard kept updating Diablo 1 vanilla, not caring about Hellfire at all (in fact, Hellfire was out of print because this until GoG release)

[qndel]

that's interesting, I wasn't aware of any other games where DLC was behind the base game