Sign in to follow this  

Bug from 1.8.0

This topic is 4868 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Recommended Posts

I have found a bug, and I think that this is a bug, because every script writter can fall into this. To see the bug in action just take the last WIP (I have the same bug working with 1.8.0 too !) Take the testframework project too. In TestExecute4Argsf (or any other function), replace this line engine->ExecuteString(0, "cfunction(9.2f, 13.3f, 18.8, 3.1415f)", NULL, 1000, 0); with this line engine->ExecuteString(0, "{cfunction(9.2f, 13.3f, 18.8, 3.1415f)", NULL, 1000, 0); Can't see any difference ? Yes, I have included a '{' at the begining of the script but hey, no '}' at the end ! This is a match error ! Now compile the testframework, and run it : You get an Assert caused by asCParser::GetToken function. Regards, AbrKen

Share this post


Link to post
Share on other sites
Found the bug already. The fix is:

Change the if statement in function asCParser::GetToken() at line 1492 in as_parser.cpp to:


if( sourcePos >= sourceLength )
{
token->type = ttEnd;
token->length = 0;
}
else
token->type = tokenizer.GetToken(&script->code[sourcePos], sourceLength - sourcePos, &token->length);




The line number is for the latest work in progress. It might not be the same in earlier versions, but it shouldn't be too hard to find from the description.

Regards,
Andreas

Share this post


Link to post
Share on other sites
- Changed my profile :)
- Changed the code (in my old 1.8.0 !).
- Confirmed the bug eradication.


Building...
Parsing: "FlipImage"
Error : (53, 0) Expected expression value
Error : (53, 0) Unexpected end of file
Compiling function 'AcceuilEng_MOUSE_0' in "AcceuilEng_MOUSE_0"
Error : (4, 1) Function 'FlipImage' not found
Compiling function 'AcceuilFra_MOUSE_0' in "AcceuilFra_MOUSE_0"
Error : (4, 1) Function 'FlipImage' not found
Completed. (errors: 4, warnings: 0)


Great thank's for reactivity and best regards,

AbrKen.

Share this post


Link to post
Share on other sites

This topic is 4868 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

Sign in to follow this