[MPlayer-dev-eng] [PATCH] Added sanity checks!

Fabian Franz FabianFranz at gmx.de
Sun Jan 12 11:45:29 CET 2003


Hi,

> > also the other parsers should be reviewed and more sanity checks added.
>
> I reviewed them all and they are clean (unless some binary file has some
> specific text + new line in first line ;-))

Hm,. not good enough... see attached patch ...

>
> > > Index: mplayer.c
> >
> > ok the patch looks clean.
> > (as clean as a patch for this mess can be :))
>
> thx :)
>
> So you could now commit ... ;) (attached patch, after reviewing of course
> ...)

I did some more testing and found out that there was indeed one sanity check 
missing ... (in all playlist-functions)

cu

Fabian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer_fallback_to_playlist_sanity_checks.diff
Type: text/x-diff
Size: 2962 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20030112/07adb14f/attachment.diff>


More information about the MPlayer-dev-eng mailing list