MPEG-PES aka DVB resolution decrease not honored

[bugs@…]

If the resolution decreases from say 720x576 -> 704x576 during playback, the
picture becomes garbled (skewed), however an increase from 704x576 -> 720x576
works.

I noticed that the relevant variables are updated only when the resolution
increases. I fixed it to always update the variables by commenting out the
"offending" if statement. As a result, resolution changes work both ways for me
at least with the following commands:

mencoder -ovc lavc -lavcopts vcodec=mpeg4 -scale 512:384
mencoder -ovc lavc -lavcopts vcodec=mpeg4
mplayer -vo x11
mplayer -vo xv

I'll add the patch that works for me. It patches against both 1.0pre7try2 and
CVS.

The fix also revealed another "bug" in the vd component, I'll post a separate
bug for that one.

[I marked this as an enhancement since Bug 190 states mid-stream resolution
changes are not officially supported.]

[bugs@…]

Fix along with a comment describing a slightly more optimal solution.

[bugs@…]

The following block (in my proposed patch)

+ if test "$lang" = en ; then
+ test "$got_en_lang" && continue
+ got_en_lang=1
+ fi

was added to avoid having "en" twice in the DOC_LANG variable (which usually happens unless --language=all was specified). HTML generation takes a lot of time, so I felt avoiding duplicates was motivated.

[bugs@…]

(In reply to comment #2)

The following block (in my proposed patch)

Apologizes. I pasted the comment to the wrong bug. Please ignore.