Fixes -Wunused-const-variable from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
Signed-off-by: Ganesh Ajjanagadde
---
libavcodec/metasound_data.c | 4
1 file changed, 4 deletions(-)
diff --git a/libavcodec/metasound_data.c b/libavcodec/metasound_d
Fixes -Wunused-const-variable from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
Signed-off-by: Ganesh Ajjanagadde
---
libavcodec/aacsbr_fixed.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/libavcodec/aacsbr_fixed.c b/libavcodec/aacsbr_fixed.c
inde
Fixes -Wenum-conversion from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
Signed-off-by: Ganesh Ajjanagadde
---
libavformat/ftp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/ftp.c b/libavformat/ftp.c
index 51f491c..dc
Fixes -Wsometimes-uninitialized from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
Signed-off-by: Ganesh Ajjanagadde
---
libavformat/webmdashenc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/webmdashenc.c b/libavformat
On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde
wrote:
> Fixes -Wunused-const-variable from
> http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavcodec/metasound_data.c | 4
> 1 file changed, 4 deleti
Thanks for the patch. A few remarks, specific to the stand-alone-commit
version or that I had not yet spotted.
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
> Signed-off-by: Stephan Holljes
> ---
> libavformat/http.c | 40 ++--
> 1 file changed,
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
> Signed-off-by: Stephan Holljes
> ---
> libavformat/http.c | 44 +++-
> 1 file changed, 39 insertions(+), 5 deletions(-)
>
> diff --git a/libavformat/http.c b/libavformat/http.c
> index bfe6801..
On Thu, Aug 20, 2015 at 05:34:22AM +0300, Vesselin Bontchev wrote:
> Hi!
>
> It would be awesome to have this patch merged in FFmpeg.
>
> Vesselin
> doc/demuxers.texi|6
> doc/general.texi |2
> libavformat/Makefile |1
> libavformat/aadec.c | 303
> +++
On Thu, Aug 20, 2015 at 04:29:12AM -0400, Ganesh Ajjanagadde wrote:
> Fixes -Wenum-conversion from
> http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavformat/ftp.c | 2 +-
> 1 file changed, 1 insertion(+), 1
On Thu, Aug 20, 2015 at 04:34:19AM -0400, Ganesh Ajjanagadde wrote:
> On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde
> wrote:
> > Fixes -Wunused-const-variable from
> > http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
> >
> > Signed-off-by: Ganesh Ajjana
Hi!
I finally realized why -vf fps=3/1001,fieldmatch,decimate does
not always work / produces ugly output.
The issue is not related to fieldmatch and/or decimate but to the
fps filter: It (sometimes) drops frames even if the framerate is
increased. So the question is: Is it a good idea that
Hi,
On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde
wrote:
> Fixes -Wunused-const-variable from
>
> http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavcodec/aacsbr_fixed.c | 1 -
> 1 file changed, 1 del
From: Sven Dueking
---
libavcodec/qsvenc.c | 36 +---
libavcodec/qsvenc.h | 12
libavcodec/qsvenc_h264.c | 12
3 files changed, 57 insertions(+), 3 deletions(-)
diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c
index 15322
Hi,
On Thu, Aug 20, 2015 at 4:29 AM, Ganesh Ajjanagadde
wrote:
> Fixes -Wsometimes-uninitialized from
>
> http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavformat/webmdashenc.c | 2 +-
> 1 file changed, 1
On Thu, Aug 20, 2015 at 11:02 AM, Nicolas George wrote:
> Thanks for the patch. A few remarks, specific to the stand-alone-commit
> version or that I had not yet spotted.
>
>
> Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
>> Signed-off-by: Stephan Holljes
>> ---
>> libavformat/htt
On Thu, Aug 20, 2015 at 11:08 AM, Nicolas George wrote:
> Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
>> Signed-off-by: Stephan Holljes
>> ---
>> libavformat/http.c | 44 +++-
>> 1 file changed, 39 insertions(+), 5 deletions(-)
>>
>> diff -
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
> Added to all while(!av_isspace(p)) loops in the server code.
You may want to send this part in a stand-alone patch, since it is actually
a bug in the current code.
> This is more or less temporal, since the user-agent field is/will be
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
> It's the opposite. When the request was HEAD, the location header is
> not set, otherwise it is.
Indeed, but still, why? I just checked with a random Apache server, and HEAD
also returns the Location header.
Regards,
--
Nicolas Geo
Fixes -Wsometimes-uninitialized from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
Signed-off-by: Ganesh Ajjanagadde
---
libavformat/webmdashenc.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavformat/webmdashenc.c b/libav
This is shorter and consistent across filters.
Signed-off-by: Paul B Mahol
---
libavfilter/vf_dejudder.c | 66 +++
1 file changed, 33 insertions(+), 33 deletions(-)
diff --git a/libavfilter/vf_dejudder.c b/libavfilter/vf_dejudder.c
index ab525b6..4705
On Thu, Aug 20, 2015 at 5:15 PM, Nicolas George wrote:
> Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
>> Added to all while(!av_isspace(p)) loops in the server code.
>
> You may want to send this part in a stand-alone patch, since it is actually
> a bug in the current code.
>
>> Thi
On Thu, Aug 20, 2015 at 5:17 PM, Nicolas George wrote:
> Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
>> It's the opposite. When the request was HEAD, the location header is
>> not set, otherwise it is.
>
> Indeed, but still, why? I just checked with a random Apache server, and HEAD
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
> Should the same be done for the Host-parameter sent by the client too?
It can be discussed.
On one hand, the Host field has a special meaning for the protocol, unlike
User-Agent that is only used for logging, so it makes sense to give
On Thu, Aug 20, 2015 at 5:32 PM, Nicolas George wrote:
> Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
>> Should the same be done for the Host-parameter sent by the client too?
>
> It can be discussed.
>
> On one hand, the Host field has a special meaning for the protocol, unlike
> U
Signed-off-by: Stephan Holljes
---
libavformat/http.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavformat/http.c b/libavformat/http.c
index a136918..18df34e 100644
--- a/libavformat/http.c
+++ b/libavformat/http.c
@@ -716,7 +716,7 @@ static int process_line(URLCont
Introduce the ability to set reply body and reply text and read the
url-query, http-version and host parameter.
Signed-off-by: Stephan Holljes
---
libavformat/http.c | 32 ++--
1 file changed, 30 insertions(+), 2 deletions(-)
Does not export User-Agent, unlike the pr
Add error codes 301 and 503, make replies more customizable
and use AVBPrint for strings.
Signed-off-by: Stephan Holljes
---
libavformat/http.c | 75 +-
1 file changed, 63 insertions(+), 12 deletions(-)
diff --git a/libavformat/http.c b/libavf
On Thu, Aug 20, 2015 at 11:24 AM, Paul B Mahol wrote:
> This is shorter and consistent across filters.
>
> Signed-off-by: Paul B Mahol
> ---
> libavfilter/vf_dejudder.c | 66
> +++
> 1 file changed, 33 insertions(+), 33 deletions(-)
>
> diff --git a/l
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
> Signed-off-by: Stephan Holljes
> ---
> libavformat/http.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
LGTM, thanks.
Note to Hendrik and the others who took over the merging: Until recently,
Michael pulled from my tree
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
> In ffserver the User-Agent is used to decide how to deal with a
> client. (WMP being a special case.)
Ok, but that is special case for a specific application.
> That sounds pretty optimistic. Maybe it is trivial for you, but from
> my
On 8/20/15, Ganesh Ajjanagadde wrote:
> On Thu, Aug 20, 2015 at 11:24 AM, Paul B Mahol wrote:
>> This is shorter and consistent across filters.
>>
>> Signed-off-by: Paul B Mahol
>> ---
>> libavfilter/vf_dejudder.c | 66
>> +++
>> 1 file changed, 33 in
This patch revision (v9) adds me to the MAINTAINERS file.
VesselinFrom a9457dead0dac503c9aeca8d49bc89d5c9b607d9 Mon Sep 17 00:00:00 2001
From: Vesselin Bontchev
Date: Sun, 19 Jul 2015 23:16:36 +0200
Subject: [PATCH] Add support for Audible AA files
https://en.wikipedia.org/wiki/Audible.com#Quali
On Thu, Aug 20, 2015 at 12:59 PM, Paul B Mahol wrote:
> On 8/20/15, Ganesh Ajjanagadde wrote:
>> On Thu, Aug 20, 2015 at 11:24 AM, Paul B Mahol wrote:
>>> This is shorter and consistent across filters.
>>>
>>> Signed-off-by: Paul B Mahol
>>> ---
>>> libavfilter/vf_dejudder.c | 66
>>> +
Hi,
On Thu, Aug 20, 2015 at 11:18 AM, Ganesh Ajjanagadde wrote:
> Fixes -Wsometimes-uninitialized from
>
> http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavformat/webmdashenc.c | 6 +++---
> 1 file change
On Thu, Aug 20, 2015 at 6:51 PM, Nicolas George wrote:
> Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit :
>> In ffserver the User-Agent is used to decide how to deal with a
>> client. (WMP being a special case.)
>
> Ok, but that is special case for a specific application.
>
>> That soun
Signed-off-by: Stephan Holljes
---
libavformat/http.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/libavformat/http.c b/libavformat/http.c
index 3fd0496..2c51491 100644
--- a/libavformat/http.c
+++ b/libavformat/http.c
@@ -48,6 +48,7 @@
#define MAX_REDIRECTS 8
#define HTTP_SINGL
On Thu, Aug 20, 2015 at 01:10:43PM -0400, Ganesh Ajjanagadde wrote:
[...]
> I know what s is doing, I just fail to see the connection between the name
> "s" and "filter private context".
s originally stands for struct iirc
--
Clément B.
pgperHJDLxk01.pgp
Description: PGP signature
On Thu, Aug 20, 2015 at 1:53 PM, Clément Bœsch wrote:
> On Thu, Aug 20, 2015 at 01:10:43PM -0400, Ganesh Ajjanagadde wrote:
> [...]
>> I know what s is doing, I just fail to see the connection between the name
>> "s" and "filter private context".
>
> s originally stands for struct iirc
If that is
Actually I just noticed that Pragma-headers may appear multiple times.
What would be a good way to handle that? I thought about expanding the
header tag from "Pragma" to "Pragma-%02d" with either an incrementing
number or just using s->nb_headers. Is there a better solution?
Regards,
Stephan
_
On Thu, Aug 20, 2015 at 02:06:18PM -0400, Ganesh Ajjanagadde wrote:
> On Thu, Aug 20, 2015 at 1:53 PM, Clément Bœsch wrote:
> > On Thu, Aug 20, 2015 at 01:10:43PM -0400, Ganesh Ajjanagadde wrote:
> > [...]
> >> I know what s is doing, I just fail to see the connection between the name
> >> "s" and
On 8/18/15, Ronald S. Bultje wrote:
> ---
> ffserver_config.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/ffserver_config.c b/ffserver_config.c
> index 06bd8ac..de8a454 100644
> --- a/ffserver_config.c
> +++ b/ffserver_config.c
> @@ -230,9 +230,9 @@ static void
On Thu, Aug 20, 2015 at 2:59 PM, Michael Niedermayer
wrote:
> On Thu, Aug 20, 2015 at 02:06:18PM -0400, Ganesh Ajjanagadde wrote:
>> On Thu, Aug 20, 2015 at 1:53 PM, Clément Bœsch wrote:
>> > On Thu, Aug 20, 2015 at 01:10:43PM -0400, Ganesh Ajjanagadde wrote:
>> > [...]
>> >> I know what s is doi
On Thu, Aug 20, 2015 at 01:14:10PM -0400, Ronald S. Bultje wrote:
> Hi,
>
> On Thu, Aug 20, 2015 at 11:18 AM, Ganesh Ajjanagadde > wrote:
>
> > Fixes -Wsometimes-uninitialized from
> >
> > http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
> >
> > Signed-off-
On Thu, Aug 20, 2015 at 09:33:31AM -0400, Ronald S. Bultje wrote:
> Hi,
>
> On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde
> wrote:
>
> > Fixes -Wunused-const-variable from
> >
> > http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
> >
> > Signed-off-by:
On Thursday, August 20, 2015 3:13 PM, Ganesh Ajjanagadde
wrote:
>
>
>On Thu, Aug 20, 2015 at 2:59 PM, Michael Niedermayer
> wrote:
>> On Thu, Aug 20, 2015 at 02:06:18PM -0400, Ganesh Ajjanagadde wrote:
>>> On Thu, Aug 20, 2015 at 1:53 PM, Clément Bœsch wrote:
>>> > On Thu, Aug 20, 2015 at 01:10:
On Fri, Jul 31, 2015 at 04:01:43PM +0300, Ludmila Glinskih wrote:
> Works only with video stream.
> First pass without seeking -- counts crcs of a frames and store it in an
> array.
> After that it seeks a lot in different places and checks if crcs of these
> frames and crcs of frames in array ar
Hi ludmila
did you see my reply on the ML (its also below)? as you didnt reply ...
On Tue, Aug 11, 2015 at 03:31:05AM +0200, Michael Niedermayer wrote:
> On Tue, Jul 28, 2015 at 09:54:06PM +0300, Ludmila Glinskih wrote:
> > Works only for flv, h263 and huffyuv decoders, for video with yuv420p pix
On 20.08.2015 01:33, Michael Niedermayer wrote:
> On Tue, Aug 18, 2015 at 11:47:45PM +0200, Andreas Cadhalpun wrote:
>> graph2dot.c |3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>> 560d5f42580115baf305ba8ff985202b248b4864
>> 0001-graph2dot-use-avfilter_pad_get_name-accessor-funct
On Thu, Aug 20, 2015 at 3:31 PM, Michael Niedermayer
wrote:
> On Thu, Aug 20, 2015 at 09:33:31AM -0400, Ronald S. Bultje wrote:
>> Hi,
>>
>> On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde
>> wrote:
>>
>> > Fixes -Wunused-const-variable from
>> >
>> > http://fate.ffmpeg.org/report.cgi?time=20
On Thu, Aug 20, 2015 at 04:44:17PM -0400, Ganesh Ajjanagadde wrote:
> On Thu, Aug 20, 2015 at 3:31 PM, Michael Niedermayer
> wrote:
> > On Thu, Aug 20, 2015 at 09:33:31AM -0400, Ronald S. Bultje wrote:
> >> Hi,
> >>
> >> On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde
> >>
> >> wrote:
> >>
>
On Thu, Aug 20, 2015 at 6:37 AM, Michael Niedermayer
wrote:
> On Thu, Aug 20, 2015 at 04:34:19AM -0400, Ganesh Ajjanagadde wrote:
>> On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde
>> wrote:
>> > Fixes -Wunused-const-variable from
>> > http://fate.ffmpeg.org/report.cgi?time=20150820031140&slo
On Thu, Aug 20, 2015 at 05:12:40PM -0400, Ganesh Ajjanagadde wrote:
> On Thu, Aug 20, 2015 at 6:37 AM, Michael Niedermayer
> wrote:
> > On Thu, Aug 20, 2015 at 04:34:19AM -0400, Ganesh Ajjanagadde wrote:
> >> On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde
> >> wrote:
> >> > Fixes -Wunused-co
---
configure | 3 +
libavformat/http.c | 194 +
2 files changed, 197 insertions(+)
diff --git a/configure b/configure
index e67ddf6..401e041 100755
--- a/configure
+++ b/configure
@@ -265,6 +265,7 @@ External library support:
--en
Fixes -Wunused-const-variable from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
Signed-off-by: Ganesh Ajjanagadde
---
libavcodec/metasound_data.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libavcodec/metasound_data.c b/libavco
On Thu, Aug 20, 2015 at 5:32 PM, Mariusz Szczepańczyk
wrote:
> ---
> configure | 3 +
> libavformat/http.c | 194
> +
> 2 files changed, 197 insertions(+)
>
> diff --git a/configure b/configure
> index e67ddf6..401e041 100755
> --- a
On Thu, Aug 20, 2015 at 11:38 PM, Ganesh Ajjanagadde
wrote:
> On Thu, Aug 20, 2015 at 5:32 PM, Mariusz Szczepańczyk
> wrote:
> > ---
> > configure | 3 +
> > libavformat/http.c | 194
> +
> > 2 files changed, 197 insertions(+)
> >
>
On Thu, Aug 20, 2015 at 05:34:52PM -0400, Ganesh Ajjanagadde wrote:
> Fixes -Wunused-const-variable from
> http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavcodec/metasound_data.c | 3 ++-
> 1 file changed,
On Thu, Aug 20, 2015 at 5:48 PM, Mariusz Szczepańczyk
wrote:
> On Thu, Aug 20, 2015 at 11:38 PM, Ganesh Ajjanagadde
> wrote:
>
>> On Thu, Aug 20, 2015 at 5:32 PM, Mariusz Szczepańczyk
>> wrote:
>> > ---
>> > configure | 3 +
>> > libavformat/http.c | 194
>> ++
On Thu, Aug 20, 2015 at 02:34:22PM +0100, Sven Dueking wrote:
> From: Sven Dueking
>
> ---
> libavcodec/qsvenc.c | 36 +---
> libavcodec/qsvenc.h | 12
> libavcodec/qsvenc_h264.c | 12
> 3 files changed, 57 insertions(+), 3 dele
On Thu, Aug 20, 2015 at 11:59 PM, Ganesh Ajjanagadde
wrote:
> On Thu, Aug 20, 2015 at 5:48 PM, Mariusz Szczepańczyk
> wrote:
> > On Thu, Aug 20, 2015 at 11:38 PM, Ganesh Ajjanagadde
> > wrote:
> >
> >> On Thu, Aug 20, 2015 at 5:32 PM, Mariusz Szczepańczyk
> >> wrote:
> >> > ---
> >> > configu
Comment was previously slightly incorrect.
Also, it was placed in the wrong location.
Signed-off-by: Ganesh Ajjanagadde
---
libavformat/hls.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavformat/hls.c b/libavformat/hls.c
index caecbde..82dd744 100644
--- a/libavf
Fixes -Wunused-function from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
Signed-off-by: Ganesh Ajjanagadde
---
libavformat/hls.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libavformat/hls.c b/libavformat/hls.c
index 82dd744..
Fixes -Wunused-const-variable from
http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
Signed-off-by: Ganesh Ajjanagadde
---
libavcodec/jpeg2000.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/jpeg2000.c b/libavcodec/jpeg2000.c
On Thu, Aug 20, 2015 at 7:28 PM, Ganesh Ajjanagadde
wrote:
> Fixes -Wunused-const-variable from
> http://fate.ffmpeg.org/report.cgi?time=20150820031140&slot=arm64-darwin-clang-apple-5.1
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> libavcodec/jpeg2000.c | 2 +-
> 1 file changed, 1 insertion(+),
On Thu, Jun 25, 2015 at 12:09 PM, Michael Niedermayer
wrote:
> On Wed, Jun 24, 2015 at 03:25:18AM +0200, Mariusz Szczepańczyk wrote:
> > On Tue, Jun 23, 2015 at 8:34 PM, Michael Niedermayer
> > wrote:
> >
> > > On Mon, Jun 22, 2015 at 12:01:33AM +0200, Mariusz Szczepańczyk wrote:
> > > > ---
> >
Unfortunately, there is no portable format specifier for PID's.
Furthermore, it is not safe to assume pid_t <= 32 bit in size,
see e.g http://unix.derkeiler.com/Mailing-Lists/AIX-L/2010-08/msg8.html.
Right now, it is ok to assume pid_t <= 32 bit in size, but this may change in
the future.
Also
On Thu, Aug 20, 2015 at 6:03 PM Ganesh Ajjanagadde
wrote:
> -avio_printf(pb, "Running as pid %d.\n", stream->pid);
> +avio_printf(pb, "Running as pid %ld.\n", (int64_t)
> stream->pid);
>
You need `"%" PRIi64` as the format specifier.
Timothy
_
On Thu, Aug 20, 2015 at 9:29 PM, Timothy Gu wrote:
> On Thu, Aug 20, 2015 at 6:03 PM Ganesh Ajjanagadde
> wrote:
>>
>> -avio_printf(pb, "Running as pid %d.\n", stream->pid);
>> +avio_printf(pb, "Running as pid %ld.\n", (int64_t)
>> stream->pid);
>
>
> You need `"%" PRIi64`
Unfortunately, there is no portable format specifier for PID's.
Furthermore, it is not safe to assume pid_t <= 32 bit in size,
see e.g http://unix.derkeiler.com/Mailing-Lists/AIX-L/2010-08/msg8.html.
Right now, it is ok to assume pid_t <= 32 bit in size, but this may change in
the future.
Also
On Thu, Aug 20, 2015 at 10:03 PM, Ganesh Ajjanagadde
wrote:
> On Thu, Aug 20, 2015 at 9:29 PM, Timothy Gu wrote:
>> On Thu, Aug 20, 2015 at 6:03 PM Ganesh Ajjanagadde
>> wrote:
>>>
>>> -avio_printf(pb, "Running as pid %d.\n", stream->pid);
>>> +avio_printf(pb, "Running as
C Standard guarantees that { 0 } will do a correct zero initialization.
We use this for consistency across the project.
Signed-off-by: Ganesh Ajjanagadde
---
ffplay.c | 2 +-
ffserver.c| 2 +-
libavcodec/aacdec_template.c | 2 +-
libavcodec/aacpsy.c
On Thu, Aug 20, 2015 at 10:19:51PM -0400, Ganesh Ajjanagadde wrote:
> C Standard guarantees that { 0 } will do a correct zero initialization.
> We use this for consistency across the project.
>
See 392b4b663c450f5522f7c1442da91f7647db6bf0
[...]
--
Clément B.
pgpdV6ih4k4YG.pgp
Description: PG
On Thu, Aug 20, 2015 at 10:17:15PM -0400, Ganesh Ajjanagadde wrote:
> On Thu, Aug 20, 2015 at 10:03 PM, Ganesh Ajjanagadde
> wrote:
> > On Thu, Aug 20, 2015 at 9:29 PM, Timothy Gu wrote:
> >> On Thu, Aug 20, 2015 at 6:03 PM Ganesh Ajjanagadde
> >> wrote:
> >>>
> >>> -avio_printf(pb,
73 matches
Mail list logo
