X-Git-Url: http://www.privoxy.org/gitweb/?p=privoxy.git;a=blobdiff_plain;f=ssplit.c;h=ef40e64492e8698e7e10dcd30b934e203be21478;hp=ff362568a58d5652b7c8853585cc3786b274a10e;hb=960ae61d1bce3289d3a79290a3d0c583620c2414;hpb=97110184221edacdb7d4dfc29de0e38d63f4d831 diff --git a/ssplit.c b/ssplit.c index ff362568..ef40e644 100644 --- a/ssplit.c +++ b/ssplit.c @@ -1,18 +1,17 @@ -const char ssplit_rcs[] = "$Id: ssplit.c,v 1.5 2002/03/24 13:25:43 swa Exp $"; /********************************************************************* * * File : $Source: /cvsroot/ijbswa/current/ssplit.c,v $ * - * Purpose : A function to split a string at specified deliminters. + * Purpose : A function to split a string at specified delimiters. * - * Copyright : Written by and Copyright (C) 2001 the SourceForge - * Privoxy team. http://www.privoxy.org/ + * Copyright : Written by and Copyright (C) 2001-2012 the + * Privoxy team. https://www.privoxy.org/ * * Based on the Internet Junkbuster originally written - * by and Copyright (C) 1997 Anonymous Coders and + * by and Copyright (C) 1997 Anonymous Coders and * Junkbusters Corporation. http://www.junkbusters.com * - * This program is free software; you can redistribute it + * This program is free software; you can redistribute it * and/or modify it under the terms of the GNU General * Public License as published by the Free Software * Foundation; either version 2 of the License, or (at @@ -30,44 +29,18 @@ const char ssplit_rcs[] = "$Id: ssplit.c,v 1.5 2002/03/24 13:25:43 swa Exp $"; * or write to the Free Software Foundation, Inc., 59 * Temple Place - Suite 330, Boston, MA 02111-1307, USA. * - * Revisions : - * $Log: ssplit.c,v $ - * Revision 1.5 2002/03/24 13:25:43 swa - * name change related issues - * - * Revision 1.4 2001/11/13 00:16:38 jongfoster - * Replacing references to malloc.h with the standard stdlib.h - * (See ANSI or K&R 2nd Ed) - * - * Revision 1.3 2001/05/29 08:54:25 jongfoster - * Rewrote the innards of ssplit() to be easier to understand, - * faster, and to use less memory. Didn't change the interface - * except to give the parameters meaningful names. - * - * Revision 1.2 2001/05/17 23:01:01 oes - * - Cleaned CRLF's from the sources and related files - * - * Revision 1.1.1.1 2001/05/15 13:59:04 oes - * Initial import of version 2.9.3 source tree - * - * *********************************************************************/ - + #include "config.h" #include #include +#include #include "ssplit.h" #include "miscutil.h" -const char ssplit_h_rcs[] = SSPLIT_H_VERSION; - -/* Define this for lots of debugging information to stdout */ -#undef SSPLIT_VERBOSE -/* #define SSPLIT_VERBOSE 1 */ - /********************************************************************* * @@ -83,24 +56,24 @@ const char ssplit_h_rcs[] = SSPLIT_H_VERSION; * 2 : delim = array of delimiters (if NULL, uses " \t"). * 3 : vec[] = results vector (aka. array) [out] * 4 : vec_len = number of usable slots in the vector (aka. array size) - * 5 : dont_save_empty_fields = zero if consecutive delimiters - * give a null output field(s), nonzero if they are just - * to be considered as single delimeter - * 6 : ignore_leading = nonzero to ignore leading field - * separators. * - * Returns : -1 => Error: vec_len is too small to hold all the + * Returns : -1 => Error: vec_len is too small to hold all the * data, or str == NULL. * >=0 => the number of fields put in `vec'. * On error, vec and str may still have been overwritten. * *********************************************************************/ -int ssplit(char *str, const char *delim, char *vec[], int vec_len, - int dont_save_empty_fields, int ignore_leading) +int ssplit(char *str, const char *delim, char *vec[], size_t vec_len) { unsigned char is_delim[256]; unsigned char char_type; int vec_count = 0; + enum char_type { + WANTED = 0, + SEPARATOR = 1, + TERMINATOR = 2, + }; + if (!str) { @@ -119,64 +92,56 @@ int ssplit(char *str, const char *delim, char *vec[], int vec_len, while (*delim) { - is_delim[(unsigned)(unsigned char)*delim++] = 1; /* separator */ + is_delim[(unsigned)(unsigned char)*delim++] = SEPARATOR; } - is_delim[(unsigned)(unsigned char)'\0'] = 2; /* terminator */ - is_delim[(unsigned)(unsigned char)'\n'] = 2; /* terminator */ + is_delim[(unsigned)(unsigned char)'\0'] = TERMINATOR; + is_delim[(unsigned)(unsigned char)'\n'] = TERMINATOR; /* Parse string */ - if (ignore_leading) + /* Skip leading separators. XXX: Why do they matter? */ + while (is_delim[(unsigned)(unsigned char)*str] == SEPARATOR) { - /* skip leading separators */ - while (is_delim[(unsigned)(unsigned char)*str] == 1) - { - str++; - } + str++; } - /* first pointer is the beginning of string */ - /* Check if we want to save this field */ - if ( (!dont_save_empty_fields) - || (is_delim[(unsigned)(unsigned char)*str] == 0) ) - { + /* The first pointer is the beginning of string */ + if (is_delim[(unsigned)(unsigned char)*str] == WANTED) + { /* - * We want empty fields, or the first character in this - * field is not a delimiter or the end of string. - * So save it. + * The first character in this field is not a + * delimiter or the end of string, so save it. */ if (vec_count >= vec_len) { return(-1); /* overflow */ } - vec[vec_count++] = (char *) str; + vec[vec_count++] = str; } - while ((char_type = is_delim[(unsigned)(unsigned char)*str]) != 2) + while ((char_type = is_delim[(unsigned)(unsigned char)*str]) != TERMINATOR) { - if (char_type == 1) + if (char_type == SEPARATOR) { /* the char is a separator */ /* null terminate the substring */ - *str++ = '\0'; + *str++ = '\0'; /* Check if we want to save this field */ - if ( (!dont_save_empty_fields) - || (is_delim[(unsigned)(unsigned char)*str] == 0) ) - { + if (is_delim[(unsigned)(unsigned char)*str] == WANTED) + { /* - * We want empty fields, or the first character in this - * field is not a delimiter or the end of string. - * So save it. + * The first character in this field is not a + * delimiter or the end of string. So save it. */ if (vec_count >= vec_len) { return(-1); /* overflow */ } - vec[vec_count++] = (char *) str; + vec[vec_count++] = str; } } else @@ -184,18 +149,10 @@ int ssplit(char *str, const char *delim, char *vec[], int vec_len, str++; } } - *str = '\0'; /* null terminate the substring */ - -#ifdef SSPLIT_VERBOSE - { - int i; - printf("dump %d strings\n", vec_count); - for (i = 0; i < vec_count; i++) - { - printf("%d '%s'\n", i, vec[i]); - } - } -#endif /* def SSPLIT_VERBOSE */ + /* null terminate the substring */ + /* XXX: this shouldn't be necessary, so assert that it isn't. */ + assert(*str == '\0'); + *str = '\0'; return(vec_count); }