Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#7858 - tiny improvement on tagtree.c

Attached to Project: Rockbox
Opened by Xinlu Huang (polygonal) - Friday, 28 September 2007, 23:17 GMT+1
Task Type Patches
Category Database
Status Unconfirmed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Private No

Details

Currently tagtree.c parses everything from tagnavi file(s), which is unnecessary because once a root menu is found, nothing more is needed since only the first root menu is used and anything used by that should be defined before the root_menu statement.

So I basically tell the parser to stop as soon as a valid root_menu is set. Obviously this fix is not a big deal, but the parser annoys me because it produces a lots of useless logf's when I tried to debug my own custom_tagnavi file that sets a different root menu.

I also removed a obsolete comment (the syntax in that comment is out of date).
   tagtree.patch (1.4 KiB) 
 tagtree.c |   16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

This task depends upon

Comment by Xinlu Huang (polygonal) - Saturday, 29 September 2007, 01:37 GMT+1
oops, sorry about the crlf ending... TortoiseSVN changed it without me knowing.
   tagtree.patch (1.4 KiB) 
 tagtree.c |   16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

Comment by Peter D'Hoye (petur) - Monday, 01 October 2007, 21:46 GMT+1
tortoiseSVN works native, so all files it works with are CR-LF on windows

Loading...