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

syntax error found #581

Closed
VerserVarlin opened this issue Sep 25, 2024 · 4 comments
Closed

syntax error found #581

VerserVarlin opened this issue Sep 25, 2024 · 4 comments
Labels
bug Something isn't working need repro This bug report needs confirmation

Comments

@VerserVarlin
Copy link

Summary

was getting a syntax error in qbx_core\modules\lib.lua. fix included

Reproduction

skip

Expected behavior

how you have the code block line 121-127

function qbx.table.size(tbl)
    local size = 0
    for _ in pairs(tbl) do
        size += 1
    end
    return size
end

here is a fix for it

function qbx.table.size(tbl)
    local size = 0
    for _ in pairs(tbl) do
        size = size + 1
    end
    return size
end

Actual behavior

skip

Additional context

No response

Current Version

1.19.0

Custom Resources

no

@VerserVarlin VerserVarlin added bug Something isn't working need repro This bug report needs confirmation labels Sep 25, 2024
@qbox-duck qbox-duck bot added this to Issues Sep 25, 2024
@github-project-automation github-project-automation bot moved this to Todo in Issues Sep 25, 2024
@Manason
Copy link
Contributor

Manason commented Sep 25, 2024

Summary

was getting a syntax error in qbx_core\modules\lib.lua. fix included

Reproduction

skip

Expected behavior

how you have the code block line 121-127

function qbx.table.size(tbl)
    local size = 0
    for _ in pairs(tbl) do
        size += 1
    end
    return size
end

here is a fix for it

function qbx.table.size(tbl)
    local size = 0
    for _ in pairs(tbl) do
        size = size + 1
    end
    return size
end

Actual behavior

skip

Additional context

No response

Current Version

1.19.0

Custom Resources

no

What syntax error were you getting? What game version are you running? Have you modified qbx_core at all?

@mafewtm
Copy link
Member

mafewtm commented Sep 25, 2024

That's what this is for

@VerserVarlin
Copy link
Author

That's what this is for

if so then i should not be getting a syntax error from it, right?

@github-project-automation github-project-automation bot moved this from Todo to Done in Issues Sep 25, 2024
@Manason
Copy link
Contributor

Manason commented Sep 25, 2024

That's what this is for

if so then i should not be getting a syntax error from it, right?

Did you figure out the reason for the syntax error?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working need repro This bug report needs confirmation
Projects
Status: Done
Development

No branches or pull requests

3 participants