Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Compile warnings in trace_api_plugin #21

Closed
linh2931 opened this issue Aug 18, 2022 · 0 comments · Fixed by #35
Closed

Compile warnings in trace_api_plugin #21

linh2931 opened this issue Aug 18, 2022 · 0 comments · Fixed by #35
Assignees
@linh2931
Labels
tech debt

Comments

@linh2931
Copy link
Member

linh2931 commented Aug 18, 2022 •
edited
Loading

Minor warnings:

plugins/trace_api_plugin/compressed_file.cpp:138:66: warning:   comparison of integer  expressions of different signedness: ‘std::__tuple_element_t<0,   std::tuple<long unsigned  int, long unsigned int> >’ {aka ‘long unsigned int’} and ‘long int’ [-Wsign-compare]

plugins/trace_api_plugin/compressed_file.cpp:289:30: warning:   comparison of integer  expressions of different signedness: ‘int’ and ‘const long  unsigned int’ [-Wsign-compare]

trace_api_plugin/trace_api_plugin.cpp:363:12: warning:  variable ‘cfg_options’ set but not used [-Wunused-but-set-variable]
@stephenpdeos stephenpdeos moved this to Todo in Team Backlog Aug 18, 2022
@linh2931 linh2931 self-assigned this Aug 21, 2022
@linh2931 linh2931 mentioned this issue Aug 22, 2022
Merged
@linh2931 linh2931 moved this from Todo to Awaiting Review in Team Backlog Aug 22, 2022
Repository owner moved this from Awaiting Review to Done in Team Backlog Aug 26, 2022
greg7mdp added a commit that referenced this issue Dec 15, 2023
@greg7mdp
Leaps builds and test pass except for those below:
3b411a8 
131/150 Test   #98: producer_schedule_hs_unit_test_eos-vm-jit ..........***Failed    1.36 sec
132/150 Test   #97: producer_schedule_hs_unit_test_eos-vm ..............***Failed    1.57 sec
133/150 Test   #96: producer_schedule_hs_unit_test_eos-vm-oc ...........***Failed    2.86 sec
144/150 Test   #21: api_unit_test_eos-vm-oc ............................***Failed   79.26 sec
146/150 Test   #23: api_unit_test_eos-vm-jit ...........................***Failed   96.61 sec
150/150 Test   #22: api_unit_test_eos-vm ...............................***Failed  467.14 sec
heifner added a commit that referenced this issue Apr 29, 2024
@heifner
Merge pull request #21 from AntelopeIO/disable-ship-tests
8cc9420 
IF: Test: Temporarily disable ship streamer tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@linh2931 linh2931

Labels
tech debt
Projects
Team Backlog
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[3.2] fix compile warnings in trace_api_plugin AntelopeIO/leap
1 participant
@linh2931