libc: Remove unused (and public) locale_isstrict()
authorJohn Marino <draco@marino.st>
Sun, 25 Oct 2015 07:19:14 +0000 (08:19 +0100)
committerJohn Marino <draco@marino.st>
Sun, 25 Oct 2015 07:29:31 +0000 (08:29 +0100)
The publicly exposed locale_isstrict function is not hooked into anything.
It was intended to implement LOCALE_STRICT environment variable handling,
but it only returns a boolean based on the value of LOCALE_STRICT, and not
even efficiently.

There is also a private "strict_mode" variable in locale_private.h that
apparently was intended to be used with LOCALE_STRICT, but it's also
seemingly useless.

The LOCALE_STRICT override still needs to be implemented if UTF-8B is
in use, but it can be handled at encoding initialization more
efficiently later.

include/locale.h
lib/libc/locale/Makefile.inc
lib/libc/locale/Symbol.map
lib/libc/locale/locale_isstrict.c [deleted file]
lib/libc/locale/xlocale_private.h

index e6d724f..05bc37a 100644 (file)
@@ -77,7 +77,6 @@ struct lconv {
 __BEGIN_DECLS
 struct lconv   *localeconv(void);
 char           *setlocale(int, const char *);
-int            locale_isstrict(void);
 
 #if __POSIX_VISIBLE >= 200809
 #include <xlocale/_locale.h>
index b394524..1264482 100644 (file)
@@ -14,7 +14,7 @@ SRCS+=        big5.c btowc.c collate.c collcmp.c euc.c fix_grouping.c \
        mbintowcr.c \
        mskanji.c nextwctype.c nl_langinfo.c nomacros.c none.c rpmatch.c \
        rune.c \
-       runetype.c setlocale.c setrunelocale.c locale_isstrict.c \
+       runetype.c setlocale.c setrunelocale.c \
        table.c \
        tolower.c toupper.c utf8.c wcrtomb.c wcsnrtombs.c \
        wcsrtombs.c wcsftime.c \
index 6aa26af..7a77ca2 100644 (file)
@@ -117,7 +117,6 @@ DF404.0 {
        isxdigit_l;
        localeconv;
        localeconv_l;
-       locale_isstrict;
        mbintowcr;
        mbintowcr_l;
        mblen;
diff --git a/lib/libc/locale/locale_isstrict.c b/lib/libc/locale/locale_isstrict.c
deleted file mode 100644 (file)
index 96d4a9f..0000000
+++ /dev/null
@@ -1,52 +0,0 @@
-/*
- * Copyright (c) 2015 The DragonFly Project.  All rights reserved.
- *
- * This code is derived from software contributed to The DragonFly Project
- * by Matthew Dillon <dillon@backplane.com>
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- *
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in
- *    the documentation and/or other materials provided with the
- *    distribution.
- * 3. Neither the name of The DragonFly Project nor the names of its
- *    contributors may be used to endorse or promote products derived
- *    from this software without specific, prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
- * FOR A PARTICULAR PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE
- * COPYRIGHT HOLDERS OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
- * INCIDENTAL, SPECIAL, EXEMPLARY OR CONSEQUENTIAL DAMAGES (INCLUDING,
- * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
- * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED
- * AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
- * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT
- * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
- * SUCH DAMAGE.
- */
-
-#include <locale.h>
-#include <stdlib.h>
-
-static int locale_is_strict = -1;
-
-int
-locale_isstrict(void)
-{
-       if (locale_is_strict < 0) {
-               const char *env;
-
-               if ((env = getenv("LOCALE_STRICT")) != NULL)
-                       locale_is_strict = *env ? atoi(env) : 1;
-               else
-                       locale_is_strict = 0;
-       }
-       return locale_is_strict;
-}
index 37b4c97..7781818 100644 (file)
@@ -135,8 +135,6 @@ struct _xlocale {
        __mbstate_t wctomb;
        /** Buffer used by nl_langinfo_l() */
        char *csym;
-       /** strict mode from LOCALE_STRICT */
-       int     strict_mode;
 };
 
 /**