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.

Sign up for GitHub

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

Jump to bottom

Use `size_t` for Janet types length and capacity and the API those types use #1438

Open

GrayJack wants to merge 28 commits into janet-lang:master from GrayJack:use-size_t

Open

Use `size_t` for Janet types length and capacity and the API those types use #1438

Update src/core/string.c

GitHub Advanced Security / CodeQL succeeded May 19, 2024 in 4s

5 new alerts

New alerts in code changed by this pull request

5 notes

See annotations below for details.

View all branch alerts.

Annotations

Check notice on line 191 in src/core/specials.c

Code scanning / CodeQL

Declaration hides variable Note

Variable len hides another variable of the same name (on

line 158

).

Check notice on line 617 in src/core/asm.c

Code scanning / CodeQL

Declaration hides variable Note

Variable i hides another variable of the same name (on

line 495

).

Check notice on line 776 in src/core/asm.c

Code scanning / CodeQL

Declaration hides variable Note

Variable i hides another variable of the same name (on

line 495

).

Check notice on line 1387 in src/core/peg.c

Code scanning / CodeQL

Declaration hides variable Note

Variable i hides another variable of the same name (on

line 1293

).

Check notice on line 547 in src/core/buffer.c

Code scanning / CodeQL

Equality test on floating-point values Note

Equality checks on floating point values can yield unexpected results.