Merge branch 'i2c' of https://github.com/endail/hx711-pico-c into i2c

endail · Feb 1, 2025 · 47fec86 · 47fec86
2 parents 0f22bc6 + 8bf2a5b
commit 47fec86
Showing 1 changed file with 21 additions and 3 deletions.
diff --git a/cppcheck_report.txt b/cppcheck_report.txt
@@ -34,15 +34,33 @@ src/hx711_multi.c:744:13: style: Variable 'interrupt_status_cb918069_eadf_49bc_9
 src/hx711_multi.c:858:9: style: Variable 'interrupt_status_cb918069_eadf_49bc_9d8c_a8a4defad20c_4' is assigned a value that is never used. [unreadVariable]
         HX711_MUTEX_BLOCK(hxm->_mut,
         ^
-src/spifixedframe.c:295:34: style: Condition 'byteIndex<expected_bytes_len' is always true [knownConditionTrueFalse]
+src/spifixedframe.c:327:34: style: Condition 'byteIndex<expected_bytes_len' is always true [knownConditionTrueFalse]
                     if(byteIndex < expected_bytes_len) {
                                  ^
-src/spifixedframe.c:281:30: note: Assuming condition 'byteIndex>=expected_bytes_len' is false
+src/spifixedframe.c:313:30: note: Assuming condition 'byteIndex>=expected_bytes_len' is false
                 if(byteIndex >= expected_bytes_len) {
                              ^
-src/spifixedframe.c:295:34: note: Condition 'byteIndex<expected_bytes_len' is always true
+src/spifixedframe.c:327:34: note: Condition 'byteIndex<expected_bytes_len' is always true
                     if(byteIndex < expected_bytes_len) {
                                  ^
+src/spifixedframe.c:209:26: style:inconclusive: Function 'spifixedframe_req_resp' argument 2 names different: declaration 'req' definition 'req_bytes'. [funcArgNamesDifferent]
+    const uint8_t* const req_bytes,
+                         ^
+include/spifixedframe.h:184:26: note: Function 'spifixedframe_req_resp' argument 2 names different: declaration 'req' definition 'req_bytes'.
+    const uint8_t* const req,
+                         ^
+src/spifixedframe.c:209:26: note: Function 'spifixedframe_req_resp' argument 2 names different: declaration 'req' definition 'req_bytes'.
+    const uint8_t* const req_bytes,
+                         ^
+src/spifixedframe.c:211:20: style:inconclusive: Function 'spifixedframe_req_resp' argument 4 names different: declaration 'resp' definition 'resp_bytes'. [funcArgNamesDifferent]
+    uint8_t* const resp_bytes,
+                   ^
+include/spifixedframe.h:186:20: note: Function 'spifixedframe_req_resp' argument 4 names different: declaration 'resp' definition 'resp_bytes'.
+    uint8_t* const resp,
+                   ^
+src/spifixedframe.c:211:20: note: Function 'spifixedframe_req_resp' argument 4 names different: declaration 'resp' definition 'resp_bytes'.
+    uint8_t* const resp_bytes,
+                   ^
 src/util.c:545:9: style: Variable 'interrupt_status_cb918069_eadf_49bc_9d8c_a8a4defad20c_0' is assigned a value that is never used. [unreadVariable]
         UTIL_INTERRUPTS_OFF_BLOCK(
         ^