[FFmpeg-devel] [PATCH] Make unsigned some variables in ffserver.c

[Baptiste Coudurier]

Hi,

Stefano Sabatini wrote:
> Hi,
> as in subject, I think it is safer and more consistent.
> 
> BTW is it normal that for the servertest to fail with:
> -fb2d22ac836e84ed8099615f9d1a8b31  ff-test_h.avi
> +18c4ba0e8e7adb781216e38de61c2e39  ff-test_h.avi
>  4f11d850f564af3359951b46ed1e571e  ff-test_l.avi
> -c63cced362722c42942e08315b393a6d  ff-test.swf
> +d976848a9e4d5d8fc2659e4841cdece5  ff-test.swf
>  37ccf79cb3a50fda99173e5ba4b44d55  ff-test_h.asf
> -0cc04a58a9ca7e22902c45836e9908be  ff-test_l.asf
> +3e2bff2fda1fe2ed334e639aa6839f0c  ff-test_l.asf
>  60550d751dfa5172d878f7dd670e3aa4  ff-test_h.rm
>  6218946abd1c4af92f05a8a6208b1e1c  ff-test_l.rm
>  e0dc91430660c619e97b5c82e0f398fc  ff-test.jpg
> -084efa2a3de06517c4f25ebed12cd0a8  ff-test_small.jpg
> +84b9702e34b9e21a84bb29519fc1e3cc  ff-test_small.jpg
>  a22cc793b9f938cc05cd0c56ccf9423c  ff-test.mjpg
> ?
> 

Not really :(

> 
> Index: ffserver.c
> ===================================================================
> --- ffserver.c	(revision 15182)
> +++ ffserver.c	(working copy)
> @@ -294,8 +294,8 @@
>  
>  /* maximum number of simultaneous HTTP connections */
>  static unsigned int nb_max_http_connections = 2000;
> -static int nb_max_connections = 5;
> -static int nb_connections;
> +static unsigned int nb_max_connections = 5;
> +static unsigned nb_connections;
>  

Ok.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Smartjog USA Inc.                                http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA