Rockbox

Tasklist

FS#9638 - temp_cue is unused and wasting memory

Attached to Project: Rockbox
Opened by Boris Gjenero (dreamlayers) - Sunday, 14 December 2008, 06:20 GMT
Last edited by Bertrik Sikken (bertrik) - Saturday, 31 January 2009, 19:42 GMT
Task Type Patches
Category Music playback
Status Closed
Assigned To Nicolas Pennequin (nicolas_p)
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

It seems that in the past there was an attempt to load up to two cue sheets into memory, using curr_cue for the current track, and temp_cue for an upcoming track. Currently temp_cue is allocated and never loaded with data, wasting 70k. In gwps-common.c, the current filename is compared with uninitialized memory in temp_cue, which probably doesn't cause problems but seems like a bad idea.

In this patch I've removed code relating to temp_cue.
This task depends upon

Closed by  Bertrik Sikken (bertrik)
Saturday, 31 January 2009, 19:42 GMT
Reason for closing:  Accepted
Additional comments about closing:  Committed as svn r19890

Loading...