lexer: Rename 'close' to 'destroy' in struct lex_reader_class.

This seems like a cleaner solution to the problem of Gnulib
sometimes defining 'close' as a macro than to add an extra
include file and a big comment.

diff --git a/src/language/lexer/lexer.c b/src/language/lexer/lexer.c
index 686aafd0d4c99fecc640828f2567a99fb414868c..5f1cc4ba2bb4709c7affce0f31479635e79c50f0 100644 (file)
--- a/src/language/lexer/lexer.c
+++ b/src/language/lexer/lexer.c
@@ -1452,8 +1452,8 @@ static void
 lex_source_destroy (struct lex_source *src)
 {
   char *file_name = src->reader->file_name;
-  if (src->reader->class->close != NULL)
-    src->reader->class->close (src->reader);
+  if (src->reader->class->destroy != NULL)
+    src->reader->class->destroy (src->reader);
   free (file_name);
   free (src->buffer);
   while (!deque_is_empty (&src->deque))

diff --git a/src/language/lexer/lexer.h b/src/language/lexer/lexer.h
index b35ea7f54e05d9da00ffbcc23b18d71ebb174b97..0a3e6c37c03de317a25f107820dd313ef8f4bd7e 100644 (file)
--- a/src/language/lexer/lexer.h
+++ b/src/language/lexer/lexer.h
@@ -20,13 +20,6 @@
 #include <stdbool.h>
 #include <stddef.h>
 
-/* The following #include avoids a potential problem when Gnulib substitutes
- * for close() by putting "#define close rpl_close" into <unistd.h>, by
- * ensuring that every source file that includes this one sees the #define.
- * (It would probably be better to rename the 'close' member of struct
- * lex_reader_class.)  */
-#include <unistd.h>
-
 #include "data/identifier.h"
 #include "data/variable.h"
 #include "libpspp/compiler.h"
@@ -79,7 +72,7 @@ struct lex_reader_class
 
        The caller will free the 'file_name' member of READER, so the
        implementation should not do so. */
-    void (*close) (struct lex_reader *reader);
+    void (*destroy) (struct lex_reader *reader);
   };
 
 /* Helper functions for lex_reader. */