ArkScript-lang / ArkScript-lang/Ark

timestamp unused

Abierto
#339 1 comentario 0 reacciones 0 asignados Ver en GitHub
❗️ priority/medium 🔥 bug 🤔 help wanted
Lenguaje dominante
C++
Estrellas
728
Forks
54
Merge medio
4 h 8 min
PR fusionados (30 d)
2

Descripción

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

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Abre src/arkreactor/VM/State.cpp e inspecciona State::configure(), especialmente la declaración del timestamp cerca de la línea 236 y su cálculo cerca de la línea 246. Reproduce la compilación de Linux GCC para confirmar la advertencia unused-but-set y, a continuación, determina si el timestamp debe utilizarse o eliminarse. Se considera terminado cuando la compilación finaliza sin esta advertencia.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
cpp
Área
compilers
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
48/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.