From 3200d04d75c5e7556ed8880b155533e881a4d1e1 Mon Sep 17 00:00:00 2001 From: Nils Wallménius Date: Thu, 20 Aug 2009 16:47:44 +0000 Subject: Make the formatter functions used by the settings return a pointer to avoid usless copying of lang strings, this brought with it a long chain of const correctness and a few random cleanups git-svn-id: svn://svn.rockbox.org/rockbox/trunk@22440 a1c6a512-1295-4272-9138-f99709370657 --- apps/plugins/chessbox/chessbox_pgn.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'apps/plugins/chessbox/chessbox_pgn.c') diff --git a/apps/plugins/chessbox/chessbox_pgn.c b/apps/plugins/chessbox/chessbox_pgn.c index cd163a5e1c..512fb0ca15 100644 --- a/apps/plugins/chessbox/chessbox_pgn.c +++ b/apps/plugins/chessbox/chessbox_pgn.c @@ -528,11 +528,10 @@ void coords_to_pgn(struct pgn_ply_node* ply){ } } -char * get_game_text(int selected_item, void *data, - char *buffer, size_t buffer_len){ +static const char* get_game_text(int selected_item, void *data, + char *buffer, size_t buffer_len){ int i; struct pgn_game_node *temp_node = (struct pgn_game_node *)data; - char text_buffer[50]; for (i=0;inext_node; @@ -540,10 +539,9 @@ char * get_game_text(int selected_item, void *data, if (temp_node == NULL){ return NULL; } - rb->snprintf(text_buffer, 50,"%s vs. %s (%s)", temp_node->white_player, + rb->snprintf(buffer, buffer_len,"%s vs. %s (%s)", temp_node->white_player, temp_node->black_player, temp_node->game_date); - rb->strlcpy(buffer, text_buffer, buffer_len); return buffer; } -- cgit v1.2.3