Re: [BUG] Wrong window aspect with playlist and dual-monitor setup

compn wrote:
> On Mon, 30 Jan 2006 19:52:33 +0100
> Frank Aurich <fa859178 <at> inf.tu-dresden.de> wrote:
> 
>> Hi there,
> 
>> When using MPlayer to play playlists (either with the -playlist option 
>> or just several files called up by mplayer in a command line) though,
>> something goes wrong after the first file has finished playing and the 
>> next file starts. Mplayer creates a new video window with the wrong 
>> aspect ratio, usually the new window is wider (I'd say about double the 
> 
> 
> try -fixed-vo (fixed-vo=1 in config)
> or maybe -screenw 1280 -screenh 1024
> 
> if i set my tv out to the stretched desktop, i get this problem...
> i set my displays to clone to avoid it, i'm not sure how to fix it.
> 
> i think the problem is mplayer checks the display res before setting
> the aspect, so it sees 2560x1024 and changes the aspect to match.
> 
> -compn

No luck :(

Both -fixed-vo and the -screenw/h options have no influence on the 
behaviour.

Frank

[PATCH] vo_directx.c

Yo,

I've been sending Otvos Atilla small win32 fixes for
his dvdnav patches and came across this small issue.
When using vo directx, the main movie always kept
opening new windows, so I dug around and came up with
the attached patch, which i'm also hoping should
squash the bug reported earlier by compn:
http://bugzilla.mplayerhq.hu/show_bug.cgi?id=317

The green colorkey now only flashes without fixed-vo
when playing multiple files (because new windows are
being created).

Please test.

Regards,
Erik

	

	
		
__________________________________________________________ 
Find your next car at http://autos.yahoo.ca

_______________________________________________
MPlayer-cygwin mailing list
MPlayer-cygwin <at> mplayerhq.hu

Re: [PATCH] vo_directx.c

Ermmm scratch this, it messed up fullscreen switching.
I'll submit a new patch. Also, the green colorkey
still flashes when the files switch resolutions.

Regards,
Erik

	

	
		
__________________________________________________________ 
Find your next car at http://autos.yahoo.ca

Re: Fix for further compilation under cygwin

Diego Biurrun (diego <at> biurrun.de) wrote:

>> Compilation finished well after change.
>> I'm not familiar with the procedure.
>> Do you need a patch or something like that?

> Yes, read DOCS/tech/patches.txt.  It's all in the FAQ, BTW.  We also
> need the output of the compilation failure.

vobsub.c:231: error: conflicting types for 'getline'
/usr/include/sys/stdio.h:31: error: previous declaration of 'getline' was here

--

-- 
mfg Thomas

[PATCH] trivial: vobsub.c cygwin fix

Simple patch to avoid the following compile error under cygwin:

vobsub.c:231: error: conflicting types for 'getline'
/usr/include/sys/stdio.h:31: error: previous declaration of 'getline' was here

--

-- 
mfg Thomas

_______________________________________________
MPlayer-cygwin mailing list
MPlayer-cygwin <at> mplayerhq.hu
http://mplayerhq.hu/mailman/listinfo/mplayer-cygwin

Re: [PATCH] trivial: vobsub.c cygwin fix

On Thursday 02 February 2006 12:45, Thomas Plank wrote:
> Simple patch to avoid the following compile error under cygwin:
>
> vobsub.c:231: error: conflicting types for 'getline'
> /usr/include/sys/stdio.h:31: error: previous declaration of 'getline' was
> here

Fixed.

Sascha

Re: Fix for further compilation under cygwin

On Thu, Feb 02, 2006 at 12:42:50PM +0100, Thomas Plank wrote:
> Diego Biurrun (diego <at> biurrun.de) wrote:
> 
> >> Compilation finished well after change.
> >> I'm not familiar with the procedure.
> >> Do you need a patch or something like that?
> 
> > Yes, read DOCS/tech/patches.txt.  It's all in the FAQ, BTW.  We also
> > need the output of the compilation failure.
> 
> vobsub.c:231: error: conflicting types for 'getline'
> /usr/include/sys/stdio.h:31: error: previous declaration of 'getline' was here

And where is the patch?

Diego

Re: [PATCH] trivial: vobsub.c cygwin fix

Thomas Plank wrote:
> Simple patch to avoid the following compile error under cygwin:
> 
> vobsub.c:231: error: conflicting types for 'getline'
> /usr/include/sys/stdio.h:31: error: previous declaration of 'getline' was here
> 
> 
> 

Just as an FYI, this is only going to be required on cygwin 1.5.19 and 
up (current version) - I've not looked at the patch (why do people 
attach text files with binary mime-types?), but I hope it tests to see 
if getline is defined in the system headers (as on linux) before it 
discards however mplayer was previously using it.
getline() was added to stdio.h to cygwin for 1.5.19 - it's caused 
problems with other apps as well, due to them assuming things about 
cygwin which are no longer true as it develops. (See apache1.3 for a 
glaringly bad example of such behaviour).

--

-- 

Spinning complacently in the darkness, covered and blinded by a blanket
of little lives, false security has lulled the madness of this world
into a slumber. Wake up! An eye is upon you, staring straight down and
keenly through, seeing all that you are and everything that you will
never be. Yes, an eye is upon you, an eye ready to blink. So face
forward, with arms wide open and mind reeling. Your future has
arrived... Are you ready to go?

Re: Fix for further compilation under cygwin

On Thu, Feb 02, 2006 at 01:18:37PM +0100, Diego Biurrun wrote:
> On Thu, Feb 02, 2006 at 12:42:50PM +0100, Thomas Plank wrote:
> > Diego Biurrun (diego <at> biurrun.de) wrote:
> > 
> > >> Compilation finished well after change.
> > >> I'm not familiar with the procedure.
> > >> Do you need a patch or something like that?
> > 
> > > Yes, read DOCS/tech/patches.txt.  It's all in the FAQ, BTW.  We also
> > > need the output of the compilation failure.
> > 
> > vobsub.c:231: error: conflicting types for 'getline'
> > /usr/include/sys/stdio.h:31: error: previous declaration of 'getline' was here
> 
> And where is the patch?

Sorry, it was in that other mail...

Diego

Re: [PATCH] trivial: vobsub.c cygwin fix

Chris Taylor (chris <at> equate.dyndns.org) wrote:

> I've not looked at the patch

Function renamed.

> (why do people attach text files with binary mime-types?)

[/main/DOCS/tech/patches.txt]

Send your patch to the mplayer-dev-eng mailing list as a base64-encoded
attachment with the subject line:
'[PATCH] very short description of the patch'.
--

-- 
mfg Thomas