Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#62 closed defect (wontfix)

MPlayer 1.0pre5 doesnt compile with new

Reported by: a.l.m.buxey@… Owned by: moritz@…
Priority: normal Component: demuxer
Version: 1.0pre5 Severity: major
Keywords: Cc:
Blocked By: Blocking:
Reproduced by developer: Analyzed by developer:


newest (live.2004.08.13.tar.gz) breaks the compilation of
MPlayer. I traced this to the new API of - and a trivial fix to
MPlayer source code stop this.

Change line 410 of libmpdemux/demux_rtp.cpp to read

env->reclaim(); delete scheduler;

alternatively, to keep compatability with older (do you want to??)
you can always do a fancier thing, define at the top of the code

#define RECLAIM_ENV(env) delete (env)
#define RECLAIM_ENV(env) (env)->reclaim()

and then live 415 (its now increased ;-) ) becomes:

RECLAIM_ENV(p_sys->env); delete scheduler;

hope this helps

Change History (1)

comment:1 Changed 16 years ago by r_togni@…

  • Resolution set to wontfix
  • Status changed from new to closed

It's a known problem, something was changed in shortly after pre5 release.
CVS is fixed to compile with latest (cvs won't work with old,
but pre5 fails.
Your fix is correct, but won't help because pre5 is already released and won't
be affected by new commits.

The problem will be fixed in pre6.

For pre5 you have to get an older version of (but their site only
offers the latest version, that's why we didn't bother to make current cvs
compile with old versions) or patch it manually so that it compiles with current

Note: See TracTickets for help on using tickets.