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

Michael Niedermayer michael at niedermayer.cc
Fri Aug 21 17:26:30 CEST 2015


On Thu, Aug 20, 2015 at 10:12:40PM -0400, Ganesh Ajjanagadde 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/msg00008.html.
> Right now, it is ok to assume pid_t <= 32 bit in size, but this may change in the future.
> Also, this patch fixes warning due to lack of an appropriate cast from
> http://fate.ffmpeg.org/report.cgi?time=20150820233505&slot=x86-opensolaris-gcc4.3.
> Note that this method of handling pid_t is in line with what nginx does.
> 
> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> ---
>  ffserver.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)

applied

thanks


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150821/4d3b827f/attachment.sig>


More information about the ffmpeg-devel mailing list