* keywords.h, keywords.c, deprecated.h, deprecated.c

(SCM_KEYWORDP, SCM_KEYWORDSYM): Deprecated and implemented in
terms of scm_is_keyword and scm_keyword_dash_symbol.

* keywords.h, keywords.c, discouraged.h, discouraged.c
(scm_make_keyword_from_dash_symbol, scm_keyword_dash_symbol,
scm_c_make_keyword): Discouraged.

* keywords.h, keywords.c (scm_symbol_to_keyword,
scm_keyword_to_symbol): Implemented in C.
(scm_is_keyword, scm_from_locale_keyword,
scm_from_locale_keywordn): New.

libguile/deprecated.c

@@ -1192,6 +1192,22 @@ SCM_SYMBOL_LENGTH (SCM sym)
   return scm_i_symbol_length (sym);
 }
 
+int
+SCM_KEYWORDP (SCM obj)
+{
+  scm_c_issue_deprecation_warning
+    ("SCM_KEYWORDP is deprecated.  Use scm_is_keyword instead.");
+  return scm_is_keyword (obj);
+}
+
+SCM
+SCM_KEYWORDSYM (SCM keyword);
+{
+  scm_c_issue_deprecation_warning
+    ("SCM_KEYWORDSYM is deprecated.  See scm_keyword_to_symbol instead.");
+  return scm_keyword_dash_symbol (keyword);
+}
+
 void
 scm_i_init_deprecated ()
 {

libguile/deprecated.h

@@ -447,6 +447,14 @@ SCM_API double scm_round (double x);
 SCM_API char *SCM_SYMBOL_CHARS (SCM sym);
 SCM_API size_t SCM_SYMBOL_LENGTH (SCM sym);
 
+/* Deprecated because the macros used to evaluate the arguments more
+   than once and because the symbol of a keyword now has no dash.
+*/
+
+SCM_API int SCM_KEYWORDP (SCM obj);
+SCM_API SCM SCM_KEYWORDSYM (SCM keyword);
+
+
 void scm_i_init_deprecated (void);
 
 #endif

libguile/discouraged.c

@@ -156,6 +156,47 @@ scm_allocate_string (size_t len)
   return scm_i_make_string (len, NULL);
 }
 
+SCM_DEFINE (scm_make_keyword_from_dash_symbol, "make-keyword-from-dash-symbol", 1, 0, 0, 
+            (SCM symbol),
+            "Make a keyword object from a @var{symbol} that starts with a dash.")
+#define FUNC_NAME s_scm_make_keyword_from_dash_symbol
+{
+  SCM dash_string, non_dash_symbol;
+
+  SCM_ASSERT (scm_is_symbol (symbol)
+	      && ('-' == scm_i_symbol_chars(symbol)[0]),
+	      symbol, SCM_ARG1, FUNC_NAME);
+
+  dash_string = scm_symbol_to_string (symbol);
+  non_dash_symbol =
+    scm_string_to_symbol (scm_c_substring (dash_string,
+					   1,
+					   scm_c_string_length (dash_string)));
+
+  return scm_symbol_to_keyword (non_dash_symbol);
+}
+#undef FUNC_NAME
+
+SCM_DEFINE (scm_keyword_dash_symbol, "keyword-dash-symbol", 1, 0, 0, 
+            (SCM keyword),
+	    "Return the dash symbol for @var{keyword}.\n"
+	    "This is the inverse of @code{make-keyword-from-dash-symbol}.")
+#define FUNC_NAME s_scm_keyword_dash_symbol
+{
+  SCM symbol = scm_keyword_to_symbol (keyword);
+  SCM parts = scm_list_2 (scm_from_locale_string ("-"),
+			  scm_symbol_to_string (symbol));
+  return scm_string_to_symbol (scm_string_append (parts));
+}
+#undef FUNC_NAME
+
+SCM
+scm_c_make_keyword (const char *s)
+{
+  return scm_from_locale_keyword (s);
+}
+
+
 void
 scm_i_init_discouraged (void)
 {

libguile/discouraged.h

@@ -157,6 +157,18 @@ SCM_API SCM scm_allocate_string (size_t len);
 #define SCM_NULLP(x)		(scm_is_null (x))
 #define SCM_NNULLP(x)		(!scm_is_null (x))
 
+/* Discouraged because they are just strange.
+ */
+
+SCM_API SCM scm_make_keyword_from_dash_symbol (SCM symbol);
+SCM_API SCM scm_keyword_dash_symbol (SCM keyword);
+
+/* Discouraged because it does not state what encoding S is in.
+ */
+
+SCM_API SCM scm_c_make_keyword (const char *s);
+
+
 void scm_i_init_discouraged (void);
 
 #endif /* SCM_ENABLE_DISCOURAGED == 1 */

libguile/keywords.c

@@ -34,29 +34,35 @@
 
 scm_t_bits scm_tc16_keyword;
 
+#define KEYWORDP(X)	(SCM_SMOB_PREDICATE (scm_tc16_keyword, (X)))
+#define KEYWORDSYM(X)	(SCM_SMOB_OBJECT (X))
+
 static int
 keyword_print (SCM exp, SCM port, scm_print_state *pstate SCM_UNUSED)
 {
-  SCM symbol = SCM_KEYWORDSYM (exp);
-
   scm_puts ("#:", port);
-  scm_print_symbol_name (scm_i_symbol_chars (symbol) + 1,
-			 scm_i_symbol_length (symbol) - 1,
-			 port);
-  scm_remember_upto_here_1 (symbol);
+  scm_display (KEYWORDSYM (exp), port);
   return 1;
 }
 
-SCM_DEFINE (scm_make_keyword_from_dash_symbol, "make-keyword-from-dash-symbol", 1, 0, 0, 
-            (SCM symbol),
-            "Make a keyword object from a @var{symbol} that starts with a dash.")
-#define FUNC_NAME s_scm_make_keyword_from_dash_symbol
+SCM_DEFINE (scm_keyword_p, "keyword?", 1, 0, 0, 
+            (SCM obj),
+	    "Return @code{#t} if the argument @var{obj} is a keyword, else\n"
+	    "@code{#f}.")
+#define FUNC_NAME s_scm_keyword_p
+{
+  return scm_from_bool (KEYWORDP (obj));
+}
+#undef FUNC_NAME
+
+SCM_DEFINE (scm_symbol_to_keyword, "symbol->keyword", 1, 0, 0,
+	    (SCM symbol),
+	    "Return the keyword with the same name as @var{symbol}.")
+#define FUNC_NAME s_scm_symbol_to_keyword
 {
   SCM keyword;
 
-  SCM_ASSERT (scm_is_symbol (symbol)
-	      && ('-' == scm_i_symbol_chars(symbol)[0]),
-	      symbol, SCM_ARG1, FUNC_NAME);
+  SCM_ASSERT (scm_is_symbol (symbol), symbol, 0, NULL);
 
   SCM_DEFER_INTS;
   keyword = scm_hashq_ref (scm_keyword_obarray, symbol, SCM_BOOL_F);
@@ -70,43 +76,33 @@ SCM_DEFINE (scm_make_keyword_from_dash_symbol, "make-keyword-from-dash-symbol",
 }
 #undef FUNC_NAME
 
+SCM_DEFINE (scm_keyword_to_symbol, "keyword->symbol", 1, 0, 0,
+	    (SCM keyword),
+	    "Return the symbol with the same name as @var{keyword}.")
+#define FUNC_NAME s_scm_keyword_to_symbol
+{
+  scm_assert_smob_type (scm_tc16_keyword, keyword);
+  return KEYWORDSYM (keyword);
+}
+#undef FUNC_NAME
+
+int
+scm_is_keyword (SCM val)
+{
+  return KEYWORDP (val);
+}
+
 SCM
-scm_c_make_keyword (char *s)
+scm_from_locale_keyword (const char *str)
 {
-  char *buf;
-  size_t len;
-  SCM string, symbol;
-
-  len = strlen (s) + 1;
-  string = scm_i_make_string (len, &buf);
-  buf[0] = '-';
-  strcpy (buf + 1, s);
-  symbol = scm_string_to_symbol (string);
-  return scm_make_keyword_from_dash_symbol (symbol);
+  return scm_symbol_to_keyword (scm_from_locale_symbol (str));
 }
 
-SCM_DEFINE (scm_keyword_p, "keyword?", 1, 0, 0, 
-            (SCM obj),
-	    "Return @code{#t} if the argument @var{obj} is a keyword, else\n"
-	    "@code{#f}.")
-#define FUNC_NAME s_scm_keyword_p
+SCM
+scm_from_locale_keywordn (const char *str, size_t len)
 {
-  return scm_from_bool (SCM_KEYWORDP (obj));
+  return scm_symbol_to_keyword (scm_from_locale_symboln (str, len));
 }
-#undef FUNC_NAME
-
-
-SCM_DEFINE (scm_keyword_dash_symbol, "keyword-dash-symbol", 1, 0, 0, 
-            (SCM keyword),
-	    "Return the dash symbol for @var{keyword}.\n"
-	    "This is the inverse of @code{make-keyword-from-dash-symbol}.")
-#define FUNC_NAME s_scm_keyword_dash_symbol
-{
-  SCM_VALIDATE_KEYWORD (1, keyword);
-  return SCM_KEYWORDSYM (keyword);
-}
-#undef FUNC_NAME
-
 
 
 void

libguile/keywords.h

@@ -28,16 +28,16 @@
 
 SCM_API scm_t_bits scm_tc16_keyword;
 
-#define SCM_KEYWORDP(X)		(SCM_SMOB_PREDICATE (scm_tc16_keyword, (X)))
-#define SCM_KEYWORDSYM(X)	(SCM_SMOB_OBJECT (X))
-
 
 
-
-SCM_API SCM scm_make_keyword_from_dash_symbol (SCM symbol);
-SCM_API SCM scm_c_make_keyword (char *s);
 SCM_API SCM scm_keyword_p (SCM obj);
-SCM_API SCM scm_keyword_dash_symbol (SCM keyword);
+SCM_API SCM scm_symbol_to_keyword (SCM symbol);
+SCM_API SCM scm_keyword_to_symbol (SCM keyword);
+
+SCM_API int scm_is_keyword (SCM val);
+SCM_API SCM scm_from_locale_keyword (const char *str);
+SCM_API SCM scm_from_locale_keywordn (const char *str, size_t len);
+
 SCM_API void scm_init_keywords (void);
 
 #endif  /* SCM_KEYWORDS_H */