timestamp unused
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 48/100
Research direction
Open src/arkreactor/VM/State.cpp and inspect State::configure(), especially the timestamp declaration near line 236 and its calculation near line 246. Reproduce the Linux GCC build to confirm the unused-but-set warning, then determine whether the timestamp should be used or removed. Done means the build completes without this warning.
Written by the indexing model from the issue text.
Description
Describe the bug
On Linux GCC Compiler we get the following warning:
[ 37%] Building CXX object CMakeFiles/ArkReactor.dir/src/arkreactor/VM/State.cpp.o
/home/runner/work/Ark/Ark/src/arkreactor/VM/State.cpp: In member function ‘void Ark::State::configure()’:
/home/runner/work/Ark/Ark/src/arkreactor/VM/State.cpp:236:21: error: variable ‘timestamp’ set but not used [-Werror=unused-but-set-variable]
236 | timestamp_t timestamp = 0;
| ^~~~~~~~~
To Reproduce
Compile on Linux using GCC
Expected behavior
There is some effort put into calculating the value of timestamp (see line 246).
So it looks like it should be used.
Additional context
This should be fixed or the variable removed.
- Dominant language
- C++
- Stars
- 729
- Forks
- 54
- Avg merge
- 4h 8m
- Merged PRs (30d)
- 2
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ArkScript-lang/Ark
-
Difficulty 3/5 1-2 days Newbie friendliness 65/100
ArkScript-lang/Ark#712 · 2 comments ·
-
🤔 help wanted ark/repl documentation enhancement
Difficulty 5/5 Over a week Newbie friendliness 35/100
ArkScript-lang/Ark#711 ·
-
❗️ priority/low 👷 ci 🤔 help wanted github_actions
Difficulty 5/5 Over a week Newbie friendliness 30/100
ArkScript-lang/Ark#708 ·
-
Link Checker Report Openautomated issue report
Difficulty 2/5 1-3 hours Newbie friendliness 55/100
ArkScript-lang/Ark#700 ·
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 25/100
ArkScript-lang/Ark#631 ·
All issues in ArkScript-lang/Ark
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
AXERA-TECH/ax-llm#77 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
games-on-whales/wolf#509 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
bug-unconfirmed
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
NVIDIA/cuda-samples#453 ·