Patchew URL: https://patchew.org/QEMU/[email protected]/
Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH v3] qemu-io: add pattern file for write command Type: series Message-id: [email protected] === TEST SCRIPT BEGIN === #!/bin/bash git rev-parse base > /dev/null || exit 0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === From https://github.com/patchew-project/qemu * [new tag] patchew/[email protected] -> patchew/[email protected] Switched to a new branch 'test' 6579ff71db qemu-io: add pattern file for write command === OUTPUT BEGIN === ERROR: suspect code indent for conditional statements (8, 11) #50: FILE: qemu-io-cmds.c:370: + if (feof(f)) { + rewind(f); WARNING: line over 80 characters #132: FILE: qemu-io-cmds.c:1116: + printf("Only one of -z, -P, and -s can be specified at the same time\n"); ERROR: Missing Signed-off-by: line(s) total: 2 errors, 1 warnings, 129 lines checked Commit 6579ff71dbaa (qemu-io: add pattern file for write command) has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/[email protected]/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to [email protected]
