[burriko]

when playing a video with a resolution of 848x480 xbmc will overlap the last 100 pixels or so with the first 100 pixels. i'm guessing that it's any video larger than 720x*.
i've cut a 10 sec sample of the video (it comes to just under 3mb) if any of the developers are interested in looking at it. i can pm you a link to it.

[Hullebulle]

if you can't share the link you can upload it here.

[burriko]

if you can't share the link you can upload it here.

done.

[jmarshall]

excellent!

burriko i need some information:

do you have a hdtv?
what are your ms dash video settings?
what resolution are you using for xbmc?
what screen mode is it using whilst playing the video? (press start on the controller to find out)
do you have pal60 or choose best resolution enabled?

i'll download the sample and have a play - but it is very dependent on what resolution you are using.

cheers,
jonathan

[burriko]

no hdtv. just a normal 4:3 pal tv.
dashboard is set to letterbox and no pal-60.
xbmc is using 720x576, and is in 720x576 mode when playing the video.
i do have 'choose best res.' and 'pal60' enabled (although it makes no difference to this video if i enable or disable them)

[jmarshall]

hi burriko,

i've checked it out and confirmed the problem. as far as i can see, we are setting the video output up correctly in xbox_video.cpp. i am currently looking into how it is copying it from mplayer's buffer onto the screen.

it appears on first look to only do this wrap around thing if the video is wider than the current device's resolution. (for instance anything wider than 720 pixels). i have yet to verify this 100% though.

will let you know how i go in sorting it out.

[jmarshall]

fixed.

if you want to change it, look for:

g_graphicscontext.get3ddevice()->createtexture( m_idevicewidth,

around line 310 in xbox_video.cpp and change it to:

g_graphicscontext.get3ddevice()->createtexture( image_width,

note that the m_ideviceheight parameter should also be changed to image_height, however this will cause problems for the currently buggy subtitle code. better leave the height as m_ideviceheight until the subtitle code is sorted out.

thanks for pointing this bug out.

cheers,
jonathan

[burriko]

thanks for getting to it so fast. i'll give it a try later on today.