On Fri, Aug 21, 2015 at 2:06 AM, Clément Bœsch <u...@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
>> <gajjanaga...@gmail.com> wrote:
>> > On Thu, Aug 20, 2015 at 9:29 PM, Timothy Gu <timothyg...@gmail.com> wrote:
>> >> On Thu, Aug 20, 2015 at 6:03 PM Ganesh Ajjanagadde 
>> >> <gajjanaga...@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@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to