-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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.
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
Warnint 64to32 6186 v4 #9169
Warnint 64to32 6186 v4 #9169
Conversation
a03a2c3
to
faa0a44
Compare
Ticket: OISF#6186 clang warns and errors on 64 to 32 bits integer conversion
d72c507
to
b3e1427
Compare
WARNING:
Pipeline 15074 |
3f2c835
to
a71708b
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #9169 +/- ##
==========================================
- Coverage 82.34% 82.33% -0.02%
==========================================
Files 968 968
Lines 273546 273589 +43
==========================================
+ Hits 225258 225265 +7
- Misses 48288 48324 +36
Flags with carried forward coverage won't be shown. Click here to find out more. |
Ticket: OISF#6186 Warnings about downcast from 64 to 32 bits
a71708b
to
a287216
Compare
Information: QA ran without warnings. Pipeline 15176 |
Rebased in #9257 |
[Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/6186
Describe changes:
-Wshorten-64-to-32
flag to clang after-Wimplicit-int-conversion
OISF/libhtp#399
Modifies #9155 with temporary disabling of Terror to get full warnings list
How can I fix the CI about format ? Locally
./scripts/clang-format.sh branch
is happy