[Anthy-dev 350] uim APIのドキュメンテーションバグ

Back to archive index

YamaKen yamak****@bp*****
2004年 1月 5日 (月) 19:20:29 JST


ヤマケンです。

uim-0.2.2のuim.hでAPI仕様を間違って記述しているコメントがあるの
で修正お願いします。

また、スタイルポリシーとして許容可能であれば返り値にconstを付け
る事によって誤使用を防止できるようになっているとありがたいです。

freedesktop.orgのsvn repositoryはまだアクセスできない状態のよう
なので、uim-0.2.2からの差分を添付します。

-------------------------------
ヤマケン yamak****@bp*****
-------------- next part --------------
diff -urN -x *~ uim-0.2.2.orig/uim/uim.c uim-0.2.2/uim/uim.c
--- uim-0.2.2.orig/uim/uim.c	Mon Dec 29 04:01:47 2003
+++ uim-0.2.2/uim/uim.c	Mon Jan  5 19:01:14 2004
@@ -176,7 +176,7 @@
   return uc->nr_modes;
 }
 
-char *
+const char *
 uim_get_mode_name(uim_context uc, int nth)
 {
   if (nth < uc->nr_modes) {
@@ -303,7 +303,7 @@
   return NULL;
 }
 
-char *
+const char *
 uim_get_im_name(uim_context uc, int nth)
 {
   struct uim_im *im = get_nth_im(uc, nth);
@@ -314,7 +314,7 @@
   return NULL;
 }
 
-char *
+const char *
 uim_get_im_language(uim_context uc, int nth)
 {
   struct uim_im *im = get_nth_im(uc, nth);
diff -urN -x *~ uim-0.2.2.orig/uim/uim.h uim-0.2.2/uim/uim.h
--- uim-0.2.2.orig/uim/uim.h	Sat Dec 20 22:26:24 2003
+++ uim-0.2.2/uim/uim.h	Mon Jan  5 18:57:14 2004
@@ -99,10 +99,10 @@
 uim_reset_context(uim_context uc);
 /* im list */
 int uim_get_nr_im(uim_context uc);
-/* you should free the result */
-char *uim_get_im_name(uim_context uc, int nth);
-/* you should free the result */
-char *uim_get_im_language(uim_context uc, int nth);
+/* you must not free the result */
+const char *uim_get_im_name(uim_context uc, int nth);
+/* you must not free the result */
+const char *uim_get_im_language(uim_context uc, int nth);
 /* 32 <= key <= 126 */
 int
 uim_press_key(uim_context uc, int key, int state);
@@ -112,8 +112,8 @@
 /* mode list */
 int
 uim_get_nr_modes(uim_context uc);
-/* you should free the result */
-char *
+/* you must not free the result */
+const char *
 uim_get_mode_name(uim_context uc, int nth);
 void
 uim_set_mode_list_update_cb(uim_context uc,


Anthy-dev メーリングリストの案内
Back to archive index