ArkScript-lang / ArkScript-lang/Ark

timestamp unused

Đang mở
#339 1 bình luận 0 reaction 0 người được giao Xem trên GitHub
❗️ priority/medium 🔥 bug 🤔 help wanted
Ngôn ngữ chính
C++
Star
728
Fork
54
Merge trung bình
4 giờ 8 phút
Pull request đã merge (30 ngày)
2

Mô tả

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

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Hướng nghiên cứu

Mở src/arkreactor/VM/State.cpp và kiểm tra State::configure(), đặc biệt là khai báo timestamp gần dòng 236 và phép tính của nó gần dòng 246. Tái hiện bản build Linux GCC để xác nhận cảnh báo unused-but-set, sau đó xác định xem timestamp có nên được sử dụng hay xóa bỏ không. Hoàn thành khi bản build kết thúc mà không có cảnh báo này.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
cpp
Lĩnh vực
compilers
Loại issue
Lỗi
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
48/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.