From: Bruno Haible <bruno@clisp.org>
Date: Thu, 26 Jun 2025 20:46:40 +0000 (+0200)
Subject: kwset: Add specification comments in .h file.
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=46ac1afa57f64c0e0c9c266f4e405928f25978fd;p=thirdparty%2Fgnulib.git

kwset: Add specification comments in .h file.

* lib/kwset.h (kwsalloc, kwsincr, kwsprep, kwsexec, kwsfree): Move
specification to here...
* lib/kwset.c (kwsalloc, kwsincr, kwsprep, kwsexec, kwsfree): ...from
here.
---

diff --git a/ChangeLog b/ChangeLog
index 30dabe44e3..a4597155e4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2025-06-26  Bruno Haible  <bruno@clisp.org>
+
+	kwset: Add specification comments in .h file.
+	* lib/kwset.h (kwsalloc, kwsincr, kwsprep, kwsexec, kwsfree): Move
+	specification to here...
+	* lib/kwset.c (kwsalloc, kwsincr, kwsprep, kwsexec, kwsfree): ...from
+	here.
+
 2025-06-26  Paul Eggert  <eggert@cs.ucla.edu>
 
 	regex: fix parse_bracket_exp double-free
diff --git a/lib/kwset.c b/lib/kwset.c
index 2784ba5371..f90b01dd34 100644
--- a/lib/kwset.c
+++ b/lib/kwset.c
@@ -148,9 +148,6 @@ static ptrdiff_t acexec (kwset_t, char const *, idx_t,
 static ptrdiff_t bmexec (kwset_t, char const *, idx_t,
                          struct kwsmatch *, bool);
 
-/* Return a newly allocated keyword set.  A nonnull TRANS specifies a
-   table of character translations to be applied to all pattern and
-   search text.  */
 kwset_t
 kwsalloc (char const *trans)
 {
@@ -178,7 +175,6 @@ kwsalloc (char const *trans)
    exact for CHAR_BIT in { 4..11, 13, 15, 17, 19 }.  */
 enum { DEPTH_SIZE = CHAR_BIT + CHAR_BIT / 2 };
 
-/* Add the given string to the contents of the keyword set.  */
 void
 kwsincr (kwset_t kwset, char const *text, idx_t len)
 {
@@ -423,7 +419,6 @@ treenext (struct tree const *tree, struct trie *next[])
   next[tree->label] = tree->trie;
 }
 
-/* Prepare a built keyword set for use.  */
 void
 kwsprep (kwset_t kwset)
 {
@@ -914,16 +909,6 @@ acexec (kwset_t kwset, char const *text, idx_t size,
            : acexec_trans (kwset, text, size, kwsmatch, longest)));
 }
 
-/* Find the first instance of a KWSET member in TEXT, which has SIZE bytes.
-   Return the offset (into TEXT) of the first byte of the matching substring,
-   or -1 if no match is found.
-   Upon a match:
-     - Store details in *KWSMATCH: index of matched keyword, start offset
-       (same as the return value), and length.
-     - If LONGEST, find the longest match that starts at this offset;
-       otherwise any match that starts at this offset will do.
-   NOTE! LONGEST does *not* mean to search for the longest KWSET member
-   across the entire string.  */
 ptrdiff_t
 kwsexec (kwset_t kwset, char const *text, idx_t size,
          struct kwsmatch *kwsmatch, bool longest)
@@ -931,7 +916,6 @@ kwsexec (kwset_t kwset, char const *text, idx_t size,
   return kwset->kwsexec (kwset, text, size, kwsmatch, longest);
 }
 
-/* Free the components of the given keyword set.  */
 void
 kwsfree (kwset_t kwset)
 {
diff --git a/lib/kwset.h b/lib/kwset.h
index 204f409808..223e077144 100644
--- a/lib/kwset.h
+++ b/lib/kwset.h
@@ -44,14 +44,35 @@ struct kwsmatch
 struct kwset;
 typedef struct kwset *kwset_t;
 
-extern kwset_t kwsalloc (char const *);
-extern void kwsincr (kwset_t, char const *, idx_t);
-extern idx_t kwswords (kwset_t) _GL_ATTRIBUTE_PURE;
-extern void kwsprep (kwset_t);
-extern ptrdiff_t kwsexec (kwset_t, char const *, idx_t,
-                          struct kwsmatch *, bool)
+/* Return a newly allocated keyword set.  A nonnull TRANS specifies a
+   table of character translations to be applied to all pattern and
+   search text.  */
+extern kwset_t kwsalloc (char const *trans);
+
+/* Add the given string to the contents of the keyword set.  */
+extern void kwsincr (kwset_t kwset, char const *text, idx_t len);
+
+extern idx_t kwswords (kwset_t kwset) _GL_ATTRIBUTE_PURE;
+
+/* Prepare a built keyword set for use.  */
+extern void kwsprep (kwset_t kwset);
+
+/* Find the first instance of a KWSET member in TEXT, which has SIZE bytes.
+   Return the offset (into TEXT) of the first byte of the matching substring,
+   or -1 if no match is found.
+   Upon a match:
+     - Store details in *MATCH: index of matched keyword, start offset
+       (same as the return value), and length.
+     - If LONGEST, find the longest match that starts at this offset;
+       otherwise any match that starts at this offset will do.
+   NOTE! LONGEST does *not* mean to search for the longest KWSET member
+   across the entire string.  */
+extern ptrdiff_t kwsexec (kwset_t kwset, char const *text, idx_t size,
+                          struct kwsmatch *match, bool longest)
   _GL_ARG_NONNULL ((4));
-extern void kwsfree (kwset_t);
+
+/* Free the components of the given keyword set.  */
+extern void kwsfree (kwset_t kwset);
 
 #ifdef __cplusplus
 }