Scripting language bug

[Kaimetsu]

NPC Code:

---

if(playerenters){

 showcharacter;

 if(1==1) message I work;

 else message GScript is bugged;

}

---

It seems that anything before a == is always assumed to be a variable or a function. Hand it a plain number and it just throws up false.

But heck, if a new scripting engine is on the way then it hopefully won't matter.

[Slaktmaster]

Holy moly.

[Bhala]

Yea i noticed that last night too.... it made it a lot harder to make my script

[Kaimetsu]

[Squeaky!]
In the meanwhile you could just use if (true), just as long and it does exactly the same.
Who knows maybe even if (1) works.
[/Squeaky!]

[Kaimetsu]

Originally posted by Kaimetsu
[Squeaky!]
In the meanwhile you could just use if (true), just as long and it does exactly the same.
Who knows maybe even if (1) works.
[/Squeaky!]

It's because I was using a defNPC. So I'd go:

NPC Code:

---

//#EDIT mode 1

//#CLIENTSIDE

blahblah;

if(mode==1){

 whatever;

}

---

So then it'd get changed into "if(1==1){" by the editor, y'see?

[Admins]

Arg optimization error, when comparing two
const values then it tries to merge, but two code
lines need to be exchanged. Fixed in next
version (the parsing engine doesn't do the optimizing
task so it was good to mention it :-) )

[Kaimetsu]

Originally posted by Kaimetsu


It's because I was using a defNPC. So I'd go:

NPC Code:

---

//#EDIT mode 1

//#CLIENTSIDE

blahblah;

if(mode==1){

 whatever;

}

---

So then it'd get changed into "if(1==1){" by the editor, y'see?

[Squeaky!]
Ok, that explains a lot. Though I fail to see it's actual use.
[/Squeaky!]

[Kaimetsu]

Originally posted by Kaimetsu
[Squeaky!]
Ok, that explains a lot. Though I fail to see it's actual use.
[/Squeaky!]

I had a magic fountain NPC that sprayed up light-effect particles, and it could originally be either Red, Green, Blue or a mixture of all. To define which colour it should be, the user entered a value into a "Colour" slot in the defNPC. Then when it came to assigning colours to the particles, it did something like this:

NPC Code:

---

for(i=0;i!=3;i++){

 if(Colour==i||Colour==-1) particlecolour[i]=random(0,1);

}

---

Because -1 was the value used to represent 'use all colours'. So when the editor went through and replaced the slot variable things with numbers, it made it into:

NPC Code:

---

for(i=0;i!=3;i++){

 if(-1==i||-1==-1) particlecolour[i]=random(0,1);

}

---

Which meant that the if condition always returned false.