- Status Closed
- Percent Complete
- Task Type Patches
- Category Database
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#7858 - tiny improvement on tagtree.c
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).
Closed by jdgordon
2010-11-17 05:08
Reason for closing: Not a Bug
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2010-11-17 05:08
Reason for closing: Not a Bug
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
see the last comment
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
oops, sorry about the crlf ending… TortoiseSVN changed it without me knowing.
tortoiseSVN works native, so all files it works with are CR-LF on windows
Resync (just offset lines)
The reasoning does not appear to be valid: A root_menu can use other menus defined after itself. For example, a root_menu defined in a tagnavi_custom.config can refer to menus defined in tagnavi.config after tagnavi_custom.config has been included.