flamethrower - Bug? eXt. 2.7

[{NADF}Geronimo]

Hi Pat!

Please take a look at this one, and please confirm it is a bug or not. I have not checked the logic in details, but my first impression is, it is wrong.
File: _ex_flamethrower.gsc
Function: isFlamethrower(weapon)

Code: [Select]

isFlamethrower(weapon)
{
if(!isDefined(weapon)) return true;

switch(weapon)
{
case "flamethrower_axis":
case "flamethrower_allies": return true;
}

return false;
}
I think "if(!isDefined(weapon)) return true;" should return false.

Thanks for a quick check  :wink:

Regards,
{NADF} Geronimo

[{NADF}Geronimo]

I found the same in file: _ex_weaponback.gsc

Code: [Select]

isFlamethrower(weapon)
{
if(!isDefined(weapon)) return true;

switch(weapon)
{
case "flamethrower_axis":
case "flamethrower_allies": return true;
}

return false;
}
Same question now, please confirm it  

[PatmanSan]

Can't remember why I made it like that... could be fail-safe, but I'll change it anyway. We'll see what happens. Thanks