[FFmpeg-devel] [PATCH] checkasm: Silence warnings about unused return value from read()

[Swinney, Jonathan]

This patch looks good to me. I would appreciate its merging.

-- 

Jonathan Swinney

On 5/25/22, 2:43 AM, "ffmpeg-devel on behalf of Martin Storsjö" <ffmpeg-devel-bounces at ffmpeg.org on behalf of martin at martin.st> wrote:

    CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you can confirm the sender and know the content is safe.



    This codepath is enabled by default on arm, if the linux perf API
    is available, unless disabled with --disable-linux-perf.
    ---
     tests/checkasm/checkasm.h | 4 +++-
     1 file changed, 3 insertions(+), 1 deletion(-)

    diff --git a/tests/checkasm/checkasm.h b/tests/checkasm/checkasm.h
    index a86db140e3..7fd1e1606d 100644
    --- a/tests/checkasm/checkasm.h
    +++ b/tests/checkasm/checkasm.h
    @@ -229,8 +229,10 @@ typedef struct CheckasmPerf {
         ioctl(sysfd, PERF_EVENT_IOC_ENABLE, 0);             \
     } while (0)
     #define PERF_STOP(t) do {                               \
    +    int ret;                                            \
         ioctl(sysfd, PERF_EVENT_IOC_DISABLE, 0);            \
    -    read(sysfd, &t, sizeof(t));                         \
    +    ret = read(sysfd, &t, sizeof(t));                   \
    +    (void)ret;                                          \
     } while (0)
     #elif CONFIG_MACOS_KPERF
     #define PERF_START(t) t = ff_kperf_cycles()
    --
    2.32.0 (Apple Git-132)

    _______________________________________________
    ffmpeg-devel mailing list
    ffmpeg-devel at ffmpeg.org
    https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

    To unsubscribe, visit link above, or email
    ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".