Fightwarn libusb 1.0+0.1 #1241
Merged
Fightwarn libusb 1.0+0.1 #1241
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jimklimov
added
CI
|
UPDATE: This PR got frankensteined into a mess of changes hard to revise due to a history of merges to keep in sync with master; so I sat to cherry-pick and will now be posting another to increment libusb-1.0 branch with a general fightwarn effort running-up to the "questionable" change noted above, then will rebase this PR on top of that with fewer changes to revise specific to this iteration. |
… int types to backend API dependent "usb_ctrl_*" and "usb_dev_handle" typedefs, then range-check and cast the numbers, and avoid build warnings
jimklimov
force-pushed
the
fightwarn-libusb-1.0+0.1
branch
from
b770870
to
4de9a75
Compare
|
PR rebased over the less significant change set, now merged into target branch. |
…on.h due to lack of WITH_LIBUSB_x_y)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR updates the libusb-1.0+0.1 branch with "fightwarn" considerations (following up from effort in #823 to get the NUT codebase CI-tested and free from more and more classes of warnings, so we can enable non-regression testing for them with future iterations), allowing to proceed with issue #300 to get libusb 1.0 API supported by NUT officially. In fact, this PR would complete the current definition of "medium" level warnings, allowing to set them as default requirement.
One important change here, that was questionable to myself but got "rationalized", is the step back from fixed C-standard fixed-size numeric value types like
size_t, totypedef ... usb_ctrl_...with types defined by respective backend API for libusb-1.0, libusb-0.1, libhid, libshut and some drivers. For statically linked drivers this should not matter in fact, and this change does not preclude (and even helps) later efforts to re-define those typedefs identically for all backends, and add more range-checks and casts to make the complete codebase build with that.This PR also reduces duplication of code (libusb 0.1 vs 1.0 definitions were repeated in many drivers and libs) offloading that to usb-common.h
At least, such is the theory - testing against real hardware would be very helpful; if you do - please comment the good, the bad and the ugly outcomes in #300.
Sibling PR #1240 addresses similar refresh for the original libusb-1.0 branch, but this one (libusb-1.0+0.1) is planned to become part of NUT master codebase when the dust settles.
During the co-evolution of these branches, it was confirmed (and later assured by git-merges) that libusb-1.0+0.1 includes and builds upon libusb-1.0. The original difference was the ability to
configurea build of NUT with one or another libusb API. As part of this PR, a newer m4 codebase to do so was applied to both branches, so now they should only differ in other bugfixes added in the libusb-1.0+0.1 branch.