[MPlayer-dev-eng] [PATCH] DJGPP support

Fabrizio Gennari fabrizio.ge at tiscali.it
Sun Mar 17 15:39:02 CET 2013


Thank you Reimar for the time you spent reviewing the patch.

The porting of MPlayer to MS-DOS was made by Khusraw, who made available 
the whole changed source (not a patch): now it can be found at 
http://omploader.org/vY3A4Mg . I only found out the SVN revision that he 
used as base (34661 appeared to fit the bill), adapted it to latest SVN, 
fixed the style here and there (but not thoroughly) and verified that it 
worked on my configuration (which can be different from his). He's the 
one who deserves credit for this (and to blame for the style 
inconsistency), and he's the one who can answer your comment.
> why #ifndef everywhere instead of the simpler #ifdef and swapping
> the if/else parts?
About
> replacing the memalign by e.g.
> av_malloc where easily possible (will also require replacing the
> corresponding free() ) might be alternative options.
I agree that replacing memalign() with av_malloc() is the cleanest 
solution, but it has an impact on all code, not just DJGPP, so better 
let experts do it. And about
> >/  +#include "libavutil/libm.h"
> /
> What is this needed for?
> If only DJGPP needs it for some reason you should add a comment,
> otherwise someone might remove it "because it works for me without".
/libavutil/libm.h/ offers replacements to some mathematical functions if 
they are not present. If this is omitted, the linker fails with 
"undefined reference to '_round'"

Here is a patch that addresses all your other comments. It also fixes a 
problem occurring when using latest FFmpeg from Git: it needs 
CONFIG_ICONV to be always defined, as 0 or 1 appropriately, because it 
just won't compile with CONFIG_ICONV undefined.

2 latest notes:
* video support requires a version of libvbe different from the one in 
vesautils. It is in the aforementioned link http://omploader.org/vY3A4Mg 
. I am attaching it here for completeness.
* for some reasons, MP3 decoding crashed consistently until 
--disable-sse --disable-sse2 --disable-sse3 was added to configure's 
parameters, despite having a modern CPU (AMD Athlon(tm) II X4 640 Processor)

Regards,
Fabrizio
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mpl.patch
Type: text/x-patch
Size: 44995 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20130317/8e7e5b18/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vbe.zip
Type: application/zip
Size: 14848 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20130317/8e7e5b18/attachment-0001.zip>


More information about the MPlayer-dev-eng mailing list