[yadster]

Comparing changelog in t3ch build with http://trac.xbmc.org/ these are the changes that have taken place between the last known working scrolling version of xbmc4xbox (XBMC-SVN_2009-09-27_rev23183-T3CH) and the first known not working version of xbmc4xbox (XBMC-SVN_2009-11-08_rev24421-T3CH). This is the best I can do without been able to test further builds.

2009-11-08 24421 [XBOX] changed: Prepare for 9.11-alpha tag
2009-11-06 24337 [XBOX] merged: Trunk changes to Platinum/Neptune path
2009-11-06 24336 [XBOX] merged; Trunk revisions 21463, 22058, 23259, 23599
2009-11-06 24332 [XBOX] merged: Trunk revisions 21849, 22058, 23259, 23574, 23597, 23598, 23599, 23645 (UPnP)
2009-11-04 24277 [XBOX] merged: Trunk revisions 23677,24088,24156,24157,24170,24172,24177,24178,24180,24197,24224
2009-11-04 24276 [XBOX] merged: Linuxport revisions 21303,24098,24123,24124,24125 + revert merge of 23607,23608 + general (setting) sync with mainline
2009-11-03 24238 [XBOX] merged: Linuxport revisions 21939 & 23873
2009-11-02 24191 [XBOX] fixed: Ticket #7523 - Textbox not rendered in lists
2009-10-31 24155 [XBOX] merged: Linuxport revisions 23813,24010,24015,24017,24025,24043,24051,24059,24060,24065,24077,24078,24082 24085,24089,24091,24095-24098,24102,24105,24106,24107,24109-24111,24113,24115 24128,24130,24142,24144-24147,24149
2009-10-31 24148 [XBOX] added: FFmpeg sources based on Linuxport r21898 [XBOX] changed: Modified FFmpeg build script so the generated DLLs are optimized for Xbox (Pentium 3 arch) [XBOX] updated: New optimized DVDPlayer DLLs. They provide a significant performance increase :-)
2009-10-30 24143 [XBOX] changed: Set sane values for the DVDPlayer's MaxDataSize on Xbox since the amount of RAM we have is limited
2009-10-30 24134 [XBOX] fixed: Ticket #7505 - Videos not being marked as Watched after watching
2009-10-30 24133 [XBOX] merged: XBPython in sync with Linuxport
2009-10-30 24132 [XBOX] changed: MPlayer tweaks for the new EDL implementation
2009-10-30 24131 [XBOX] fixed: We no longer have the soundsduringplayback-setting so don't check for it
2009-10-29 24122 [XBOX] merged: Linuxport revisions 21561-21569,21571,21574-21583,21591-21593,21595-21614,21616-21636,21639,21640 21644,22209-22211,22244-22248,22251,22252,22987,23346,23765,23781,23792,23793 This mainly brings the EDL & CMyth updates from mainline to Xbox
2009-10-26 24048 [XBOX] merged: Linuxport revisions (mainly GUIlib Updates) 22974,23069,23126,23543,23811,23844,23851,23870,23871,23956,24007
2009-10-26 24036 [XBOX] fixed: Forgot to commit one file
2009-10-25 24020 [XBOX] merged: Linuxport revisions 23060,23065,23081,23082,23236,23240,23279,23492,23493,23607-23611,23613 23615-23618,23620,23621,23624,23626,23634,23636,23638-23641,23643,23644,23648 23651,23653,23654,23655,23663,23664,23665,23671,23679,23693-23696 23697-23700,23703,23705,23728-23730,23733,23735-23737,23741-23745,23746,23752 23753,23759-23764,23766-23768,23775,23780,23783,23784,23788-23789,23794,23808 23815,23833,23843,23853,23855,23861,23862,23869,23891,23895,23917,23920,23937 23943,23945,23955,23957-23959,23960,23962,23963,23984-23987,23994,23995,23996 24010,24015,24017 This merge will even have it's own codename: "Merge from hell" :/
2009-10-25 24009 [XBOX] fixed: Ticket #7452 - MPlayer Not playing some flv files. We should simply force DVDPlayer for .flv-files as this probably will never be fixed in our MPlayer
2009-10-24 23975 [XBOX] fixed: Autoexec.py would get executed twice since both special://xbmc & special://home point to the same location
2009-10-24 23965 [XBOX] merged: Linuxport revisions 23110,23161,23167,23180,23185,23189,23192,23198,23202,23205,23206,23212 23234,23254,23255,23257,23259,23269,23272,23273,23284,23307,23315,23339-23342 23353,23359,23360,23378,23379,23381-23384,23387,23401,23402,23408,23409,23413 23419,23422,23431,23435,23436,23478,23489,23500,23521,23534,23537,23539,23542 23548,23552-23557,23562,23580,23590,23594,23604
2009-10-24 23964 [XBOX] changed: Force .ogv files to be played with DVDPlayer
2009-10-23 23932 [XBOX] changed: Don't spam our log with this
2009-10-23 23921 [XBOX] added: Ticket #7420 - CUT Thumbnail Caching. Thanks to Ez0n3 & Bomb Bloke for the patch
2009-10-21 23887 [XBOX] fixed: Aargh! Changeset #23358 missed a bit
2009-10-20 23865 fixed: Added missing include to DVDPlayerSubtitle.cpp
2009-10-05 23445 [XBOX] revert: added: Created http-local, https-local, ftp-local, ftps-local "aliased" filesystems/protocols. It's too much of a hack and it's better to implement it via protocol-options (but you'll have to know about their existance ofc -> thanks spiff ;-)).
2009-10-05 23439 [XBOX] changed: Thread count is now set from as.xml so we should properly set sane Xbox values for it.
2009-10-05 23438 [XBOX] added: Created http-local, https-local, ftp-local, ftps-local "aliased" filesystems/protocols. They work the same as their normal counterparts except that XBMC treats them as ordinary filesystems without any restrictions. This allows us to use fanart and such with ftp & http filesystems. Will backport this later on to Linuxport
2009-10-05 23434 [XBOX] fixed: Ticket #7280 - Daylight Saving Time bug in GMT+10 AEDT timezone. Thanks to AnthonyB!
2009-10-03 23358 [XBOX] fixed: DHCP wouldn't recover when ie. using wireless bridges (worked around a bug (probably) in XDK or the Xbox's BIOS). changed: Further cleanup of CNetwork. Code should be a lot more consistent now changed: Don't have our RssReader wait for the network. Just fail and let it retry later on.
2009-09-30 23274 [XBOX] fixed: Network wouldn't recover in some cases at startup (hopefully the last fix considering network)

[eddebaby]

I've done a little bit of investigating into this. Looking at the changes on trac for GUITextBox.cpp and using the revision numbers given by yadster (Last Known Working revision is 23183, first Confirmed Not Working revision 24421) it seems the bug appeared after revision 23095 and before revision 24613 (the latest). Luckily that's only 4 changes.

Code:

[b]Chgset  Date      Author       Log Message[/b]
[24613] 6 months  arnova       [XBOX] merged: trunk/mainline revisions …
[24238] 6 months  arnova       [XBOX] merged: Linuxport revisions 21939 & 23873
[24191] 6 months  arnova       [XBOX] fixed: Ticket #7523 - Textbox not rendered in lists
[24020] 7 months  arnova       [XBOX] merged: Linuxport revisions …
[23095] 8 months  AlTheKiller  We're relocating trunk to branches/xbox and branches/linuxport to trunk…

[b]Copied from trunk/XBMC/guilib/GUITextBox.cpp:[/b]
[22992] 8 months  arnova       [XBOX] fixed: Regression bugs in GUILib related to r22071 causing all kinds of weird things like chopped off text in the GUI
[22038] 9 months  arnova       [XBOX] merged: Linuxport revisions …
[20807] 11 months arnova       [XBOX] merged: Linuxport revisions …

If you look at the changes made in r22992 (identical to r23095) which seem to have been made to fix Xbox specific bugs, you can see that quite a few of these are reverted back to the linux branch's code in r24191.

Changelog of r22038 vs r22992: here ([XBOX] fixed: Regression bugs in GUILib related to r22071…)
Changelog of r22992 vs r24191: here ([XBOX] fixed: Ticket #7523 - Textbox not rendered in lists)

The changes from DWORD to integers can probably be safely ignored and assumed to be correct, but my guess would be that the problem lies in the other changes.
Look at this changelog between r24020 and r24191 to eliminate most of the DWORD > int adjustments for easier comparison: here

I hope my information can be of some help to arnova or another coder. If I do manage to set-up my own compiled build I'll play around with this myself.

[arnova]

I've done a little bit of investigating into this. Looking at the changes on trac for GUITextBox.cpp and using the revision numbers given by yadster (Last Known Working revision is 23183, first Confirmed Not Working revision 24421) it seems the bug appeared after revision 23095 and before revision 24613 (the latest). Luckily that's only 4 changes.

Code:

[b]Chgset  Date      Author       Log Message[/b]
[24613] 6 months  arnova       [XBOX] merged: trunk/mainline revisions …
[24238] 6 months  arnova       [XBOX] merged: Linuxport revisions 21939 & 23873
[24191] 6 months  arnova       [XBOX] fixed: Ticket #7523 - Textbox not rendered in lists
[24020] 7 months  arnova       [XBOX] merged: Linuxport revisions …
[23095] 8 months  AlTheKiller  We're relocating trunk to branches/xbox and branches/linuxport to trunk…

[b]Copied from trunk/XBMC/guilib/GUITextBox.cpp:[/b]
[22992] 8 months  arnova       [XBOX] fixed: Regression bugs in GUILib related to r22071 causing all kinds of weird things like chopped off text in the GUI
[22038] 9 months  arnova       [XBOX] merged: Linuxport revisions …
[20807] 11 months arnova       [XBOX] merged: Linuxport revisions …

If you look at the changes made in r22992 (identical to r23095) which seem to have been made to fix Xbox specific bugs, you can see that quite a few of these are reverted back to the linux branch's code in r24191.

Changelog of r22038 vs r22992: here ([XBOX] fixed: Regression bugs in GUILib related to r22071…
Changelog of r22992 vs r24191: here ([XBOX] fixed: Ticket #7523 - Textbox not rendered in lists)

The changes from DWORD to integers can probably be safely ignored and assumed to be correct, but my guess would be that the problem lies in the other changes.
Look at this changelog between r24020 and r24191 to eliminate most of the DWORD > int adjustments for easier comparison: here

I hope my information can be of some help to arnova or another coder. If I do manage to set-up my own compiled build I'll play around with this myself.

Are we sure then this is an Xbox-only issue, is what I'm wondering about?

EDIT: You may also want to try to revert the merge of r21939 and see whether that fixes the problem.

[eddebaby]

Are we sure then this is an Xbox-only issue, is what I'm wondering about?

Windows builds r29409 and r23944 both work fine. It does seem to be xbox-specific.

EDIT: You may also want to try to revert the merge of r21939 and see whether that fixes the problem.

I haven't got a compilation studio set-up yet. I can't do this at the moment.

[greatant1337]

Are there any other places other then plots/reviews where this is busted? If so, where?

Doesn't work in Video Library either. I assume that's what you meant.

[Bomb Bloke]

What I was hoping for was an area of XBMC that didn't require a library setup in order to see the problem.

Anyone wanna upload their library database files or something?

Or dare I hope that r29950 fixed this...?

[arnova]

Fixed in r30129. Again a real b*@#tch of a regression bug :/

[yadster]

Fixed in r30129. Again a real b*@#tch of regression bug :/

Thanks Arnova, just been trying the latest build and scrolling is all peachy again.

[VelvetKevorkian]

thx, waited for this fix for months

[elaguss]

Yeahh! finally has solved this bug. thnks so much 4 the last rev of XBMC for Xbox(nighlty build).
From http://sshcs.com/xbmc/:
The first rev that has solved the problem is:

Code:

Revision: 30129
Author: arnova
Date: 2010-05-13 20:23:01
Modified:  /branches/xbox/guilib/GUITextBox.cpp
Modified:  /branches/xbox/guilib/GUITextBox.h
[XBOX] fixed: Regression causing autoscrolling to be broken for eg. plots & reviews

THNKS!!

[Dan Dare]

Thanks a lot Arnova, this also fixes the double action you had to take to scroll certain pages (as in twice up or twice down to get it scrolled in windows like Scripts output). I feel bad cause I should've reported it, we plan to start a new forum on xbmc4xbox with changes as soon as they appear, so people get together and relate to the same problems, otherwise they might think it's their skin or their build, and just leave it and wait...