dev builds
themes manual
device status forums
mailing lists
IRC bugs
dev guide

Rockbox mail archive

Subject: Re: Blue_Dude: r25494 - trunk/apps

Re: Blue_Dude: r25494 - trunk/apps

From: Magnus Holmgren <>
Date: Tue, 06 Apr 2010 18:33:05 +0200

On 2010-04-05 22:53, wrote:

> New Revision: 25494
> Log Message: Restructure some bookmarking code, preparatory to adding
> version info to bookmarks. Saves some bin size as a bonus. No
> functional changes yet.

Some comments, though I don't know exactly what you plan to change (and
are you aware of FS#9407?):

Hm, are the flags really needed? With the struct, it seems like
F_BMFILES flag is the only one... But if they are kept, longer names
would be nice.

Why not pass struct bookmark_values as an argument? No need to keep it
as a static then, and safe to skip (most of) the flags.

resume_first_index can be removed, it isn't used.

Finally, I prefer to not "hide" function calls in macros. Doesn't seem
motivated here, IMHO.

Received on 2010-04-06

Page was last modified "Jan 10 2012" The Rockbox Crew