ArkScript-lang / ArkScript-lang/Ark

timestamp unused

Offen
#339 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
❗️ priority/medium 🔥 bug 🤔 help wanted
Vorherrschende Sprache
C++
Sterne
728
Forks
54
Ø Merge
4 Std. 8 Min.
Gemergte PRs (30 T.)
2

Beschreibung

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

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Öffne src/arkreactor/VM/State.cpp und untersuche State::configure(), insbesondere die Deklaration des Zeitstempels nahe Zeile 236 und seine Berechnung nahe Zeile 246. Reproduziere den Linux-GCC-Build, um die unused-but-set-Warnung zu bestätigen, und ermittle anschließend, ob der Zeitstempel verwendet oder entfernt werden sollte. Erledigt ist die Aufgabe, wenn der Build ohne diese Warnung abgeschlossen wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
compilers
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.