- Status Closed
- Percent Complete
- Task Type Bugs
- Category User Interface
- Assigned To No-one
- Operating System
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#1780 - File type handling in tree.c is limited and incorrect
The file type handling in tree.c needs improvements.
Storing the file type as ordinary file attributes is
limited and logically wrong.
Rockbox defines 6 "normal" file attributes and 11 file
type attributes and packs that in a 16-bit word
(possible by using 2 bits for the text file). Adding a
bookmark file type will only make this problem worse.
As implemented in tree.c, this worked fine until the
.rockbox browsing feature was added. Browsing for .cfg
or .wps files will also show .txt files.
As tree.c should (and mostly does) treat a file as
being of one type only, using attributes to store the
information is logically wrong and also misleading (as
shown by the .rockbox browsing code).
I suggest that Rockbox either use the top 8 bits of the
entry.attr field (in tree.h) for a file type enum (not
neccessarily as a C enum), or, maybe better, split attr
into two bytes, attr and type. I can do the changes, if
needed.
2005-03-09 10:52
Reason for closing: Out of Date
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
Logged In: YES
user_id=1110
This report is very old and concerns a Rockbox version from eons ago.
Lots happen in the Rockbox all the time,
and if you still have
problems with
this matter I suggest you file a new
entry.
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
This (splitting into attr and type) sounds like a good change. Feel free
to submit a patch.
This report is very old and concerns a Rockbox version from eons ago.
Lots happen in the Rockbox all the time, and if you still have
problems with this matter I suggest you file a new entry.