MickJT-MPlayer

1.0.1 released on OS4Depot:

* Disabled experimental AHI code that was freezing some machines
* Suppressed an error preventing some .avis playing with vo_comp_yuv/yuv2
* Fixed some path issues preventing OSD menus from working
* P96_PIP: Fixed issue causing some videos to play distorted
* P96_PIP: Remember window position when returning from fullscreen
* Added "Stay on Top" functionality for all video outputs
* Change keyboard shortcut for "Load File". Conflicted with "Loop".
* Cosmetic tweaks to the About window.
* Disabled some debug output that was printing to shell


--- Original post below ---

Hello,

It's just turned Christmas day here in my timezone and I've uploaded a new MPlayer build to OS4Depot. Naturally, there's already been a bug report Seeing as I'm going to be unavailable most of the day, if any of you with altivec capable machines are experiencing lockups or crashes, please try this binary and report back here, and I will check on it as soon as I have time. Keep in mind my programming knowledge is extremely limited for C and I probably can't fulfill feature requests.

(Links removed)

@MickJT

Thank you for all your hard work on behaft of the Amiga community, and a very merry Christmas to you!

Big thank you and merry merry Christmas to you! 🤗

Nice work!

I'm getting the following output and a black window with your updated version above:

9> mplayer "dh12:tv/Doctor Who/04 - Tom Baker/1977-15-092-1-4 Horror Of Fang Rock.avi"
Cannot find HOME directory.
MickJT-MPlayer-1.0 1.3.0 (C) 2000-2016 MPlayer Team

Playing dh12:tv/Doctor Who/04 - Tom Baker/1977-15-092-1-4 Horror Of Fang Rock.avi.
libavformat version 57.25.100 (internal)
AVI file format detected.
[aviheader] Video stream found, -vid 0
[aviheader] Audio stream found, -aid 1
[aviheader] Audio stream found, -aid 2
VIDEO: [DX50] 640x480 24bpp 25.000 fps 1687.0 kbps (205.9 kbyte/s)
VO: [comp_yuv2] Welcome man !.
gfx_nodri = 0
gfx_novsync = 0
benchmark = 0
vsync is enabled = 1
Opening video filter: [screenshot]
==========================================================================
Opening video decoder: [ffmpeg] FFmpeg's libavcodec codec family
libavcodec version 57.24.102 (internal)
Selected video codec: [ffodivx] vfm: ffmpeg (FFmpeg MPEG-4)
==========================================================================
Load subtitles in dh12:tv/Doctor Who/04 - Tom Baker/
==========================================================================
Opening audio decoder: [mpg123] MPEG 1.0/2.0/2.5 layers I, II, III
AUDIO: 44100 Hz, 2 ch, s16be, 192.0 kbit/13.61% (ratio: 24000->176400)
Selected audio codec: [mpg123] afm: mpg123 (MPEG 1.0/2.0/2.5 layers I, II, III)
==========================================================================
AO: [ahi_dev] 44100Hz 2ch s16be (2 bytes per sample)
Starting playback...
Movie-Aspect is undefined - no prescaling applied.
[swscaler @ 0x50c9b088]bicubic scaler, from yuv420p to rgb24 using AltiVec
[swscaler @ 0x50c9b088]ALTIVEC: Color Space RGB24
[swscaler @ 0x50c9b088]using unscaled yuv420p -> rgb24 special converter
VO: [comp_yuv2] 640x480 => 640x480 Planar YV12
screen w 1920 h 1080 aspect 1.78:1
VO: Disabling blanker
from 3d5c9000 to 3d641000
from 3d641000 to 3d67d000
from 3d641200 to 3d67d200
VO: [comp_yuv2] allocated DR buffer 0
from 3ae07000 to 3ae7f000
from 3ae7f000 to 3aebb000
from 3ae7f200 to 3aebb200
VO: [comp_yuv2] allocated DR buffer 1
Movie-Aspect is 1.33:1 - prescaling to correct movie aspect.
[swscaler @ 0x50c9b088]ALTIVEC: Color Space RGB24
VO: [comp_yuv2] 640x480 => 640x480 Planar YV12
Select error: No such file or directory
AmigaOS4.1 Bug select() don't reset or set errno value
[VD_FFMPEG] DRI failure.

then nothing but the next two lines repeating:

[mpeg4 @ 0x50b7aa10]get_buffer() failed (stride changed)
Error while decoding frame!

Tested on my Sam440 .. PIP seems working but i have spare output when i close or resize the video:

Have My_Screen

Will test it further, great to see an update

@samo79

By spare output, you just mean the "Have My_Screen" printed to shell? I think would be present in LiveForIt-MPlayer too, right? It's in cgx_common.c. I could remove that line. Is it causing any issues?

@all

If you have an altivec capable machine, please try some .mp4 files using the version on OS4Depot. I would like to know if the altivec build on OS4Depot crashes sporadically and if the build in post #1 fixes it.

@MickJT

I've been testing many MP4 videos with the OS4Depot release and I got no crash.

@Sam79

Yes, "Have my Screen" was already there with LiveForIt-Mplayer, it appears when you resize the window while playing a video.

Thanks!!

The Non Altivec version works great on my X5000

Merry Xmas

@MickJT

Tested a few mp4 and have so far no problems with both versions.
X1000 with OS4.1FE.

@MickJT

With upload 2 I can't play the big buck bunny avi file. It goes into an endless something about a bug.

@Srtest

I'll need something a little more specific than that :) Does it work with the one on OS4Depot? Usually that would be the one with the bug. If you have any shell output that'd be appreciated. I'll check back later, in the meantime, zzZZzzzz.

P.S Try with -vo comp or -vo sdl. Still, shell output if you can grab it. Might be useful to delete/rename the config file or remove quiet=1 from it.

@MickJT

Quote:

No, just it's annoying and useless, need to be removed

Still an output (without any text) also when you press "P" or "spacebar" for pause .. still to remove this aswell

Another little issue ... still various but try to collect them step by step

The screenshot option from menu always generate an image called "mplayer.png" instead to pick the real name of the video (as it happen if you just press "s" to pick a grab of the video)

Should be easy to fix

The functionality "Stay on top" (Video options) does not work ...
I tested the old MPlayer from afxgroup and it works on that version .. probably the functionality get broken by some modification of LiveForIt ...

New one, again a minor thing but still ...

Tested MickJT version with MPlayer GUI and noted a strange typo in title window (GUI)

Here it is:

"MickJT-MPlayer-1.0 1.3.0 (C) 2000-2016 MPlayer Team000-2010 MPlayerTeamTeam"

But should be:

"MickJT-MPlayer-1.0 1.3.0 (C) 2000-2016 MPlayer Team""

By the way, what was the sense to named this version as a personal "MickJT" version instead of just using the original scheme of LiveForIt including his old readme and so on ?

I understood you started from his latest version right ?
So just add your name as contributor in about window and to the readme .. less confusion no ?

And to say more, it was even strange to have a version called "LiveForIt" instead of just call it MPlayer as to end that version was partially based to the initial work of afxgroup ... not to mention we have also a MUI version ported by kas1e !!

So please just call it "MPlayer" without all that blablabla so we can replace the old LiveForIt and MPlayer versions marking them as obsolete --> 1 version = 0 confusion

@samo79

Just so know who worked on it.

for example I'm not responsible for older versions, that is distributed under name "afxgroup". I made lots of changes and take provide in improvements I made.

Also I'm not person to contact if something is not working in MUI MPlayer, or any other MPlayer build.

I think "MickJT" has always called his build MickJT, that is perfectly fine with me.

Version is just a number, it does not matter, as long know this older newer then older one.

@LiveForIt

Yep, however with 1 single common repository and all info in documentation/about windows everyone will know who contributed and in what way they did ...

Ideally we could have only 2 versions, the MUI one from kas1e can't be merged for obvious reasons but your version, the ancient one from afxgroup (and this latest modifications from MickJT) could be merged in a unique release .. once completed of course as still there some issues here and there ...

Quote:

Well i don't agree, it's not just a name ... because if now, for example Mick will solve an issue and he will add anything, then that things should be availible also in your version .. and considering that he started from your version it's an absolute nonsense to have 2 separate versions ... same story for the old version of Andrea .. pratically all his previous work was merged in your version (or improved/fixed) so what is the sense to have his version around ?

@MickJT

Select error: Illegal seek
AmigaOS4.1 Bug select() don't reset or set errno value
Cache empty, consider increasing -cache and/or -cache-min. [performance issue]
Seek failed
Select error: Illegal seek
AmigaOS4.1 Bug select() don't reset or set errno value
Cache empty, consider increasing -cache and/or -cache-min. [performance issue]
Seek failed

My cache settings work with other mp4 or mpg files. These settings are also nothing out of the ordinary.

@Srtest

AmigaOS4.1 Bug select() don't reset or set errno value

Yes that error was put in there so people know, how SDK should work can have look at it...

Cache empty, consider increasing -cache and/or -cache-min. [performance issue]

Read the mplayer docs or google it, your settings are wrong.