Tweak clean_encoding_name() API to avoid need to cast away const.

Kris Jurka

diff --git a/src/backend/utils/mb/encnames.c b/src/backend/utils/mb/encnames.c
index 47a1af6e88c1979f5f6df5fb63e75bf8cfaae12b..07a96dac2c8bf364b038e75b0c254c5a2b1376af 100644 (file)
--- a/src/backend/utils/mb/encnames.c
+++ b/src/backend/utils/mb/encnames.c
@@ -460,10 +460,10 @@ pg_valid_server_encoding(const char *name)
  * ----------
  */
 static char *
-clean_encoding_name(char *key, char *newkey)
+clean_encoding_name(const char *key, char *newkey)
 {
-       char       *p,
-                          *np;
+       const char *p;
+       char       *np;
 
        for (p = key, np = newkey; *p != '\0'; p++)
        {
@@ -508,7 +508,7 @@ pg_char_to_encname_struct(const char *name)
                                 errmsg("encoding name too long")));
 #endif
        }
-       key = clean_encoding_name((char *) name, buff);
+       key = clean_encoding_name(name, buff);
 
        while (last >= base)
        {