Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No error reported with "ZERO_VECTOR.x" and similar constructions #63

Closed
buildersbrewery opened this issue Nov 29, 2017 · 2 comments
Closed

Comments

@buildersbrewery
Copy link

The following code does NOT compile in Second Life:

Error should be at (5, 30) at the '.'

default
{
    state_entry()
    {
        float f = ZERO_VECTOR.x;
        llOwnerSay((string)f);
    }
}

however lslint does not complain. Tested with v1.0.8.

@Sei-Lisa
Copy link

Sei-Lisa commented Nov 30, 2017

Thanks for the report. It's a consequence of commit 24127a6 (the one that introduces adds constants to builtins.txt), and it's irritating, because in that commit, constants are treated by the syntax analyzer like any other variables, making it more difficult to catch that situation.

@Sei-Lisa
Copy link

#64 should fix this, though I'd appreciate feedback regarding the error message.

@Makopo Makopo closed this as completed in #64 Dec 3, 2017
Makopo pushed a commit that referenced this issue Dec 3, 2017
e.g. ZERO_VECTOR.x

Introduced in 24127a6 when constants were added to builtins.txt.

Fixes #63.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants