[Mplayer-cvslog] CVS: main/libmpcodecs vd_realvid.c,1.20,1.21

[Arpi]

Hi,

> Hi,
> 
> > Hi!
> > 
> > Maybe you missed the second patch in the same email, to demux_real.
> 
> yes, sorry :(

hmm:

+		    if (sh->format==0x30335652) {
+			((unsigned short*)(sh->bih+1))[4]=4*(unsigned short)stream_read_char(demuxer->stream); //widht
+			((unsigned short*)(sh->bih+1))[5]=4*(unsigned short)stream_read_char(demuxer->stream); //height
+		    } 
+		    else {
+			((unsigned short*)(sh->bih+1))[4]=sh->disp_w; //rv20 needs it
+			((unsigned short*)(sh->bih+1))[5]=sh->disp_h; //
+		    }

so the W/4 and H/4 (hmm, maybe real work with 4x4 macroblocks?) is stored
in/after the codecdata part. some quick tests shown that it's true for RV20
too, and so it fixes the broken RV20 streams in teh WxH dir.
why is that if (sh->format==0x30335652) there, then? :)
and, it seem sthat the above change is far enough to fix these streams, teh
vd_realvid.c hack is not needed. am i right, or are there any so-broken streams
requireing that part too?


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu
    "However, many people beg for its inclusion in Debian. Why?" - Gabucino
  "Because having new software in Debian is good." - Josselin Mouette
"Because having good software in Debian is new." - Gabucino