Rockbox

Tasklist

FS#9634 - Pictureflow: fix FS#9135 (again, now really)

Attached to Project: Rockbox
Opened by Thomas Martitz (kugel.) - Saturday, 13 December 2008, 22:45 GMT
Last edited by Jonas Häggqvist (rasher) - Sunday, 14 December 2008, 00:40 GMT
Task Type Patches
Category Plugins
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.0
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

My fix that was committed yesterday for that worked during testing, until I moved the deletion a slight bit down in the function. I moved it down because snprintf'ing to this global variable is done several times in that function, and I needed to add just more snprintfs to make it keep it workign. That broke the fix again without me noticing.

The patch also cleans up this particular global var, which just confused me during resolving that bug.

Sorry for the inconveniences :(
This task depends upon

Closed by  Jonas Häggqvist (rasher)
Sunday, 14 December 2008, 00:40 GMT
Reason for closing:  Accepted
Additional comments about closing:  Committed in revision 19432.
Comment by Thomas Martitz (kugel.) - Saturday, 13 December 2008, 22:57 GMT
Hmm, we agreed to not use MAX_PATH+1, so don't do it.
Comment by Thomas Martitz (kugel.) - Saturday, 13 December 2008, 22:57 GMT
with this one

Loading...