Index: apps/plugins/text_editor.c =================================================================== --- apps/plugins/text_editor.c (revision 20700) +++ apps/plugins/text_editor.c (working copy) @@ -154,6 +154,7 @@ if (!rb->read(fd,&t,1)) { rb->strcpy(eol,"\n"); + rb->close(fd); return 0; } if (t == '\r') Index: apps/plugins/spacerocks.c =================================================================== --- apps/plugins/spacerocks.c (revision 20700) +++ apps/plugins/spacerocks.c (working copy) @@ -347,7 +347,7 @@ #define SIZE_ENEMY_COLLISION 5*SCALE #define ATTRACT_FLIP_TIME 100 #define NUM_STARS 50 -#define NUM_TRAIL_POINTS 70 +#define NUM_TRAIL_POINTS 70 #define NUM_ROTATIONS 16 #define SIN_COS_SCALE 10000 @@ -1076,7 +1076,7 @@ if( enemy.position.x < ship.position.x) enemy_missile.position.dx = 1; else - enemy_missile.position.dx = -1; + enemy_missile.position.dx = -1; } if(enemy_missile.position.dx == 0 && @@ -1714,7 +1714,7 @@ but to do this we need to ascertain if the spacehip as moved on screen for more than a certain amount. */ - create_trail_blaze(THRUST_COLOUR, &ship.position); + create_trail_blaze(THRUST_COLOUR, &ship.position); } } @@ -1929,7 +1929,7 @@ CENTER_LCD_Y + CENTER_LCD_Y/2 - 4, "Game Over"); rb->snprintf(s, sizeof(s), "score %d ", current_score); rb->lcd_putsxy(1,LCD_HEIGHT-8, s); - show_game_over--; + show_game_over--; if(!show_game_over) start_attract_mode(); break; @@ -1979,8 +1979,8 @@ button = rb->button_get(false); #ifdef HAS_BUTTON_HOLD - if (rb->button_hold()) - game_state = PAUSE_MODE; + if (rb->button_hold() && game_state == PLAY_MODE) + game_state = PAUSE_MODE; #endif switch(button) Index: apps/plugins/star.c =================================================================== --- apps/plugins/star.c (revision 20700) +++ apps/plugins/star.c (working copy) @@ -166,7 +166,7 @@ (CONFIG_KEYPAD == SANSA_CLIP_PAD) || \ (CONFIG_KEYPAD == SANSA_M200_PAD) -#define STAR_QUIT BUTTON_POWER +#define STAR_QUIT BUTTON_POWER #define STAR_LEFT BUTTON_LEFT #define STAR_RIGHT BUTTON_RIGHT #define STAR_UP BUTTON_UP @@ -180,7 +180,7 @@ #elif (CONFIG_KEYPAD == SANSA_FUZE_PAD) -#define STAR_QUIT (BUTTON_HOME|BUTTON_REPEAT) +#define STAR_QUIT (BUTTON_HOME|BUTTON_REPEAT) #define STAR_LEFT BUTTON_LEFT #define STAR_RIGHT BUTTON_RIGHT #define STAR_UP BUTTON_UP @@ -1050,7 +1050,7 @@ } while(!menu_quit) { - switch(rb->do_menu(&menu, &selection, vp, false)) + switch(rb->do_menu(&menu, &selection, vp, true)) { case 0: menu_quit = true; Index: apps/plugins/rocklife.c =================================================================== --- apps/plugins/rocklife.c (revision 20700) +++ apps/plugins/rocklife.c (working copy) @@ -129,7 +129,7 @@ static bool load_cellfile(const char *file, char *pgrid){ int fd; fd = rb->open(file, O_RDONLY); - if (fd==-1) + if (fd<0) return false; init_grid(pgrid); Index: apps/plugins/lib/helper.c =================================================================== --- apps/plugins/lib/helper.c (revision 20700) +++ apps/plugins/lib/helper.c (working copy) @@ -25,8 +25,6 @@ /* Force the backlight on */ void backlight_force_on(void) { - if(!rb) - return; if (rb->global_settings->backlight_timeout > 0) rb->backlight_set_timeout(0); #if CONFIG_CHARGING @@ -38,8 +36,6 @@ /* Reset backlight operation to its settings */ void backlight_use_settings(void) { - if (!rb) - return; rb->backlight_set_timeout(rb->global_settings->backlight_timeout); #if CONFIG_CHARGING rb->backlight_set_timeout_plugged(rb->global_settings-> @@ -51,8 +47,6 @@ /* Force the backlight on */ void remote_backlight_force_on(void) { - if (!rb) - return; if (rb->global_settings->remote_backlight_timeout > 0) rb->remote_backlight_set_timeout(0); #if CONFIG_CHARGING @@ -64,8 +58,6 @@ /* Reset backlight operation to its settings */ void remote_backlight_use_settings(void) { - if (!rb) - return; rb->remote_backlight_set_timeout(rb->global_settings-> remote_backlight_timeout); #if CONFIG_CHARGING @@ -79,8 +71,6 @@ /* Force the buttonlight on */ void buttonlight_force_on(void) { - if (!rb) - return; if (rb->global_settings->buttonlight_timeout > 0) rb->buttonlight_set_timeout(0); } @@ -88,8 +78,6 @@ /* Reset buttonlight operation to its settings */ void buttonlight_use_settings(void) { - if (!rb) - return; rb->buttonlight_set_timeout(rb->global_settings->buttonlight_timeout); } #endif /* HAVE_BUTTON_LIGHT */ @@ -97,15 +85,11 @@ #ifdef HAVE_BACKLIGHT_BRIGHTNESS void backlight_brightness_set(int brightness) { - if (!rb) - return; rb->backlight_set_brightness(brightness); } void backlight_brightness_use_setting(void) { - if (!rb) - return; rb->backlight_set_brightness(rb->global_settings->brightness); } #endif /* HAVE_BACKLIGHT_BRIGHTNESS */ Index: apps/plugins/dict.c =================================================================== --- apps/plugins/dict.c (revision 20700) +++ apps/plugins/dict.c (working copy) @@ -147,8 +147,8 @@ #endif /* data files */ -#define DICT_INDEX ROCKBOX_DIR "/rocks/apps/dict.index" -#define DICT_DESC ROCKBOX_DIR "/rocks/apps/dict.desc" +#define DICT_INDEX PLUGIN_APPS_DIR "/dict.index" +#define DICT_DESC PLUGIN_APPS_DIR "/dict.desc" /* the main plugin function */ enum plugin_status plugin_start(const void* parameter) Index: apps/plugins/stopwatch.c =================================================================== --- apps/plugins/stopwatch.c (revision 20700) +++ apps/plugins/stopwatch.c (working copy) @@ -32,7 +32,7 @@ #define LAP_Y TIMER_Y+1 #define MAX_LAPS 64 -#define STOPWATCH_FILE ROCKBOX_DIR "/apps/stopwatch.dat" +#define STOPWATCH_FILE PLUGIN_APPS_DIR "/stopwatch.dat" /* variable button definitions */ #if CONFIG_KEYPAD == RECORDER_PAD Index: apps/plugins/keybox.c =================================================================== --- apps/plugins/keybox.c (revision 20700) +++ apps/plugins/keybox.c (working copy) @@ -22,7 +22,7 @@ #include "lib/md5.h" PLUGIN_HEADER -#define KEYBOX_FILE PLUGIN_DIR "/apps/keybox.dat" +#define KEYBOX_FILE PLUGIN_APPS_DIR "/keybox.dat" #define BLOCK_SIZE 8 #define MAX_ENTRIES 12*BLOCK_SIZE /* keep this a multiple of BLOCK_SIZE */ #define FIELD_LEN 32 /* should be enough for anyone ;) */ Index: apps/plugins/pegbox.c =================================================================== --- apps/plugins/pegbox.c (revision 20700) +++ apps/plugins/pegbox.c (working copy) @@ -39,8 +39,8 @@ #define PB_USB 2 #define PB_QUIT 1 -#define DATA_FILE PLUGIN_DIR "/games/pegbox.data" -#define SAVE_FILE PLUGIN_DIR "/games/pegbox.save" +#define DATA_FILE PLUGIN_GAMES_DIR "/pegbox.data" +#define SAVE_FILE PLUGIN_GAMES_DIR "/pegbox.save" #define ROWS 8 /* Number of rows on each board */ #define COLS 12 /* Number of columns on each board */ Index: apps/keymaps/keymap-gigabeat.c =================================================================== --- apps/keymaps/keymap-gigabeat.c (revision 20700) +++ apps/keymaps/keymap-gigabeat.c (working copy) @@ -68,8 +68,8 @@ static const struct button_mapping button_context_wps[] = { - { ACTION_WPS_PLAY, BUTTON_A|BUTTON_REL, BUTTON_A }, - { ACTION_WPS_STOP, BUTTON_POWER|BUTTON_REL, BUTTON_POWER }, + { ACTION_WPS_PLAY, BUTTON_A|BUTTON_REL, BUTTON_A }, + { ACTION_WPS_STOP, BUTTON_POWER|BUTTON_REL, BUTTON_POWER }, { ACTION_WPS_SKIPNEXT, BUTTON_RIGHT|BUTTON_REL, BUTTON_RIGHT }, { ACTION_WPS_SKIPPREV, BUTTON_LEFT|BUTTON_REL, BUTTON_LEFT }, @@ -79,39 +79,39 @@ { ACTION_WPS_STOPSEEK, BUTTON_LEFT|BUTTON_REL, BUTTON_LEFT|BUTTON_REPEAT }, { ACTION_WPS_STOPSEEK, BUTTON_RIGHT|BUTTON_REL, BUTTON_RIGHT|BUTTON_REPEAT }, - { ACTION_WPS_ABSETB_NEXTDIR, BUTTON_A|BUTTON_RIGHT, BUTTON_NONE }, - { ACTION_WPS_ABSETA_PREVDIR, BUTTON_A|BUTTON_LEFT, BUTTON_NONE }, - { ACTION_WPS_ABRESET, BUTTON_A|BUTTON_SELECT, BUTTON_NONE }, + { ACTION_WPS_ABSETB_NEXTDIR,BUTTON_A|BUTTON_RIGHT, BUTTON_NONE }, + { ACTION_WPS_ABSETA_PREVDIR,BUTTON_A|BUTTON_LEFT, BUTTON_NONE }, + { ACTION_WPS_ABRESET, BUTTON_A|BUTTON_SELECT, BUTTON_NONE }, { ACTION_WPS_VOLDOWN, BUTTON_DOWN|BUTTON_REPEAT, BUTTON_NONE }, { ACTION_WPS_VOLDOWN, BUTTON_DOWN, BUTTON_NONE }, { ACTION_WPS_VOLDOWN, BUTTON_VOL_DOWN, BUTTON_NONE }, - { ACTION_WPS_VOLDOWN, BUTTON_VOL_DOWN|BUTTON_REPEAT, BUTTON_NONE }, + { ACTION_WPS_VOLDOWN, BUTTON_VOL_DOWN|BUTTON_REPEAT, BUTTON_NONE }, { ACTION_WPS_VOLUP, BUTTON_UP|BUTTON_REPEAT, BUTTON_NONE }, { ACTION_WPS_VOLUP, BUTTON_UP, BUTTON_NONE }, - { ACTION_WPS_VOLUP, BUTTON_VOL_UP|BUTTON_REPEAT, BUTTON_NONE }, - { ACTION_WPS_VOLUP, BUTTON_VOL_UP, BUTTON_NONE }, + { ACTION_WPS_VOLUP, BUTTON_VOL_UP|BUTTON_REPEAT, BUTTON_NONE }, + { ACTION_WPS_VOLUP, BUTTON_VOL_UP, BUTTON_NONE }, - { ACTION_WPS_PITCHSCREEN, BUTTON_A|BUTTON_UP, BUTTON_A }, - { ACTION_WPS_PITCHSCREEN, BUTTON_A|BUTTON_DOWN, BUTTON_A }, + { ACTION_WPS_PITCHSCREEN, BUTTON_A|BUTTON_UP, BUTTON_A }, + { ACTION_WPS_PITCHSCREEN, BUTTON_A|BUTTON_DOWN, BUTTON_A }, { ACTION_WPS_QUICKSCREEN, BUTTON_MENU|BUTTON_REPEAT, BUTTON_MENU }, { ACTION_WPS_MENU, BUTTON_MENU|BUTTON_REL, BUTTON_MENU }, { ACTION_WPS_CONTEXT, BUTTON_SELECT|BUTTON_REPEAT, BUTTON_SELECT }, - { ACTION_WPS_ID3SCREEN, BUTTON_A|BUTTON_MENU, BUTTON_NONE }, + { ACTION_WPS_ID3SCREEN, BUTTON_A|BUTTON_MENU, BUTTON_NONE }, { ACTION_WPS_BROWSE, BUTTON_SELECT|BUTTON_REL, BUTTON_SELECT }, LAST_ITEM_IN_LIST }; /* button_context_wps */ static const struct button_mapping button_context_list[] = { - { ACTION_LISTTREE_PGUP, BUTTON_A|BUTTON_UP, BUTTON_A }, - { ACTION_LISTTREE_PGUP, BUTTON_UP|BUTTON_REL, BUTTON_A|BUTTON_UP }, - { ACTION_LISTTREE_PGUP, BUTTON_A|BUTTON_UP|BUTTON_REPEAT, BUTTON_NONE }, - { ACTION_LISTTREE_PGDOWN, BUTTON_A|BUTTON_DOWN, BUTTON_A }, - { ACTION_LISTTREE_PGDOWN, BUTTON_DOWN|BUTTON_REL, BUTTON_A|BUTTON_DOWN }, - { ACTION_LISTTREE_PGDOWN, BUTTON_A|BUTTON_DOWN|BUTTON_REPEAT, BUTTON_NONE }, + { ACTION_LISTTREE_PGUP, BUTTON_A|BUTTON_UP, BUTTON_A }, + { ACTION_LISTTREE_PGUP, BUTTON_UP|BUTTON_REL, BUTTON_A|BUTTON_UP }, + { ACTION_LISTTREE_PGUP, BUTTON_A|BUTTON_UP|BUTTON_REPEAT, BUTTON_NONE }, + { ACTION_LISTTREE_PGDOWN, BUTTON_A|BUTTON_DOWN, BUTTON_A }, + { ACTION_LISTTREE_PGDOWN, BUTTON_DOWN|BUTTON_REL, BUTTON_A|BUTTON_DOWN }, + { ACTION_LISTTREE_PGDOWN, BUTTON_A|BUTTON_DOWN|BUTTON_REPEAT, BUTTON_NONE }, #ifdef HAVE_VOLUME_IN_LIST { ACTION_LIST_VOLUP, BUTTON_VOL_UP|BUTTON_REPEAT, BUTTON_NONE }, { ACTION_LIST_VOLUP, BUTTON_VOL_UP, BUTTON_NONE }, @@ -123,7 +123,7 @@ }; /* button_context_list */ static const struct button_mapping button_context_tree[] = { - { ACTION_TREE_WPS, BUTTON_A|BUTTON_REL, BUTTON_A }, + { ACTION_TREE_WPS, BUTTON_A|BUTTON_REL, BUTTON_A }, { ACTION_TREE_STOP, BUTTON_POWER, BUTTON_NONE }, { ACTION_TREE_STOP, BUTTON_POWER|BUTTON_REL, BUTTON_POWER }, { ACTION_TREE_STOP, BUTTON_POWER|BUTTON_REPEAT, BUTTON_NONE }, @@ -134,12 +134,12 @@ static const struct button_mapping button_context_listtree_scroll_with_combo[] = { { ACTION_NONE, BUTTON_A, BUTTON_NONE }, { ACTION_TREE_PGLEFT, BUTTON_A|BUTTON_LEFT, BUTTON_A }, - { ACTION_TREE_PGLEFT, BUTTON_LEFT|BUTTON_REL, BUTTON_A|BUTTON_LEFT }, + { ACTION_TREE_PGLEFT, BUTTON_LEFT|BUTTON_REL, BUTTON_A|BUTTON_LEFT }, { ACTION_TREE_PGLEFT, BUTTON_A|BUTTON_LEFT, BUTTON_LEFT|BUTTON_REL }, - { ACTION_TREE_ROOT_INIT, BUTTON_A|BUTTON_LEFT|BUTTON_REPEAT, BUTTON_A|BUTTON_LEFT }, + { ACTION_TREE_ROOT_INIT, BUTTON_A|BUTTON_LEFT|BUTTON_REPEAT, BUTTON_A|BUTTON_LEFT }, { ACTION_TREE_PGLEFT, BUTTON_A|BUTTON_LEFT|BUTTON_REPEAT, BUTTON_NONE }, { ACTION_TREE_PGRIGHT, BUTTON_A|BUTTON_RIGHT, BUTTON_A }, - { ACTION_TREE_PGRIGHT, BUTTON_RIGHT|BUTTON_REL, BUTTON_A|BUTTON_RIGHT }, + { ACTION_TREE_PGRIGHT, BUTTON_RIGHT|BUTTON_REL, BUTTON_A|BUTTON_RIGHT }, { ACTION_TREE_PGRIGHT, BUTTON_A|BUTTON_RIGHT, BUTTON_RIGHT|BUTTON_REL }, { ACTION_TREE_PGRIGHT, BUTTON_A|BUTTON_RIGHT|BUTTON_REPEAT, BUTTON_NONE }, LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_CUSTOM|CONTEXT_TREE), @@ -167,7 +167,7 @@ { ACTION_STD_PREVREPEAT, BUTTON_LEFT|BUTTON_REPEAT, BUTTON_NONE }, { ACTION_STD_NEXT, BUTTON_RIGHT, BUTTON_NONE }, { ACTION_STD_NEXTREPEAT, BUTTON_RIGHT|BUTTON_REPEAT, BUTTON_NONE }, - { ACTION_SETTINGS_RESET, BUTTON_A, BUTTON_NONE }, + { ACTION_SETTINGS_RESET, BUTTON_A, BUTTON_NONE }, LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_STD) }; /* button_context_settings */ @@ -181,36 +181,36 @@ { ACTION_STD_PREVREPEAT, BUTTON_UP|BUTTON_REPEAT, BUTTON_NONE }, { ACTION_STD_NEXT, BUTTON_DOWN, BUTTON_NONE }, { ACTION_STD_NEXTREPEAT, BUTTON_DOWN|BUTTON_REPEAT, BUTTON_NONE }, - { ACTION_SETTINGS_RESET, BUTTON_A, BUTTON_NONE }, + { ACTION_SETTINGS_RESET, BUTTON_A, BUTTON_NONE }, LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_STD) }; /* button_context_settingsgraphical */ static const struct button_mapping button_context_yesno[] = { - { ACTION_YESNO_ACCEPT, BUTTON_SELECT, BUTTON_NONE }, + { ACTION_YESNO_ACCEPT, BUTTON_SELECT, BUTTON_NONE }, LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_STD) }; /* button_context_settings_yesno */ static const struct button_mapping button_context_colorchooser[] = { - { ACTION_STD_OK, BUTTON_A|BUTTON_REL, BUTTON_NONE }, + { ACTION_STD_OK, BUTTON_A|BUTTON_REL, BUTTON_NONE }, LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_CUSTOM|CONTEXT_SETTINGS), }; /* button_context_colorchooser */ static const struct button_mapping button_context_eq[] = { - { ACTION_STD_OK, BUTTON_SELECT|BUTTON_REL, BUTTON_NONE }, + { ACTION_STD_OK, BUTTON_SELECT|BUTTON_REL, BUTTON_NONE }, LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_CUSTOM|CONTEXT_SETTINGS), }; /* button_context_eq */ /** Bookmark Screen **/ static const struct button_mapping button_context_bmark[] = { - { ACTION_BMS_DELETE, BUTTON_A, BUTTON_NONE }, + { ACTION_BMS_DELETE, BUTTON_A, BUTTON_NONE }, LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_LIST), }; /* button_context_bmark */ static const struct button_mapping button_context_time[] = { - { ACTION_STD_CANCEL, BUTTON_POWER, BUTTON_NONE }, - { ACTION_STD_OK, BUTTON_A, BUTTON_NONE }, + { ACTION_STD_CANCEL, BUTTON_POWER, BUTTON_NONE }, + { ACTION_STD_OK, BUTTON_A, BUTTON_NONE }, LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_SETTINGS), }; /* button_context_time */ @@ -238,8 +238,8 @@ { ACTION_PS_NUDGE_RIGHT, BUTTON_RIGHT, BUTTON_NONE }, { ACTION_PS_NUDGE_RIGHTOFF, BUTTON_RIGHT|BUTTON_REL, BUTTON_NONE }, { ACTION_PS_TOGGLE_MODE, BUTTON_MENU, BUTTON_NONE }, - { ACTION_PS_RESET, BUTTON_A, BUTTON_NONE }, - { ACTION_PS_EXIT, BUTTON_POWER, BUTTON_NONE }, + { ACTION_PS_RESET, BUTTON_A, BUTTON_NONE }, + { ACTION_PS_EXIT, BUTTON_POWER, BUTTON_NONE }, LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_STD) }; /* button_context_pitchcreen */ @@ -249,14 +249,14 @@ { ACTION_KBD_LEFT, BUTTON_LEFT|BUTTON_REPEAT, BUTTON_NONE }, { ACTION_KBD_RIGHT, BUTTON_RIGHT, BUTTON_NONE }, { ACTION_KBD_RIGHT, BUTTON_RIGHT|BUTTON_REPEAT, BUTTON_NONE }, - { ACTION_KBD_CURSOR_LEFT, BUTTON_A|BUTTON_LEFT, BUTTON_NONE }, - { ACTION_KBD_CURSOR_LEFT, BUTTON_A|BUTTON_LEFT|BUTTON_REPEAT, BUTTON_NONE }, - { ACTION_KBD_CURSOR_RIGHT, BUTTON_A|BUTTON_RIGHT, BUTTON_NONE }, - { ACTION_KBD_CURSOR_RIGHT, BUTTON_A|BUTTON_RIGHT|BUTTON_REPEAT, BUTTON_NONE }, + { ACTION_KBD_CURSOR_LEFT, BUTTON_A|BUTTON_LEFT, BUTTON_NONE }, + { ACTION_KBD_CURSOR_LEFT, BUTTON_A|BUTTON_LEFT|BUTTON_REPEAT, BUTTON_NONE }, + { ACTION_KBD_CURSOR_RIGHT, BUTTON_A|BUTTON_RIGHT, BUTTON_NONE }, + { ACTION_KBD_CURSOR_RIGHT, BUTTON_A|BUTTON_RIGHT|BUTTON_REPEAT, BUTTON_NONE }, { ACTION_KBD_SELECT, BUTTON_SELECT, BUTTON_NONE }, - { ACTION_KBD_PAGE_FLIP, BUTTON_A|BUTTON_MENU, BUTTON_NONE }, - { ACTION_KBD_DONE, BUTTON_A|BUTTON_REL, BUTTON_A }, - { ACTION_KBD_ABORT, BUTTON_POWER|BUTTON_REL, BUTTON_POWER }, + { ACTION_KBD_PAGE_FLIP, BUTTON_A|BUTTON_MENU, BUTTON_NONE }, + { ACTION_KBD_DONE, BUTTON_A|BUTTON_REL, BUTTON_A }, + { ACTION_KBD_ABORT, BUTTON_POWER|BUTTON_REL, BUTTON_POWER }, { ACTION_KBD_BACKSPACE, BUTTON_MENU, BUTTON_NONE }, { ACTION_KBD_BACKSPACE, BUTTON_MENU|BUTTON_REPEAT, BUTTON_NONE }, { ACTION_KBD_UP, BUTTON_UP, BUTTON_NONE }, Index: apps/plugins/sudoku/sudoku.h =================================================================== --- apps/plugins/sudoku/sudoku.h (revision 20700) +++ apps/plugins/sudoku/sudoku.h (working copy) @@ -135,13 +135,13 @@ #define SUDOKU_BUTTON_POSSIBLE BUTTON_REC #elif (CONFIG_KEYPAD == SANSA_FUZE_PAD) -#define SUDOKU_BUTTON_QUIT (BUTTON_HOME|BUTTON_REPEAT) -#define SUDOKU_BUTTON_UP BUTTON_UP -#define SUDOKU_BUTTON_DOWN BUTTON_DOWN -#define SUDOKU_BUTTON_LEFT BUTTON_LEFT -#define SUDOKU_BUTTON_RIGHT BUTTON_RIGHT +#define SUDOKU_BUTTON_QUIT (BUTTON_HOME|BUTTON_REPEAT) +#define SUDOKU_BUTTON_UP BUTTON_UP +#define SUDOKU_BUTTON_DOWN BUTTON_DOWN +#define SUDOKU_BUTTON_LEFT BUTTON_LEFT +#define SUDOKU_BUTTON_RIGHT BUTTON_RIGHT #define SUDOKU_BUTTON_TOGGLEBACK BUTTON_SCROLL_BACK -#define SUDOKU_BUTTON_TOGGLE BUTTON_SCROLL_FWD +#define SUDOKU_BUTTON_TOGGLE BUTTON_SCROLL_FWD #define SUDOKU_BUTTON_MENU (BUTTON_POWER | BUTTON_REL) #define SUDOKU_BUTTON_POSSIBLE (BUTTON_SELECT | BUTTON_UP) Index: apps/plugin.c =================================================================== --- apps/plugin.c (revision 20700) +++ apps/plugin.c (working copy) @@ -634,13 +634,13 @@ the API gets incompatible */ #if defined(HAVE_LCD_MODES) - lcd_set_mode, + lcd_set_mode, #endif #if defined(HAVE_LCD_MODES) #if HAVE_LCD_MODES & LCD_MODE_PAL256 - lcd_blit_pal256, - lcd_pal256_update_pal, + lcd_blit_pal256, + lcd_pal256_update_pal, #endif #endif }; Index: apps/plugin.h =================================================================== --- apps/plugin.h (revision 20700) +++ apps/plugin.h (working copy) @@ -787,19 +787,19 @@ void (*semaphore_release)(struct semaphore *s); #endif - const char *appsversion; + const char *appsversion; /* new stuff at the end, sort into place next time the API gets incompatible */ #if defined(HAVE_LCD_MODES) - void (*lcd_set_mode)(int mode); + void (*lcd_set_mode)(int mode); #endif #if defined(HAVE_LCD_MODES) #if HAVE_LCD_MODES & LCD_MODE_PAL256 - void (*lcd_blit_pal256)(unsigned char *src, int src_x, int src_y, int x, int y, - int width, int height); - void (*lcd_pal256_update_pal)(fb_data *palette); + void (*lcd_blit_pal256)(unsigned char *src, int src_x, int src_y, int x, int y, + int width, int height); + void (*lcd_pal256_update_pal)(fb_data *palette); #endif #endif };