dovecot-2.2: lib-fts: Removed supports() function from filter API

[dovecot at dovecot.org]

details:   http://hg.dovecot.org/dovecot-2.2/rev/e2a0b43e73ef
changeset: 18591:e2a0b43e73ef
user:      Timo Sirainen <tss at iki.fi>
date:      Sat May 09 15:00:28 2015 +0300
description:
lib-fts: Removed supports() function from filter API
I think the original idea was that this could be used to automatically
create filters for many languages, but this probably won't be needed or
wanted.

diffstat:

 src/lib-fts/fts-filter-lowercase.c        |   7 -------
 src/lib-fts/fts-filter-normalizer-icu.c   |  16 +---------------
 src/lib-fts/fts-filter-private.h          |   1 -
 src/lib-fts/fts-filter-stemmer-snowball.c |  17 -----------------
 src/lib-fts/fts-filter-stopwords.c        |  10 ----------
 5 files changed, 1 insertions(+), 50 deletions(-)

diffs (149 lines):

diff -r 12aeb1ac8f0d -r e2a0b43e73ef src/lib-fts/fts-filter-lowercase.c
--- a/src/lib-fts/fts-filter-lowercase.c	Sat May 09 14:56:33 2015 +0300
+++ b/src/lib-fts/fts-filter-lowercase.c	Sat May 09 15:00:28 2015 +0300
@@ -6,12 +6,6 @@
 #include "fts-filter-private.h"
 #include "fts-language.h"
 
-static bool
-fts_filter_lowercase_supports(const struct fts_language *lang ATTR_UNUSED)
-{
-	return TRUE;
-}
-
 static void
 fts_filter_lowercase_destroy(struct fts_filter *filter)
 {
@@ -47,7 +41,6 @@
 }
 
 static const struct fts_filter_vfuncs normalizer_filter_vfuncs = {
-	fts_filter_lowercase_supports,
 	fts_filter_lowercase_create,
 	fts_filter_lowercase_filter,
 	fts_filter_lowercase_destroy
diff -r 12aeb1ac8f0d -r e2a0b43e73ef src/lib-fts/fts-filter-normalizer-icu.c
--- a/src/lib-fts/fts-filter-normalizer-icu.c	Sat May 09 14:56:33 2015 +0300
+++ b/src/lib-fts/fts-filter-normalizer-icu.c	Sat May 09 15:00:28 2015 +0300
@@ -114,13 +114,6 @@
 	*_dst = dst;
 }
 
-static bool fts_filter_normalizer_icu_supports(const struct fts_language *lang)
-{
-	if (lang == NULL || lang->name == NULL)
-		return FALSE;
-	return TRUE;
-}
-
 static void fts_filter_normalizer_icu_destroy(struct fts_filter *filter)
 {
 	struct fts_filter_normalizer *np =
@@ -242,19 +235,13 @@
 
 #else
 
-static bool
-fts_filter_normalizer_icu_supports(const struct fts_language *lang ATTR_UNUSED)
-{
-	return FALSE;
-}
-
 static int
 fts_filter_normalizer_icu_create(const struct fts_language *lang ATTR_UNUSED,
 				 const char *const *settings ATTR_UNUSED,
 				 struct fts_filter **filter_r ATTR_UNUSED,
 				 const char **error_r)
 {
-	*error_r = "libicu support not built in - can't use "ICU_NORMALIZER_FILTER_NAME;
+	*error_r = "libicu support not built in - can't use "ICU_NORMALIZER_FILTER_NAME;
 	return -1;
 }
 
@@ -274,7 +261,6 @@
 #endif
 
 static const struct fts_filter_vfuncs normalizer_filter_vfuncs = {
-	fts_filter_normalizer_icu_supports,
 	fts_filter_normalizer_icu_create,
 	fts_filter_normalizer_icu_filter,
 	fts_filter_normalizer_icu_destroy
diff -r 12aeb1ac8f0d -r e2a0b43e73ef src/lib-fts/fts-filter-private.h
--- a/src/lib-fts/fts-filter-private.h	Sat May 09 14:56:33 2015 +0300
+++ b/src/lib-fts/fts-filter-private.h	Sat May 09 15:00:28 2015 +0300
@@ -12,7 +12,6 @@
 
 */
 struct fts_filter_vfuncs {
-	bool (*supports)(const struct fts_language *lang);
 	int (*create)(const struct fts_language *lang,
 	              const char *const *settings,
 	              struct fts_filter **filter_r,
diff -r 12aeb1ac8f0d -r e2a0b43e73ef src/lib-fts/fts-filter-stemmer-snowball.c
--- a/src/lib-fts/fts-filter-stemmer-snowball.c	Sat May 09 14:56:33 2015 +0300
+++ b/src/lib-fts/fts-filter-stemmer-snowball.c	Sat May 09 15:00:28 2015 +0300
@@ -16,17 +16,6 @@
 	struct sb_stemmer *stemmer;
 };
 
-static bool
-fts_filter_stemmer_snowball_supports(const struct fts_language *lang)
-{
-	struct sb_stemmer *stemmer = sb_stemmer_new(lang->name, NULL);
-	if (stemmer != NULL) {
-		sb_stemmer_delete(stemmer);
-		return TRUE;
-	}
-	return FALSE;
-}
-
 static void fts_filter_stemmer_snowball_destroy(struct fts_filter *filter)
 {
 	struct fts_filter_stemmer_snowball *sp =
@@ -105,11 +94,6 @@
 
 #else
 
-static bool
-fts_filter_stemmer_snowball_supports(const struct fts_language *lang ATTR_UNUSED)
-{
-	return FALSE;
-}
 static int
 fts_filter_stemmer_snowball_create(const struct fts_language *lang ATTR_UNUSED,
                                    const char *const *settings ATTR_UNUSED,
@@ -134,7 +118,6 @@
 
 #endif
 static const struct fts_filter_vfuncs snowball_stemmer_filter_vfuncs = {
-	fts_filter_stemmer_snowball_supports,
 	fts_filter_stemmer_snowball_create,
 	fts_filter_stemmer_snowball_filter,
 	fts_filter_stemmer_snowball_destroy
diff -r 12aeb1ac8f0d -r e2a0b43e73ef src/lib-fts/fts-filter-stopwords.c
--- a/src/lib-fts/fts-filter-stopwords.c	Sat May 09 14:56:33 2015 +0300
+++ b/src/lib-fts/fts-filter-stopwords.c	Sat May 09 15:00:28 2015 +0300
@@ -23,15 +23,6 @@
 	const char *stopwords_dir;
 };
 
-/* TODO: Write this function or remove it from api. */
-static bool fts_filter_stopwords_supports(const struct fts_language *lang)
-{
-	/* TODO: former NULL check is for unit test _fail_create() */
-	if (lang == NULL || lang->name == NULL)
-		return FALSE;
-	return TRUE;
-}
-
 static int fts_filter_stopwords_read_list(struct fts_filter_stopwords *filter,
 					  const char **error_r)
 {
@@ -124,7 +115,6 @@
 }
 
 const struct fts_filter_vfuncs stopwords_filter_vfuncs = {
-	fts_filter_stopwords_supports,
 	fts_filter_stopwords_create,
 	fts_filter_stopwords_filter,
 	fts_filter_stopwords_destroy