Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Operating System/Drivers
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by jdgordon - 2007-03-28
Last edited by jdgordon - 2007-03-28

FS#6922 - recode filetypes.c, decrease bin by ~1400bytes on h120)

The annoying string buffer error came up in IRC this arvo and the idea was to move those strings to the audio buffer (its all done before the audio_init and only ever once so its safe).

This patch does that.

The patch basically starts filetypes.c from scratch to hopefully make it more readable and managable.
The only important implementation change I did was remove the ability to automatically register any rocks with the name blaa-<ext>.rock which I (and others in irc) never even knew was possible.

Now, the reason I didnt commit this is because even though it saves 1400bytes on the h120 build, it uses an extra 1100bytes RAM (compared the audio buffer size with svn on my sansa). Now, do we care about that? The code could probably use less ram, but sacrafise bin size….
oh, and ram usage will be about half of that on HW_CODEC… last thing, can someone with HW_CODEC make sure it loads without any splashes saying too many extensions? I have a sneeking feeling it will need to be increased….. (edit: sim says its ok…)

Closed by  jdgordon
2007-03-29 06:16
Reason for closing:  Out of Date

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing