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

Reply via email to