- Status Closed
- Percent Complete
- Task Type Patches
- Category Remote
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version
- Due in Version Version 3.0
-
Due Date
Undecided
- Votes
- Private
FS#5615 - settings.h modification for easier remote implementation
Small change defining remote buttons in settings.h
Closed by petur
2006-07-01 00:22
Reason for closing: Accepted
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
2006-07-01 00:22
Reason for closing: Accepted
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
check the committed code to see how
it\'s supposed to be done. If not
clear, read irc log of 20060701 starting
01:36 ;)
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 patch has no functionality, right?
It just creates defines that are nowhere used…
or do I miss something?
Yep…that’s right… It’s just that in the other patches for remote support I’ll use it, so it would make my life easier creating patches and testing if it was already committed.
I think it would be better to include this with the first patch that actually uses them
Oh bugger…but then I have to change code, add this patch, test the code remove the patch and then make a new patch! my fingers are getting sore! ;)
Also the patches I’m making on flyspray don’t work unless you install this one too. I could include it in every patch but then I would have to change them all as soon as one is committed! Life is hard eh?
submit a simple patch that includes this one, and continue once it is committed?
*maybe* I have a bit of time to look at it sunday evening….