XBMC4Xbox

Nice work. The problem is that I can't properly comment on it, simply because I don't have any knowledge about DirectX stuff and such (never touched it). CC'ing Exobuzz, maybe he knows a bit more...

I'm afaid I don't have much either, but I'm trying to learn. Can't say I picked this spot in the code for any other reason than it is where I landed the first time walking through the process of loading fanart in the code. It does have the benefit of being fairly non-invasive - it only really touches things the one place during CPicture:Load()...

The patch does work, but of course the trick is actually creating the dds files to load in the first place, and it omits that part. I have that built into a external utility for testing (you can get it from the thread I linked to on the forums). I am working on trying to get the dds creation to work within xbmc, but have not made much progress yet.

I figured if I could get this much in for now, it would at least let a few brave people use it with the external utility until the dds creation gets properly done in xbmc.

Is this patch being considered for inclusion as is, or do you guys want to wait until dds creation is done? If there are no objections it would be helpful to get it in so that I no longer have to distribute a patched .xbe for those that want to test this using the offline utility.

As long as this patch doesn't negatively affect anything else in xbmc4xbox, I would personally like to see it implemented.

At the moment to be able to bug test it I'd have to get galvanash to build me a patched xbe, and with the current run of weather changes happening, it makes it difficult to keep up to date in that regard. (I haven't got a dedicated test xbox at the moment).

With the rest of the codebase currently being so stable, I really believe this is a great avenue to be taking. Anything that assists in reducing memory usage and speeding up loading times is where it's at for the old xbox.

Patch added in r30797. Cheers!