changeset 9358:ec6f741a8dc6

Fix various issues with the 'trim' module.
author Bruno Haible <bruno@clisp.org>
date Thu, 18 Oct 2007 12:57:57 +0200
parents 75e43b0e5159
children 568979a9f3d0
files ChangeLog lib/trim.c modules/trim
diffstat 3 files changed, 22 insertions(+), 10 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2007-10-18  Colin Watson <cjwatson@debian.org>  (tiny change)
+            Bruno Haible  <bruno@clisp.org>
+
+	* lib/trim.c: Include config.h unconditionally. Include trim.h always.
+	Include ctype.h always. Include stdlib.h, not mbuiter.h, for MB_CUR_MAX.
+	* modules/trim (Depends-on): Add mbchar.
+	(configure.ac): Add gl_FUNC_MBRTOWC.
+	(Makefile.am): Augment lib_SOURCES.
+
 2007-10-17  Paul Eggert  <eggert@cs.ucla.edu>
 
 	Modify glob.c to use fstatat and dirfd, to simplify it.
--- a/lib/trim.c
+++ b/lib/trim.c
@@ -1,5 +1,5 @@
 /* Removes leading and/or trailing whitespaces
-   Copyright (C) 2006 Free Software Foundation, Inc.
+   Copyright (C) 2006-2007 Free Software Foundation, Inc.
 
    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
@@ -16,21 +16,21 @@
 
 /* Written by Davide Angelocola <davide.angelocola@gmail.com> */
 
-#ifdef HAVE_CONFIG_H
-# include <config.h>
-#endif
+#include <config.h>
+
+/* Specification.  */
+#include "trim.h"
+
+#include <ctype.h>
 
 #if HAVE_MBRTOWC 
 # include <stddef.h>
+# include <stdlib.h>
 # include "mbchar.h"
 # include "mbiter.h"
-# include "mbuiter.h"		/* FIXME: for MB_CUR_MAX */
-#else
-# include <ctype.h>
 #endif
 
 #include "xalloc.h"
-#include "trim.h"
 
 char *
 trim2(const char *s, int how)
@@ -62,7 +62,7 @@
       if (how != TRIM_LEADING) 
 	{
 	  int state = 0;
-	  char *r;
+	  char *r; /* used only while state = 2 */
 	  
 	  mbi_init (i, d, strlen (d));
 
--- a/modules/trim
+++ b/modules/trim
@@ -2,16 +2,19 @@
 trim() removes leading and/or trailing whitespaces
 
 Files:
+lib/trim.h
 lib/trim.c
-lib/trim.h
 
 Depends-on:
 xalloc
+mbchar
 mbiter
 
 configure.ac:
+gl_FUNC_MBRTOWC
 
 Makefile.am:
+lib_SOURCES += trim.c
 
 Include:
 #include "trim.h"