[MPlayer-dev-eng] MNG support for MPlayer

Stefan Schuermans stefan at blinkenarea.org
Fri Sep 19 15:39:20 CEST 2008


Diego Biurrun wrote:
> On Thu, Sep 18, 2008 at 10:47:00PM +0200, Stefan Schuermans wrote:
>> --- configure	(revision 27638)
>> +++ configure	(working copy)
>> @@ -4683,6 +4687,33 @@
>>  
>> +echocheck "MNG support"
>> +if test "$_mng" = auto ; then
>> +  _mng=no
>> +  cat > $TMPC << EOF
>> +#include <libmng.h>
>> +int main(void) {
>> +  const char * p_ver = mng_version_text() ?: "";
>> +  printf("libmng: %s\n", p_ver);
>> +  return p_ver[0] == 0;
>> +}
>> +EOF
>> +  if cc_check -lmng -lz $_ld_lm ; then
>> +    if tmp_run ; then
>> +      _mng=yes
>> +    else
>> +      _res_comment="no library version"
>> +    fi
>> +  fi
>> +fi
> 
> I wonder why tmp_run is necessary?  And yes, I know that the libpng test
> does it as well...

If I've understood this correctly, cc_check tries to compile and link 
the C source and tmp_run tries to run the result. I think it is 
neccessary to run the code to detect if linking against libmng works. 
I've seen it at least once (with some broken libmng installation) that 
compiling and linking worked, but the binary did not run. I'm not sure 
any more if it crashed or did not load due to not finding libmng.so.

> Your code is laboriously commented.  Possibly some of it could be
> doxygenized.

I did that and hope the style of the the doxygen comments is okay.

Greetings,
Stefan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: MPlayer-svn20080919-mng.diff
Type: text/x-patch
Size: 25360 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080919/77d19cc6/attachment.bin>


More information about the MPlayer-dev-eng mailing list