#include <unistd.h>
#include <fcntl.h>
#include "abook.h"
+#include <assert.h>
#include "database.h"
#include "list.h"
#include "misc.h"
parse_database(in);
- if ( items == 0 )
- return 2;
-
- return 0;
+ return (items == 0) ? 2 : 0;
}
int
item[EMAIL] = strdup("");
for(i=0; i<ITEM_FIELDS; i++)
- if( item[i] && (strlen(item[i]) > _MAX_FIELD_LEN(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]);
else
return;
- database = abook_realloc(database,
+ database = (list_item *)abook_realloc(database,
sizeof(list_item) * list_capacity);
- selected = abook_realloc(selected, list_capacity);
+ selected = (char *)abook_realloc(selected, list_capacity);
}
int
char *
get_surname(char *s)
{
- int i, a;
- int len = strlen(s);
- char *name = strdup(s);
-
- for( a = 0, i = len - 1; i >= 0; i--, a++ ) {
- name[a] = s[i];
- if(name[a] == ' ')
- break;
- }
+ char *p = s + strlen(s);
- name[ a ] = 0;
+ assert(s != NULL);
- revstr(name);
+ while(p > s && *(p - 1) != ' ')
+ p--;
- return name;
+ return strdup(p);
}
static int
s1 = get_surname(a[NAME]);
s2 = get_surname(b[NAME]);
- if( !(ret = safe_strcmp(s1, s2)) )
- ret = safe_strcmp(a[NAME], b[NAME]);
+ if( !(ret = safe_strcoll(s1, s2)) )
+ ret = safe_strcoll(a[NAME], b[NAME]);
free(s1);
free(s2);
itemcpy(a, i1);
itemcpy(b, i2);
- return safe_strcmp( a[NAME], b[NAME] );
+ return safe_strcoll( a[NAME], b[NAME] );
}
void
return -2; /* error */
findstr = strdup(str);
- findstr = strupper(findstr);
+ 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]]);
- if( tmp && strstr(strupper(tmp), findstr) ) {
+ if( tmp && strstr(strlower(tmp), findstr) ) {
ret = e.item;
goto out;
}
struct db_enumerator
init_db_enumerator(int mode)
{
- struct db_enumerator new;
+ struct db_enumerator e;
- new.item = -1; /* important - means "start from beginning" */
- new.mode = mode;
+ e.item = -1; /* important - means "start from beginning" */
+ e.mode = mode;
- return new;
+ return e;
}