[MPlayer-dev-eng] serious warnings and bugs

Arpi arpi at thot.banki.hu
Sun Dec 23 23:14:30 CET 2001


Hi,

Just compiled libmpdemux with -Wall, and got tons of bugs in form of
warnings! just a few examples:

tv.c: In function tream_open_tv':
tv.c:107: warning: eturn' with no value, in function returning non-void
-> such functions should return void, or return should be added/fixed
   such things may cause random crashes (works for few people and doesn't
  work for others - see my http bugreport for example)

asf_streaming.c: In function sf_http_request':
asf_streaming.c:414: warning: too few arguments for format
asf_streaming.c:431: warning: too few arguments for format
-> should be fixed ASAP to avoid crash in printf()

network.c:588: warning: implicit declaration of function sf_streaming_start'
-> it will avoid possible declaration or parameter type incompatibilites to be
detected by gcc :(

rtp.c:178: warning: passing arg 2 of ind' from incompatible pointer type
-> should be checked, and either fixed or typecasting

again: above liens are just examples, there are much more of them over the
whole code. it's time for some review or cleanup. fixing warnings always
solve some misterious bugs.


A'rpi / Astral & ESP-team

--
mailto:arpi at thot.banki.hu
http://esp-team.scene.hu



More information about the MPlayer-dev-eng mailing list