win: hid_get_feature_report: Correct number of bytes_returned (libusb#286)

Youw · web-flow · commit 6fcb0bb2282d · 2021-06-19T07:23:35.000+03:00
This originally tested/confirmed by @mcuee. Confirmed the expected behavior by Tim Roberts https://sourceforge.net/p/libusb/mailman/message/37303650/. Relates to: libusb#232.
diff --git a/windows/hid.c b/windows/hid.c
@@ -881,11 +881,6 @@ int HID_API_EXPORT HID_API_CALL hid_get_feature_report(hid_device *dev, unsigned
 		return -1;
 	}
 
-	/* bytes_returned does not include the first byte which contains the
-	   report ID. The data buffer actually contains one more byte than
-	   bytes_returned. */
-	bytes_returned++;
-
 	return bytes_returned;
 #endif
 }

Original file line number	Diff line number	Diff line change
`@@ -881,11 +881,6 @@ int HID_API_EXPORT HID_API_CALL hid_get_feature_report(hid_device *dev, unsigned`
`881`	`881`	`return -1;`
`882`	`882`	`}`
`883`	`883`
`884`		`- /* bytes_returned does not include the first byte which contains the`
`885`		`- report ID. The data buffer actually contains one more byte than`
`886`		`- bytes_returned. */`
`887`		`- bytes_returned++;`
`888`		`-`
`889`	`884`	`return bytes_returned;`
`890`	`885`	`#endif`
`891`	`886`	`}`