ArkScript-lang / ArkScript-lang/Ark

timestamp unused

Aperta
#339 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
❗️ priority/medium 🔥 bug 🤔 help wanted
Lingua principale
C++
Stelle
728
Fork
54
Merge medio
4h 8m
PR unite (30g)
2

Descrizione

**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.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Apri src/arkreactor/VM/State.cpp e ispeziona State::configure(), in particolare la dichiarazione del timestamp vicino alla riga 236 e il relativo calcolo vicino alla riga 246. Riproduci la build Linux GCC per confermare l’avviso unused-but-set, quindi determina se il timestamp debba essere utilizzato o rimosso. L’attività è completata quando la build termina senza questo avviso.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
compilers
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
48/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.