Skip to content

backend/avision.c adf_reset(): In debug message "adf_reset: read ... failed...

Adam Richter requested to merge adamjrichter/backends:master into master

backend/avision.c adf_reset(): In debug message "adf_reset: read ... failed ...",

  1. fix arithmetic: "2-n" was probably copied from an earlier line, but the size value that this debug message should print seems to me to be 4-n, as that was the size value passed to the call that this debug message monitors.

  2. Fix a compiler warning about printf integer format type mismatch (use %zu for size_t instead of %lu). "%z" is apparently already used in debug statements in the backend directory tree in hp5590_cmds.c, ricoh2.c and genesys/row_buffer.h.

If I understand correctly, this commit appears to have passed whatever testing pipeline was automatically set up on my local fork.

I am submitting this simple fix ahead of the more complex data copying optimization that I posted to the sane-devel mailing list because this simpler change is an easier test of the submission process, and I would like to wait a day to see if anyone wants to comment on the mailing list about that more complex patch.

Please let me know if you would prefer that follow a different process in a way. Thanks in advance for your attention to this submission.

Merge request reports