cjit

fix: refactor core CJIT code #342

Sign in to view logs

Triggered via pull request December 26, 2024 20:13

jaromil

synchronize #90

fix/refactor

Status Success

Total duration 2m 4s

Artifacts –

main.yml

on: pull_request

🚨 REUSE Compliance

🐧 Linux x86 test

🪟 Windows native test

🍎 OSX native test

🤖 Semantic release

🐧 Linux shared binary release build

🐧 Musl Linux binary release build

🪟 Windows native binary release build

🍎 OSX native binary release build

📢 Public release

🧻 Remove tag on fail

🦠 Virus scan of released binaries

Annotations

15 warnings

🚨 REUSE Compliance

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

[cpplint] src/cjit.c#L51: src/cjit.c#L51

If you can, use sizeof(filename) instead of 63 as the 2nd arg to snprintf. [runtime/printf] [3]

[cpplint] src/cjit.c#L79: src/cjit.c#L79

If you can, use sizeof(tempDir) instead of 259 as the 2nd arg to snprintf. [runtime/printf] [3]

[cpplint] src/cjit.c#L95: src/cjit.c#L95

Almost always, snprintf is better than strcpy [runtime/printf] [4]

[cpplint] src/cjit.c#L154: src/cjit.c#L154

Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]

[cpplint] src/cjit.c#L185: src/cjit.c#L185

Almost always, snprintf is better than strcpy [runtime/printf] [4]

[cpplint] src/cjit.c#L186: src/cjit.c#L186

Almost always, snprintf is better than strcat [runtime/printf] [4]

[cpplint] src/cjit.c#L193: src/cjit.c#L193

Almost always, snprintf is better than strcpy [runtime/printf] [4]

[cpplint] src/cjit.c#L195: src/cjit.c#L195

Almost always, snprintf is better than strcpy [runtime/printf] [4]

[cpplint] src/cjit.c#L307: src/cjit.c#L307

Add #include <cstdio> for vsnprintf [build/include_what_you_use] [4]

[cpplint] src/main.c#L151: src/main.c#L151

Almost always, snprintf is better than strcpy [runtime/printf] [4]

[cpplint] src/main.c#L156: src/main.c#L156

Almost always, snprintf is better than strcpy [runtime/printf] [4]

[cpplint] src/main.c#L170: src/main.c#L170

Add #include <cstdio> for fprintf [build/include_what_you_use] [4]

🚨 C lint

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

🐧 Linux x86 test

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636