April 2006 - MPlayer-dev-eng - lists.mplayerhq.hu

[PATCH] CPU detection related
by Zuxy 17 May '06

17 May '06

Hi, cputable.h.diff contains information update, bug fixes and punctuation modifications to the original cputable.h. I suppose there's no reason to reject it:-) cpudetect.h.diff and cpudetect.c.diff together change the original behavior of mplayer: it first determines whether the CPU has a built-in namestring; if so, then the namestring is displayed and codenames in cputable.h aren't used; if not it'll fall back to the original behavior, looking up the cputable.h. I've explained the reasons for the last two patches in another thread. However, I don't go radical by removing cputable.h at all and depending on built-in namestring only, partly because only P4 or newer Intel CPUs have a namestring (all AMD CPUs supporting cpuid support namestring), and in memory of those who have collected all the odd codenames as well:-) The team are to decide whether to apply only cputable.h.diff, or apply all the three. In the latter case mplayer may contain dead codes and data, those that should never be executed or accessed on a commercially available CPU. -- Zuxy Beauty is truth, While truth is beauty. PGP KeyID: E8555ED6

6 23

RE: [MPlayer-dev-eng] [PATCH] S3 Virge on fbdev
by Mark Sanderson 17 May '06

17 May '06

On Tuesday, 18 April 2006 9:00 a.m., Guillaume POIRIER wrote: > > > Mark, do you plan to stick around and fix any issues that users > > > may > > bring up? > > Yes > > Okay then! Any objection regarding merging that file? None :)

2 2

[PATCH] Various improvement in TOOLS/cpuinfo.c
by Zuxy Meng 17 May '06

17 May '06

Hi, The attached patch 1. fixes inline asm problems to make it produce right answer with icc; hopefully the cpuid/ebx/PIC staff won't matter since it's not used in a library; 2. renames some flags to resemble the behavior of /proc/cpuinfo, most noticeably sse3->pni; 3. adds cyrix_arr support so the mtrr function in vidix might work; I don't have a M1 or M2 for testing, though. 4. enables cpuinfo to print correct names for Pentium III. 5. makes cpuinfo recognize extended families and models. Some testing for 3 is welcome. Cyrix M1 or M2 should have cyrix_arr but no mtrr, and 5x86, MediaGX or GXm should have neither. -- Zuxy Beauty is truth, While truth is beauty. PGP KeyID: E8555ED6

10 34

rfc, uniform labeling of messages?
by Reynaldo H. Verdejo Pinochet 16 May '06

16 May '06

Ok, there is a huge bunch of help messages prefixed in no known form whatsoever but almost all they use some variation of [foobar], [im this module]. etc. Well, even tho this could be a nono from the start, attached patch makes all message have a prefix stating what module they where generated from. just for you to see if it fits your eyes. who knows, maybe we can put some order on the message spitting chaos mplayer is. Regards Reynaldo

5 7

[PATCH] Allow aspect change in TS and DVB
by Carl Eugen Hoyos 09 May '06

09 May '06

Hi! I posted a bug report last week on users that aspect changes in dvb are not supported by mplayer. Following fix allows MPEG2 transport streams to change aspect when using -vc libmpeg2. Tested with the file I uploaded to incoming/aspect.ts and DVB. Hope the patch is correct, Carl Eugen Hoyos Index: libmpdemux/demuxer.h =================================================================== RCS file: /cvsroot/mplayer/main/libmpdemux/demuxer.h,v retrieving revision 1.93 diff -u -r1.93 demuxer.h --- libmpdemux/demuxer.h 24 Apr 2006 10:58:40 -0000 1.93 +++ libmpdemux/demuxer.h 25 Apr 2006 01:45:25 -0000 @@ -319,6 +319,7 @@ char* demux_info_get(demuxer_t *demuxer, char *opt); int demux_info_print(demuxer_t *demuxer); int demux_control(demuxer_t *demuxer, int cmd, void *arg); +void fill_aspect_info (float * aspect); #ifdef HAVE_OGGVORBIS /* Found in demux_ogg.c */ Index: libmpdemux/video.c =================================================================== RCS file: /cvsroot/mplayer/main/libmpdemux/video.c,v retrieving revision 1.62 diff -u -r1.62 video.c --- libmpdemux/video.c 23 Apr 2006 12:29:10 -0000 1.62 +++ libmpdemux/video.c 25 Apr 2006 01:45:26 -0000 @@ -328,32 +328,7 @@ // printf("picture.fps=%d\n",picture.fps); // fill aspect info: - switch(picture.aspect_ratio_information){ - case 2: // PAL/NTSC SVCD/DVD 4:3 - case 8: // PAL VCD 4:3 - case 12: // NTSC VCD 4:3 - sh_video->aspect=4.0/3.0; - break; - case 3: // PAL/NTSC Widescreen SVCD/DVD 16:9 - case 6: // (PAL?)/NTSC Widescreen SVCD 16:9 - sh_video->aspect=16.0/9.0; - break; - case 4: // according to ISO-138182-2 Table 6.3 - sh_video->aspect=2.21; - break; - case 9: // Movie Type ??? / 640x480 - sh_video->aspect=0.0; - break; - default: - mp_msg(MSGT_DECVIDEO,MSGL_ERR,"Detected unknown aspect_ratio_information in mpeg sequence header.\n" - "Please report the aspect value (%i) along with the movie type (VGA,PAL,NTSC," - "SECAM) and the movie resolution (720x576,352x240,480x480,...) to the MPlayer" - " developers, so that we can add support for it!\nAssuming 1:1 aspect for now.\n", - picture.aspect_ratio_information); - case 1: // VGA 1:1 - do not prescale - sh_video->aspect=0.0; - break; - } + fill_aspect_info (&sh_video->aspect); // display info: sh_video->format=picture.mpeg1?0x10000001:0x10000002; // mpeg video sh_video->fps=picture.fps; @@ -387,6 +362,35 @@ return 1; } +void fill_aspect_info (float * aspect){ + switch(picture.aspect_ratio_information){ + case 2: // PAL/NTSC SVCD/DVD 4:3 + case 8: // PAL VCD 4:3 + case 12: // NTSC VCD 4:3 + *aspect=4.0/3.0; + break; + case 3: // PAL/NTSC Widescreen SVCD/DVD 16:9 + case 6: // (PAL?)/NTSC Widescreen SVCD 16:9 + *aspect=16.0/9.0; + break; + case 4: // according to ISO-138182-2 Table 6.3 + *aspect=2.21; + break; + case 9: // Movie Type ??? / 640x480 + *aspect=0.0; + break; + default: + mp_msg(MSGT_DECVIDEO,MSGL_ERR,"Detected unknown aspect_ratio_information in mpeg sequence header.\n" + "Please report the aspect value (%i) along with the movie type (VGA,PAL,NTSC," + "SECAM) and the movie resolution (720x576,352x240,480x480,...) to the MPlayer" + " developers, so that we can add support for it!\nAssuming 1:1 aspect for now.\n", + picture.aspect_ratio_information); + case 1: // VGA 1:1 - do not prescale + *aspect=0.0; + break; + } +} + void ty_processuserdata( unsigned char* buf, int len ); static void process_userdata(unsigned char* buf,int len){ Index: libmpcodecs/vd.c =================================================================== RCS file: /cvsroot/mplayer/main/libmpcodecs/vd.c,v retrieving revision 1.84 diff -u -r1.84 vd.c --- libmpcodecs/vd.c 24 Mar 2006 08:12:02 -0000 1.84 +++ libmpcodecs/vd.c 25 Apr 2006 01:45:26 -0000 @@ -289,6 +289,8 @@ } if(sh->aspect>0.01){ int w; + if (sh->ds->demuxer->file_format == DEMUXER_TYPE_MPEG_TS && sh->format == 0x10000002) + fill_aspect_info(&sh->aspect); mp_msg(MSGT_CPLAYER,MSGL_INFO,MSGTR_MovieAspectIsSet,sh->aspect); w=(int)((float)screen_size_y*sh->aspect); w+=w%2; // round // we don't like horizontal downscale || user forced width:

5 19

helix codecs
by Enrico Weigelt 09 May '06

09 May '06

Hi folks, is there any work on including helix codecs in progress ? cu -- "*Who am I*? I am Susan Ivanova, Commander, daughter of Andrei and Sophie Ivanova. I am the right hand of vengeance, and the boot that is going to kick your sorry ass all the way back to Earth... I am Death Incarnate, and the last living thing that you are ever going to see. God sent me." -- Cmdr. Ivanova

6 17

[BUG] libavcodec issue with bframes
by Giacomo Comes 09 May '06

09 May '06

It is already known to some developers, but probably not to everybody that current CVS does not encode properly using mencoder/libavcodec and bframes. Here is how to check the bug. Make 100 jpeg frames to encode: # the following bash line will create 100 jpegs each one with the frame number # included in the image for ((a=1;a<101;a++)) ; do convert -size 100x75 xc:green -pointsize 24 -fill 'rgba(255,255,255,128)' -draw "gravity SouthEast text 12,42 '$a'" $(printf "%05d" $a).jpg ; done # the following command line will make a (dvd) mpeg that has all the frames # correctly encoded. mencoder -of mpeg -mpegopts vaspect=4/3:format=dvd -ovc lavc -lavcopts vcodec=mpeg2video:vrc_buf_size=1835:keyint=15:aspect=4/3:vrc_maxrate=7500:vbitrate=7500:vmax_b_frames=0 -o test.mpg mf://*.jpg -fps 25.000 -ofps 25 -vf scale=720:576,harddup -oac copy # the following command line will make a (dvd) mpeg where the frames 2 and 3 are # not encoded. mencoder -of mpeg -mpegopts vaspect=4/3:format=dvd -ovc lavc -lavcopts vcodec=mpeg2video:vrc_buf_size=1835:keyint=15:aspect=4/3:vrc_maxrate=7500:vbitrate=7500:vmax_b_frames=1 -o test.mpg mf://*.jpg -fps 25.000 -ofps 25 -vf scale=720:576,harddup -oac copy # Also adding the option -noskip to the last command line will make it to fail # to create the mpeg with many error messages: [mpeg2video @ 0x87c6e48]Error, Invalid timestamp=0, last=0:0.000 [0:0] # The problem exists also with avi. # next command makes an avi with all the frames: mencoder -ovc lavc -lavcopts vcodec=mpeg4 -o test.avi mf://*.jpg -vf scale=800:600 -fps 25.000 -ofps 25 -oac copy # next one has frames 2, 3 and 4 missing: mencoder -ovc lavc -lavcopts vcodec=mpeg4:vmax_b_frames=1 -o test.avi mf://*.jpg -vf scale=800:600 -fps 25.000 -ofps 25 -oac copy # also adding the option -noskip the previous command fails with errors: [mpeg4 @ 0x87c6e48]Error, Invalid timestamp=0, last=0 A-V:0.000 [0:0] And last: the avi created with current CVS with bframes cannot be played with mplayer 1.0pre7 (avi without bframes can be played). These problems have been recently introduced. I have tested CVS of 7th february and it doesn't shows the missing frames problem. Giacomo P.S. You can check the missing frames in two ways: start mplayer and pause it immediatly and then press the Step Forward key (.) or dump the frames with: mplayer -vo png test.avi

8 15

Last call for pre8
by Roberto Togni 09 May '06

09 May '06

Hi all This is my last attempt at releasing pre8 before Linuxtag. Changes from previous report: * old release critical bugs: - 3gp with amr audio broken: fixed * new release critical bugs: - b-frames + lavc in mencoder are reported to be broken * other stuff (from the old pre8 thread): - the utf8 stuff: solved, delayed or critical? - vf_ass: commit or wait? - the win32 gui? solved or delayed? iirc the author said to wait for pre9 - anything else? * Target: weekend * Open tasks (help needed): - check and complete changelog, including ffmpeg part (else release as is), or tell me that it's up to date - write a nice news entry (else I'll happily provide a nasty one) - security patch for pre7 (and maybe an advisor?) use the gentoo patches? forget about it? what about cvs? Commit safe pending patches now, don't break cvs, and come visiting us at Linuxtag next wek. Ciao, Roberto

14 30

[PATCH] dvdnav
by Ötvös Attila 08 May '06

08 May '06

Hi All. I am created a patch mplayer and gmplayer to using dvd menu. Implemented: - still frame menu (partially) - spu menu buttons (partially) - can be used mouse with menu selection (partially) - simple light box menu button (new OSDTYPE) - time length of vts domain stream - can be selection the subtitle language with "vobsub_lang" command ('j') - can be dvd menu trought set audio and subtitle languages - new menu item in gmplayer (in gtk menu) - can be set dvd menu languages The patch is big (185 kbyte), hence donwload this about link: http://members.chello.hu/dc/mplayer/ (I am sorry to say that my English knowledge isn't good.) Best regards! Attila

16 106

[PATCH] Add support for GNOME screensaver
by Piotr Kaczuba 06 May '06

06 May '06

Hello! I've wrote a patch to support GNOME screensaver in MPlayer, i.e. to disable it when playing a movie and reenabling it after that. The code requires that dbus and dbus-glib is present, as it uses GNOME screensaver's DBUS API for the task. It's integrated into x11_common.c and so it's only compiled when X11 is present. Please comment on it, especially on the changes in configure. Maybe they can be improved. Piotr

14 33