Re: [FFmpeg-devel] [PATCH]lavf/sga: Fix several format specifiers

2021-03-08 Thread Paul B Mahol
On Mon, Mar 8, 2021 at 9:22 AM Anton Khirnov wrote: > Quoting Carl Eugen Hoyos (2021-03-04 22:47:57) > > Am Do., 4. März 2021 um 22:23 Uhr schrieb Moritz Barsnick < > barsn...@gmx.net>: > > > > > > On Thu, Mar 04, 2021 at 20:37:26 +0100, Carl Eugen Hoyos wrote: > > > > attached patch fixes a few

Re: [FFmpeg-devel] [PATCH]lavf/sga: Fix several format specifiers

2021-03-08 Thread Anton Khirnov
Quoting Carl Eugen Hoyos (2021-03-04 22:47:57) > Am Do., 4. März 2021 um 22:23 Uhr schrieb Moritz Barsnick : > > > > On Thu, Mar 04, 2021 at 20:37:26 +0100, Carl Eugen Hoyos wrote: > > > attached patch fixes a few warnings when compiling for 32bit. > > > > > av_log(s, AV_LOG_DEBUG, "START %s\n

Re: [FFmpeg-devel] [PATCH]lavf/sga: Fix several format specifiers

2021-03-04 Thread Moritz Barsnick
On Thu, Mar 04, 2021 at 22:47:57 +0100, Carl Eugen Hoyos wrote: > > #include > > It is included through common.h - do we add it to every C file > that would need it if common.h were not included? Okay, that's quite indirectly and implicitly via libavutil/internal.h. I don't know the policy on thi

Re: [FFmpeg-devel] [PATCH]lavf/sga: Fix several format specifiers

2021-03-04 Thread Carl Eugen Hoyos
Am Do., 4. März 2021 um 22:23 Uhr schrieb Moritz Barsnick : > > On Thu, Mar 04, 2021 at 20:37:26 +0100, Carl Eugen Hoyos wrote: > > attached patch fixes a few warnings when compiling for 32bit. > > > av_log(s, AV_LOG_DEBUG, "START %s\n", where); > > -av_log(s, AV_LOG_DEBUG, "pos: %lX\n", a

Re: [FFmpeg-devel] [PATCH]lavf/sga: Fix several format specifiers

2021-03-04 Thread Moritz Barsnick
On Thu, Mar 04, 2021 at 20:37:26 +0100, Carl Eugen Hoyos wrote: > attached patch fixes a few warnings when compiling for 32bit. > av_log(s, AV_LOG_DEBUG, "START %s\n", where); > -av_log(s, AV_LOG_DEBUG, "pos: %lX\n", avio_tell(s->pb)); > +av_log(s, AV_LOG_DEBUG, "pos: %"PRIX64"\n", av

Re: [FFmpeg-devel] [PATCH]lavf/sga: Fix several format specifiers

2021-03-04 Thread Paul B Mahol
lgtm ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH]lavf/sga: Fix several format specifiers

2021-03-04 Thread Carl Eugen Hoyos
Hi, attached patch fixes a few warnings when compiling for 32bit. Please comment, Carl Eugen From c9941035b19d52fe8d8cbe2183fec823ea7a59d4 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Thu, 4 Mar 2021 20:34:36 +0100 Subject: [PATCH] lavf/sga: Silence several warnings because of wrong for