[MPlayer-dev-eng] Fwd: [MPlayer-users] a fix for the trouble compiling mplayer with new 7-16 live.com [adland123 at yahoo.com]

[Roberto Togni]

Forwarded from user list

Please comment and commit

Ciao,
 Roberto


On 2004.07.18 08:55, adland wrote:
just a quick test after looking at the new 7-16 code for live.com

error found was
/usr/local/live/UsageEnvironment/include/UsageEnvironment.hh: In
function `void
    demux_close_rtp(demuxer_t*)':
/usr/local/live/UsageEnvironment/include/UsageEnvironment.hh:79: error:
`virtual UsageEnvironment::~UsageEnvironment()' is protected
demux_rtp.cpp:410: error: within this context

which means destructor is not public anymore
if we look at line 79 of
/usr/local/live/UsageEnvironment/include/UsageEnvironment.hh
it is
virtual ~UsageEnvironment(); // we are deleted only by reclaim()
so we are told to now use the new public reclaim() instead

below is a patch which is not backwards compatible to older live.com
versions
and seems to work with newer version.

of course it should be tested.

--- main/libmpdemux/demux_rtp.cpp       2004-06-03 14:42:00.000000000
-0400
+++ updated/libmpdemux/demux_rtp.cpp    2004-07-18 02:26:52.000000000
-0400
@@ -407,7 +407,8 @@
   delete rtpState->sdpDescription;
   delete rtpState;

-  delete env; delete scheduler;
+  env->reclaim();
+  delete scheduler;
 }


_______________________________________________
MPlayer-users mailing list
MPlayer-users at mplayerhq.hu
http://mplayerhq.hu/mailman/listinfo/mplayer-users