mlterm-dev-ja Mailing List for mlterm(Multi Lingual TERMinal emulator)

Hi,

I've committed a following change.

	* zero-clear struct termios in kiklib/src/kik_pty_*.c

Now termios params that are not changed by mlterm / inherited from parent process
became zero.

regards,

minami

Hi,

I've committed follwing changes.

	* refined XDND support: text/uri-list is also accepted now.

	* minor bugfix: While parsing termcap, line[-1] was accessed
	 when an empty line was read.

	* plugged leak: window->ch_gc was not freed.

# According to valgrind (http://developer.kde.org/~sewardj/),
# mlterm itself is now free from memory leaks at least in my environment :)


regards,

minami

Hi All,

I have committed a fix for the incorrect size hints for the window.

Thanks,

--
Seiichi SATO

Hi,

I've committed support for window icons. Both of the WMHints style and
the "extended window manager hint" style icon are generated.

An icon for mlterm window is now can be specified using "icon_path" entry
in .mlterm/main like this:

icon_path = /usr/share/pixmaps/mlterm.png

To use this feature, you have to configure mlterm with gdk-pixbuf.
i.e. specify --enable-gdk-pixbuf and do NOT specify --enable-imlib.

regards,

minami

Hi,

 I'v committed following changes to CVS.

	* Background picture was not drawn when pseudo transparency had been
	  turned off. fixed.
	  (Thanks to Kubota Tomohiro san)
 
	* "winsize" structure were not fully initialized during pty setup. fixed.
	  (Thanks to Nanashi san) (!)

	* Compilation failed on Linux glibc 2.1.x because of incomplete langinfo.h.
	  Built-in (dummy) kik_langinfo is now used when CODESET is not defined.

(!)This fix enables mutt/slrn users to display X-Faces using slrnface. 

regards,

minami

南です。

ioctl(TIOCSWINSZ) の引数 struct winsize ですが、
現在は ws_col/ws_row のみ値を設定されています。

大抵の環境では winsize は
struct winsize {
        unsigned short ws_row;
        unsigned short ws_col;
        unsigned short ws_xpixel;
        unsigned short ws_ypixel;
};
だと思われるので、ws_xpixel/ypixel には未初期化の値が使われてることになります。 

しかし、端末の大きさとしてこれらの値を信用するアプリケーションが存在しており、
このため環境によっては問題がおきているようです。

とりあえず下のパッチで slrnface(http://dave.willfork.com/slrnface/)
は動作するらしいのですが、これでは問題のおきる環境がありましたら教えてください。

Index: mlterm/ml_pty.c
===================================================================
RCS file: /cvsroot/mlterm/mlterm/mlterm/ml_pty.c,v
retrieving revision 1.3
diff -u -B -r1.3 ml_pty.c
--- mlterm/ml_pty.c     28 Jul 2002 15:55:39 -0000      1.3
+++ mlterm/ml_pty.c     26 Aug 2002 07:02:17 -0000
@@ -167,6 +167,8 @@

        ws.ws_col = cols ;
        ws.ws_row = rows ;
+       ws.ws_xpixel = 0 ;
+       ws.ws_ypixel = 0 ;

        if( ioctl( pty->master , TIOCSWINSZ , &ws) < 0)
        {

南です。

On Sat, 24 Aug 2002 02:05:51 +0900
Seiichi SATO <ss...@sh...> wrote:

> extended window manager hint に対応していない window manager (Window Maker,
> FluxBox, FVWM) でテストしました。

ありがとうございます。

> フォントのサイズを変更した時に、落ちるようです。

すみません。判定を間違えてました。
>                    if (!*prop)
を if(!prop) で修正できると思います。 

佐藤です。

On Sun, Aug 18, 2002 at 07:46:03PM +0900,
 MINAMI Hirokazu wrote:

> extended window manager hint にいくらか対応するパッチ
> (アイコンのサポートと最大化の制御)を書いてみました。
> 
> どちらもwindow manager側での対応が必要なので、どの環境で
> きちんと動作するかは把握できてません。最新のCVSへの差分を
>   http://minami.unlimeted.com/mlterm/mlterm-ewmh.0818.diff
> に置いておくので、テストしていただけるとありがたいです。

extended window manager hint に対応していない window manager (Window Maker,
FluxBox, FVWM) でテストしました。
フォントのサイズを変更した時に、落ちるようです。

# アイコンの方は、問題ありませんでした。


(gdb) run
Starting program: /usr/local/bin/mlterm

Program received signal SIGSEGV, Segmentation fault.
0x080557a3 in x_window_is_maximized (win=0x80fd240) at x_window.c:1175
1175                    if (!*prop)
(gdb) bt
#0  0x080557a3 in x_window_is_maximized (win=0x80fd240) at x_window.c:1175
#1  0x0805583d in x_window_remaximize (win=0x80fc028) at x_window.c:1197
#2  0x0805b71c in font_size_changed (screen=0x80fc028) at x_screen.c:3937
#3  0x0805b778 in change_font_size (screen=0x80fc028, font_size=12)
    at x_screen.c:3976
#4  0x0805c41b in set_config (p=0x80fc028, key=0x80f0d5f "fontsize",
    value=0x80f0d68 "12") at x_screen.c:4778
#5  0x08072dc4 in parse_vt100_escape_sequence (vt100_parser=0x80f0d58)
    at ml_vt100_parser.c:1960
#6  0x08073292 in ml_parse_vt100_sequence (vt100_parser=0x80f0d58)
    at ml_vt100_parser.c:2314
#7  0x08070d1c in ml_term_parse_vt100_sequence (term=0x80c49e8)
    at ml_term.c:173
#8  0x08069044 in receive_next_event () at x_term_manager.c:1801
#9  0x08069b5c in x_term_manager_event_loop () at x_term_manager.c:2324
#10 0x08066b65 in main (argc=1, argv=0xbffffc04) at main.c:53
(gdb) list
1170                            XFree(prop);
1171                    return 0;
1172            }
1173            else
1174            {
1175                    if (!*prop)
1176                            return 0;
1177                    while(nitems>0){
1178                            nitems--;
1179                            if (((Atom *)prop)[nitems] == XA_NET_WM_STATE_MAXIMIZED_HORZ(win->display))
(gdb) print *prop
Cannot access memory at address 0x0
(gdb) print prop
$1 = (unsigned char *) 0x0

--
佐藤精一

南です。

linux glibc 2.1.x では、 langinfo.h は存在するものの
CODESET が未定義なために kiklib のコンパイルに失敗するようです。

こんな対処でどうかと思うのですが、いかがでしょう。

Index: kiklib/src/kik_langinfo.c
===================================================================
RCS file: /cvsroot/mlterm/mlterm/kiklib/src/kik_langinfo.c,v
retrieving revision 1.3
diff -u -B -r1.3 kik_langinfo.c
--- kiklib/src/kik_langinfo.c	2 Feb 2002 09:20:27 -0000	1.3
+++ kiklib/src/kik_langinfo.c	19 Aug 2002 14:02:58 -0000
@@ -7,7 +7,7 @@
 
 /* --- global functions --- */
 
-#ifndef  HAVE_LANGINFO_H
+#ifdef  USE_BUILTIN_LANGINFO
 
 char *
 __kik_langinfo(
Index: kiklib/src/kik_langinfo.h
===================================================================
RCS file: /cvsroot/mlterm/mlterm/kiklib/src/kik_langinfo.h,v
retrieving revision 1.3
diff -u -B -r1.3 kik_langinfo.h
--- kiklib/src/kik_langinfo.h	2 Feb 2002 09:20:27 -0000	1.3
+++ kiklib/src/kik_langinfo.h	19 Aug 2002 14:02:58 -0000
@@ -13,6 +13,10 @@
 
 #include  <langinfo.h>
 
+#endif
+
+#ifdef CODESET 
+
 #define  kik_langinfo( item)  nl_langinfo( item)
 
 #else
@@ -85,6 +89,8 @@
 #define CRNCYSTR	50
  
 #define CODESET		51
+
+#define USE_BUILTIN_LANGINFO
 
 #define  kik_langinfo( item)  __kik_langinfo( item)
 

南です。

extended window manager hint にいくらか対応するパッチ
(アイコンのサポートと最大化の制御)を書いてみました。

どちらもwindow manager側での対応が必要なので、どの環境で
きちんと動作するかは把握できてません。最新のCVSへの差分を
  http://minami.unlimeted.com/mlterm/mlterm-ewmh.0818.diff
に置いておくので、テストしていただけるとありがたいです。

- アイコンについては、--enable-gdk-pixbuf付きで
(--enable-imlibは指定*しない*で) configureして構築したあと、
~/.mlterm/mainに
  icon_path = /path/to/icon.png
とか書いて起動すればgnomeのworkspace switcherや、
afterstep/wmaker/fvwm/twmのiconなどで指示したファイルが
使われるようになるはずです。

- 最大化の制御については、windowが最大化された状態で
font sizeを変更したとき、従来は最大化フラグは立ったまま
(WMによってはリサイズ禁止のまま)で窓の大きさが変わっていた
のが、窓の大きさを維持するようになるはずです。

$B:dK\$G$9!#(B

http://www-106.ibm.com/developerworks/linux/library/l-ltc27.html
$B$K$h$k$H!"(BInternationalized Terminal Emulator framework
  http://www-124.ibm.com/developer/opensource/linux/patches/i18n/iterm-0.1.tar.gz
$B$H$$$&$b$N$,$"$k$h$&$G$9$M!#(B

# $B$"!"5WJ]ED$5$s$,(B i1...@xf..., lin...@nl... $B$K(B
# $BEj9F$5$l$F$$$k!#(B
----------------------------------- 
$B:dK\(B $B9@B'(B <hs...@mt...> 
 http://www2u.biglobe.ne.jp/~hsaka/

佐藤です。

添付したパッチは、-g, --geometry オプションで指定した値が、サイズヒントに
正しく反映されないバグを修正するパッチです。

理解している範囲で、mlterm っぽく直したつもりですが あんまり自信ないです。
どうでしょうか?

--
佐藤精一 <ss...@sh...>

	荒木です:-)

	Subject: Re: [Mlterm-dev-ja] commit log
	From: MINAMI Hirokazu <mi...@ch...>
	Message-ID: <200...@ch...>
	Date: Mon, 29 Jul 2002 21:57:07 +0900

>  ~/.mlterm/termcap の xterm: に ut を書いて、mlterm -y=xterm で起動
>  →w3m http://mlterm.sf.net/index.ja.html とかして日本語を表示
>  →端末画面を縮小
>
> とかするとまだ落ちるようです。

        すみません、そのとおりでした。

> まだきちんと追ってないのですが、今回の対処で
>  ml_line_fill() で文字列の末尾を越えて書き込んでしまう可能性は
> 潰せてるはずなのでしょうか?

        潰せていません_o_

        修正を commit しておきました。

	では
--
kiken
ara...@us...