[FFmpeg-devel] [PATCH] ffserver: cast PID to int64_t before printing

Ganesh Ajjanagadde gajjanag at mit.edu
Fri Aug 21 15:20:50 CEST 2015


On Fri, Aug 21, 2015 at 2:06 AM, Clément Bœsch <u at pkh.me> wrote:
> On Thu, Aug 20, 2015 at 10:17:15PM -0400, Ganesh Ajjanagadde wrote:
>> On Thu, Aug 20, 2015 at 10:03 PM, Ganesh Ajjanagadde
>> <gajjanagadde at gmail.com> wrote:
>> > On Thu, Aug 20, 2015 at 9:29 PM, Timothy Gu <timothygu99 at gmail.com> wrote:
>> >> On Thu, Aug 20, 2015 at 6:03 PM Ganesh Ajjanagadde <gajjanagadde at gmail.com>
>> >> 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.
>> >
>> > You are right. Thanks, will post updated patch.
>>
>> So I used PRId64 instead of PRIi64.
>> Is there an issue with that?
>>
>
> I don't think so

You are right, see e.g
https://stackoverflow.com/questions/1893490/difference-between-format-specifiers-i-and-d-in-printf
and relevant standards quote therein.

>
> --
> Clément B.
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list