Patchew URL: https://patchew.org/QEMU/20210316104745.2196286-1-kra...@redhat.com/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20210316104745.2196286-1-kra...@redhat.com Subject: [PULL 0/3] Audio 20210316 patches === 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 === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu * [new tag] patchew/20210316104745.2196286-1-kra...@redhat.com -> patchew/20210316104745.2196286-1-kra...@redhat.com Switched to a new branch 'test' 54c97ba coreaudio: Handle output device change 3013a57 coreaudio: Extract device operations 45d1cb4 coreaudio: Drop support for macOS older than 10.6 === OUTPUT BEGIN === 1/3 Checking commit 45d1cb4bb7c7 (coreaudio: Drop support for macOS older than 10.6) 2/3 Checking commit 3013a5778d0a (coreaudio: Extract device operations) ERROR: space prohibited between function name and open parenthesis '(' #76: FILE: audio/coreaudio.c:383: + coreaudio_playback_logerr (status, ERROR: space prohibited between function name and open parenthesis '(' #83: FILE: audio/coreaudio.c:388: + dolog ("Could not initialize playback - Unknown Audiodevice\n"); ERROR: space prohibited between function name and open parenthesis '(' #94: FILE: audio/coreaudio.c:396: + coreaudio_playback_logerr (status, ERROR: space prohibited between function name and open parenthesis '(' #122: FILE: audio/coreaudio.c:415: + coreaudio_playback_logerr (status, ERROR: line over 90 characters #123: FILE: audio/coreaudio.c:416: + "Could not set device buffer frame size %" PRIu32 "\n", WARNING: line over 80 characters #124: FILE: audio/coreaudio.c:417: + (uint32_t)core->audioDevicePropertyBufferFrameSize); ERROR: space prohibited between function name and open parenthesis '(' #135: FILE: audio/coreaudio.c:425: + coreaudio_playback_logerr (status, WARNING: line over 80 characters #141: FILE: audio/coreaudio.c:429: + core->hw.samples = core->bufferCount * core->audioDevicePropertyBufferFrameSize; ERROR: space prohibited between function name and open parenthesis '(' #149: FILE: audio/coreaudio.c:435: + coreaudio_playback_logerr (status, ERROR: space prohibited between function name and open parenthesis '(' #164: FILE: audio/coreaudio.c:445: + coreaudio_playback_logerr (status, WARNING: line over 80 characters #166: FILE: audio/coreaudio.c:447: + core->outputStreamBasicDescription.mSampleRate); ERROR: space prohibited between function name and open parenthesis '(' #181: FILE: audio/coreaudio.c:459: + coreaudio_playback_logerr (status, "Could not set IOProc\n"); ERROR: space prohibited between function name and open parenthesis '(' #218: FILE: audio/coreaudio.c:501: + dolog("Could not create mutex\nReason: %s\n", strerror (err)); ERROR: space prohibited between function name and open parenthesis '(' #225: FILE: audio/coreaudio.c:508: + audio_pcm_init_info (&hw->info, as); ERROR: space prohibited between function name and open parenthesis '(' #240: FILE: audio/coreaudio.c:523: +static void coreaudio_fini_out (HWVoiceOut *hw) total: 12 errors, 3 warnings, 224 lines checked Patch 2/3 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 3/3 Checking commit 54c97bac0510 (coreaudio: Handle output device change) ERROR: space prohibited between function name and open parenthesis '(' #74: FILE: audio/coreaudio.c:335: + coreaudio_unlock (core, "audioDeviceIOProc(old device)"); ERROR: line over 90 characters #160: FILE: audio/coreaudio.c:495: + if (status != kAudioHardwareBadDeviceError && status != kAudioHardwareNoError) { WARNING: line over 80 characters #170: FILE: audio/coreaudio.c:504: + if (status != kAudioHardwareBadDeviceError && status != kAudioHardwareNoError) { ERROR: line over 90 characters #195: FILE: audio/coreaudio.c:529: + if (status != kAudioHardwareBadDeviceError && status != kAudioHardwareNoError) { ERROR: space prohibited between function name and open parenthesis '(' #196: FILE: audio/coreaudio.c:530: + coreaudio_logerr (status, "Could not resume playback\n"); ERROR: line over 90 characters #204: FILE: audio/coreaudio.c:538: + if (status != kAudioHardwareBadDeviceError && status != kAudioHardwareNoError) { ERROR: space prohibited between function name and open parenthesis '(' #233: FILE: audio/coreaudio.c:567: + coreaudio_unlock (core, __func__); ERROR: space prohibited between function name and open parenthesis '(' #263: FILE: audio/coreaudio.c:607: + coreaudio_playback_logerr (status, ERROR: space prohibited between function name and open parenthesis '(' #291: FILE: audio/coreaudio.c:635: + dolog("Could not destroy mutex\nReason: %s\n", strerror (err)); WARNING: line over 80 characters #312: FILE: audio/coreaudio.c:657: + coreaudio_logerr(status, "Could not remove voice property change listener\n"); total: 8 errors, 2 warnings, 321 lines checked Patch 3/3 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/20210316104745.2196286-1-kra...@redhat.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com