[Armadillo:04674] Re: [a500][hermit][patch] setenv できない文字列がある問題について

[nakai]

nakaiです。

> nakai> これは、多分プロンプト文字列を消さないためのものですよね？
> nakai> こちらも確認してマージさせていただきたいと思います。
>
> はい、ここはその前後 10行しか見ていません。
> CTRL-C とかはケアしてませんが、まぁでも、とりあえず気持ちいいですよ。

こちらも確認してみました。
もう１箇所手をいれるなら、control characterのところに、
{{{
--- command.c.old       2009-10-15 17:03:50.000000000 +0900
+++ command.c   2009-10-15 17:05:16.000000000 +0900
@@ -72,6 +72,8 @@ static void command_loop(const command_l
                                if (interactive) {
                                        hputchar('^');
                                        hputchar(c + '@');
+                                       cmdbuf[n++] = '^';
+                                       cmdbuf[n++] = c + '@';
                                }
                        } else {
                                /* ordinary character; echo. */
}}}
のようにすると、CTRL-Cや方向キーの入力時もケアできますね。
Home/End/PageUp/Downなどはケアできませんが。。



URATAN Shigenobu さんは書きました:
> うらたんです。
> 
> 
> In article <email@hidden>, nakai <email@hidden> writes:
> 
> nakai> 「setenv p」の件、ご報告有難う御座います。
> nakai> 確認してみましたが、確かに変な動作になります。
> 
> 再現しましたか！ よかった。
> 
> 
> nakai> 教えていただきました修正内容ですが、問題ないと思います。
> nakai> パラメータ文字列先頭アドレスを取得する関数なので、
> nakai> Flashメモリ上のアドレスに'\0'を代入する必要がありませんでした。
> ...
> nakai> 本不具合は、次回のアップデート時に修正したいと思います。
> 
> 私は param.c の中しか見てないので 後をよろしくお願いいたします。
> # 最初 src/target/setenv.c を目を皿にして見ていたのはナイショ。
> 
> 
> In article <email@hidden>, nakai <email@hidden> writes:
> 
> nakai> これは、多分プロンプト文字列を消さないためのものですよね？
> nakai> こちらも確認してマージさせていただきたいと思います。
> 
> はい、ここはその前後 10行しか見ていません。
> CTRL-C とかはケアしてませんが、まぁでも、とりあえず気持ちいいですよ。
> 
> 
> 以上ですが、よろしくお願いいたします。