[MPlayer-users] Most recent updates have caused a problem

[Reimar Döffinger]

On Thu, Jul 15, 2010 at 07:20:51PM +1000, Murray Blakeman wrote:
> On 15/07/10 05:59 AM, Reimar Döffinger wrote:
> >
> >Hrmpf. I supect a case of Solaris being backwards-compatible to the
> >point of idiocy, i.e. implementing signal() in a way that makes a
> >race-condition impossible to avoid.
> >In that case MPlayer will have other problems, but they are probably
> >too rare for you to notice.
> >Try attached patch please.
> >
> That patch looks like it's fixed the problem with the cache. :-)

Applied.

> I've now got to sort out the issue with the video jumping every few seconds.
> 
> I noticed that when it happens the "A-V:  0.018 ct: -0.016" numbers
> rise and then settles.

Sure it's not the other way round? Probably something block MPlayer
in some way for a short time.

> The "#include <unistd.h>" must also include /usr/include/sys/types.h
> where I believe the declaration of caddr_t resides.

Fixes are welcome, but the main problem is your system is completely broken.
First, headers are supposed to include everything they need.
And unistd.h definitely should _not_ end up defining a caddr_t type,
that is just broken.

> If I use this code in a recent "configure" the build breaks during
> compilation which isn't surprising I guess.

Actually it is, since only that configure check was changed recently,
the "real" code was not.