X-Git-Url: https://git.deb.at/w?a=blobdiff_plain;f=database.c;h=ef4b67bb51bc27ce7eaee4bc5fb4fee4569907cd;hb=4b8f9231090ada43f7e16987ec46ac7f45a914ec;hp=f7bb7459373391db14a95a9f39a71886efcddc26;hpb=ef379a20d142cb95ba08c7887c8e3245269b3fdd;p=pkg%2Fabook.git

diff --git a/database.c b/database.c
index f7bb745..ef4b67b 100644
--- a/database.c
+++ b/database.c
@@ -15,6 +15,7 @@
 #include "abook.h"
 #include <assert.h>
 #include "database.h"
+#include "gettext.h"
 #include "list.h"
 #include "misc.h"
 #include "options.h"
@@ -24,23 +25,19 @@
 #	include "config.h"
 #endif
 
-static void	free_item(int i);
-
-
 list_item *database = NULL;
 
 int items = 0;
 
 #define INITIAL_LIST_CAPACITY	30
 
-int list_capacity = 0;
+static int list_capacity = 0;
 
 extern int first_list_item;
 extern int curitem;
 extern char *selected;
 
 extern char *datafile;
-extern char *rcfile;
 
 /*
  * field definitions
@@ -57,26 +54,26 @@ extern char *rcfile;
  */
 
 struct abook_field abook_fields[ITEM_FIELDS] = {
-	{"Name",	"name",		TAB_CONTACT},/* NAME */
-	{"E-mails",	"email",	TAB_CONTACT},/* EMAIL */
-	{"Address",	"address",	TAB_ADDRESS},/* ADDRESS */
-	{"Address2",	"address2",	TAB_ADDRESS},/* ADDRESS2 */
-	{"City",	"city",		TAB_ADDRESS},/* CITY */
-	{"State/Province","state",	TAB_ADDRESS},/* STATE */
-	{"ZIP/Postal Code","zip",	TAB_ADDRESS},/* ZIP */
-	{"Country",	"country",	TAB_ADDRESS},/* COUNTRY */
-	{"Home Phone",	"phone",	TAB_PHONE},/* PHONE */
-	{"Work Phone",	"workphone",	TAB_PHONE},/* WORKPHONE */
-	{"Fax",		"fax",		TAB_PHONE},/* FAX */
-	{"Mobile",	"mobile",	TAB_PHONE},/* MOBILEPHONE */
-	{"Nickname/Alias", "nick",      TAB_OTHER},/* NICK */
-	{"URL",		"url",		TAB_OTHER},/* URL */
-	{"Notes",	"notes",	TAB_OTHER},/* NOTES */
-	{"Custom1",	"custom1",	TAB_CUSTOM},/* CUSTOM1 */
-	{"Custom2",	"custom2",	TAB_CUSTOM},/* CUSTOM2 */
-	{"Custom3",	"custom3",	TAB_CUSTOM},/* CUSTOM3 */
-	{"Custom4",	"custom4",	TAB_CUSTOM},/* CUSTOM4 */
-	{"Custom5",	"custom5",	TAB_CUSTOM},/* CUSTOM5 */
+	{N_("Name"),	"name",		TAB_CONTACT},/* NAME */
+	{N_("E-mails"),	"email",	TAB_CONTACT},/* EMAIL */
+	{N_("Address"),	"address",	TAB_ADDRESS},/* ADDRESS */
+	{N_("Address2"),	"address2",	TAB_ADDRESS},/* ADDRESS2 */
+	{N_("City"),	"city",		TAB_ADDRESS},/* CITY */
+	{N_("State/Province"),"state",	TAB_ADDRESS},/* STATE */
+	{N_("ZIP/Postal Code"),"zip",	TAB_ADDRESS},/* ZIP */
+	{N_("Country"),	"country",	TAB_ADDRESS},/* COUNTRY */
+	{N_("Home Phone"),	"phone",	TAB_PHONE},/* PHONE */
+	{N_("Work Phone"),	"workphone",	TAB_PHONE},/* WORKPHONE */
+	{N_("Fax"),		"fax",		TAB_PHONE},/* FAX */
+	{N_("Mobile"),	"mobile",	TAB_PHONE},/* MOBILEPHONE */
+	{N_("Nickname/Alias"), "nick",      TAB_OTHER},/* NICK */
+	{N_("URL"),		"url",		TAB_OTHER},/* URL */
+	{N_("Notes"),	"notes",	TAB_OTHER},/* NOTES */
+	{N_("Custom1"),	"custom1",	TAB_CUSTOM},/* CUSTOM1 */
+	{N_("Custom2"),	"custom2",	TAB_CUSTOM},/* CUSTOM2 */
+	{N_("Custom3"),	"custom3",	TAB_CUSTOM},/* CUSTOM3 */
+	{N_("Custom4"),	"custom4",	TAB_CUSTOM},/* CUSTOM4 */
+	{N_("Custom5"),	"custom5",	TAB_CUSTOM},/* CUSTOM5 */
 };
 
 
@@ -104,39 +101,38 @@ parse_database(FILE *in)
 
 	for(;;) {
 		line = getaline(in);
-		if( feof(in) ) {
-			if( item[NAME] && sec )
+		if(feof(in)) {
+			if(item[NAME] && sec)
 				add_item2database(item);
 			else
 				free_list_item(item);
 			break;
 		}
 
-		if( !*line || *line == '\n' || *line == '#' ) {
-			free(line);
-			continue;
-		} else if( *line == '[' ) {
+		if(!*line || *line == '\n' || *line == '#') {
+			goto next;
+		} else if(*line == '[') {
 			if( item[NAME] && sec )
 				add_item2database(item);
 			else
 				free_list_item(item);
 			memset(&item, 0, sizeof(item));
 			sec = 1;
-			if ( !(tmp = strchr(line, ']')))
+			if(!(tmp = strchr(line, ']')))
 				sec = 0; /*incorrect section lines are skipped*/
-		} else if((tmp = strchr(line, '=') ) && sec ) {
+		} else if((tmp = strchr(line, '=') ) && sec) {
 			*tmp++ = '\0';
-			for(i=0; i<ITEM_FIELDS; i++)
-				if( !strcmp(abook_fields[i].key, line) ) {
-					item[i] = strdup(tmp);
+			for(i = 0; i < ITEM_FIELDS; i++)
+				if(!strcmp(abook_fields[i].key, line)) {
+					item[i] = xstrdup(tmp);
 					goto next;
 				}
 		}
 next:
-		free(line);
+		xfree(line);
 	}
 
-	free(line);
+	xfree(line);
 	return 0;
 }
 
@@ -147,10 +143,10 @@ load_database(char *filename)
 {
 	FILE *in;
 
-	if( database != NULL )
+	if(database != NULL)
 		close_database();
 
-	if ( (in = abook_fopen(filename, "r")) == NULL )
+	if ((in = abook_fopen(filename, "r")) == NULL)
 		return -1;
 
 	parse_database(in);
@@ -198,7 +194,7 @@ save_database()
 	if( (out = abook_fopen(datafile, "w")) == NULL )
 		return -1;
 
-	if( list_is_empty() ) {
+	if(list_is_empty()) {
 		fclose(out);
 		unlink(datafile);
 		return 1;
@@ -255,13 +251,13 @@ validate_item(list_item item)
 	char *tmp;
 
 	if(item[EMAIL] == NULL)
-		item[EMAIL] = strdup("");
+		item[EMAIL] = xstrdup("");
 
 	for(i=0; i<ITEM_FIELDS; i++)
 		if( item[i] && ((int)strlen(item[i]) > _MAX_FIELD_LEN(i) ) ) {
 			tmp = item[i];
 			item[i][_MAX_FIELD_LEN(i)-1] = 0;
-			item[i] = strdup(item[i]);
+			item[i] = xstrdup(item[i]);
 			free(tmp);
 		}
 }
@@ -286,12 +282,12 @@ adjust_list_capacity()
 int
 add_item2database(list_item item)
 {
-	if( item[NAME] == NULL || ! *item[NAME] ) {
+	if(item[NAME] == NULL || ! *item[NAME]) {
 		free_list_item(item);
 		return 1;
 	}
 
-	if( ++items > list_capacity)
+	if(++items > list_capacity)
 		adjust_list_capacity();
 
 	validate_item(item);
@@ -307,27 +303,26 @@ remove_selected_items()
 {
 	int i, j;
 
-	if( list_is_empty() )
+	if(list_is_empty())
 		return;
 
-	if( ! selected_items() )
-		selected[ curitem ] = 1;
+	if(!selected_items())
+		selected[curitem] = 1;
 
-	for( j = LAST_ITEM; j >= 0; j-- ) {
-		if( selected[j] ) {
+	for(j = LAST_ITEM; j >= 0; j--) {
+		if(selected[j]) {
 			free_item(j); /* added for .4 data_s_ */
-			for( i = j; i < LAST_ITEM; i++ ) {
-				itemcpy(database[ i ], database[ i + 1 ]);
-				selected[ i ] = selected[ i + 1 ];
+			for(i = j; i < LAST_ITEM; i++) {
+				itemcpy(database[i], database[i + 1]);
+				selected[i] = selected[i + 1];
 			}
 			items--;
 		}
 	}
 
-	if( curitem > LAST_ITEM && items > 0 )
+	if(curitem > LAST_ITEM && items > 0)
 		curitem = LAST_ITEM;
 
-
 	adjust_list_capacity();
 
 	select_none();
@@ -343,7 +338,7 @@ get_surname(char *s)
 	while(p > s && *(p - 1) != ' ')
 		p--;
 
-	return strdup(p);
+	return xstrdup(p);
 }
 
 static int
@@ -380,7 +375,7 @@ namecmp(const void *i1, const void *i2)
 	itemcpy(a, i1);
 	itemcpy(b, i2);
 
-	return safe_strcoll( a[sort_field], b[sort_field] );
+	return safe_strcoll(a[sort_field], b[sort_field]);
 }
 
 static int
@@ -408,8 +403,8 @@ sort_by_field(int field)
 	if(field < 0) {
 		field = name2field(opt_get_str(STR_SORT_FIELD));
 		if(field < 0) {
-			statusline_msg("Not valid field value defined "
-				"in configuration");
+			statusline_msg(_("Not valid field value defined "
+				"in configuration"));
 			return;
 		}
 	}
@@ -443,13 +438,15 @@ find_item(char *str, int start, int search_fields[])
 	if(list_is_empty() || !is_valid_item(start))
 		return -2; /* error */
 
-	findstr = strdup(str);
+	findstr = xstrdup(str);
 	findstr = strlower(findstr);
 
 	e.item = start - 1; /* must be "real start" - 1 */
 	db_enumerate_items(e) {
-		for( i = 0; search_fields[i] >= 0; i++ ) {
-			tmp = safe_strdup(database[e.item][search_fields[i]]);
+		for(i = 0; search_fields[i] >= 0; i++) {
+			if(database[e.item][search_fields[i]] == NULL)
+				continue;
+			tmp = xstrdup(database[e.item][search_fields[i]]);
 			if( tmp && strstr(strlower(tmp), findstr) ) {
 				ret = e.item;
 				goto out;