Improve extraction of the signed int.

Co-authored-by: Copilot <[email protected]>
Signed-off-by: Michael Keller <[email protected]>

Author: mikeller

src/hw_ostc_parser.c

@@ -1210,7 +1210,10 @@ hw_ostc_parser_internal_foreach (hw_ostc_parser_t *parser, dc_sample_callback_t
 				}
 
 				unsigned int scrubberState = array_uint16_le(data + offset);
-				int scrubberTimeMinutes = (scrubberState & 0x0800) ? (-1 ^ 0x0FFF) | (scrubberState & 0x0FFF) : scrubberState & 0x0FFF; // extract 12 bit signed int
+				int scrubberTimeMinutes = scrubberState & 0x0FFF; // Extract the 12-bit value
+				if (scrubberState & 0x0800) { // Check if the sign bit is set
+					scrubberTimeMinutes -= 0x1000; // Perform sign extension
+				}
 				if (parser->first_scrubber_time_minutes == INT_MAX) {
 					parser->first_scrubber_time_minutes = scrubberTimeMinutes;
 				}
@@ -1222,17 +1225,16 @@ hw_ostc_parser_internal_foreach (hw_ostc_parser_t *parser, dc_sample_callback_t
 						.event.flags = SAMPLE_FLAGS_SEVERITY_STATE,
 					};
 
-
 					if (scrubberState & OSTC4_SCRUBBER_STATE_ERROR_FLAG) {
 						if (!parser->scrubber_error_reported) {
-						    sample.event.flags = SAMPLE_FLAGS_SEVERITY_ALARM;
-						    parser->scrubber_error_reported = true;
+							sample.event.flags = SAMPLE_FLAGS_SEVERITY_ALARM;
+							parser->scrubber_error_reported = true;
 						}
 						snprintf(buf, BUFLEN, "Scrubber exhausted: %d minutes remaining", scrubberTimeMinutes);
 					} else if (scrubberState & OSTC4_SCRUBBER_STATE_WARNING_FLAG) {
 						if (!parser->scrubber_warning_reported) {
-						    sample.event.flags = SAMPLE_FLAGS_SEVERITY_WARN;
-						    parser->scrubber_warning_reported = true;
+							sample.event.flags = SAMPLE_FLAGS_SEVERITY_WARN;
+							parser->scrubber_warning_reported = true;
 						}
 						snprintf(buf, BUFLEN, "Scrubber warning: %d minutes remaining", scrubberTimeMinutes);
 					} else {