Izumi Tsutsui

Results 13 comments of Izumi Tsutsui

FYI, @shibafu528 has patches for twitter-text to use simpleidn https://github.com/mmriis/simpleidn instead of idn-ruby https://github.com/shibafu528/twitter-text/commit/80cdb0c76e64a71401c1bd4cb2f58327c82d99ff with some modification https://github.com/shibafu528/simpleidn/commit/69420ce9777a4b2a338dd981b31a3d4fc39dd72b

> An "empty" attr in some form is probably the best way forward here, yes. That way, we get the default mode bits from `create_mode()` in a sane way. I...

Confirmed, at least on NetBSD/i386 10.0 and patched pkgsrc/x11/mlterm (without gdk_pixbuf2 option). Thanks! ![mlterm-libpng-emoji](https://github.com/user-attachments/assets/a8473132-7138-49bd-a55d-d519eec19e1d)

Also confirmed on installed version: ![mlterm-libpng-emoji-2](https://github.com/user-attachments/assets/7e177ba2-8704-4b1f-999a-c2e624a3c93e)

Here is a tcpdump log on Ubuntu unfs3 server (192.168.20.204) and NetBSD client (192.168.20.1) with the following commands: ``` mount_nfs 192.168.20.204:/home/tsutsui/tmp /mnt cd /mnt mkdir a rmdir a touch a...

本業がアレだと趣味レビューするガッツが出ねえ、みたいな日々で遅くなりましたが * 要件については、感想はあれど要望を出す立場にはないのでサイズ可変仕様は戻しました (参照用に続きでコミットしましたが、このままマージするより squash すべき) * フォント幅については VT382などでは縦横比が 30x12 と 2:1 でないフォントも使うようなので必要と考えて残してあります * `print_picture()` の引数についてはアイコンサイズ固定ならば従来から変更無いのでこのPRではスコープ外になるかと * 「返り値で右移動量を出す」仕様について、自分で提案しておきながら「値としては起動時に1回取得すればいいだけじゃん」という気もするので、インデント量を返す関数を作るか、フォント幅を返す関数を返す(アイコン幅は固定前提)ほうがいいのかも(どの仕様がいいのか切り分けが思いついていない)

あー。雑にコミットしたら余分な差分も入ってしまいました ```diff - indent_icon = ((6 * mul_icon) + fontwidth - 1) / fontwidth; + indent_icon = ((6 * SIXEL_MUL_ICO) + fontwidth) / fontwidth; ``` これは「フォント幅が6の倍数だった場合フォントとアイコンが接触してしまう」ので「最低1〜6ドットは離す」の修正です。(元は 0〜5ドット離す、になっていた) で、こんなこと書かずに ``` indent_icon...

乱暴に rebase ついでにコミットを以下に分割しました。 * アバターアイコンサイズをフォント高さから算出する * アバターアイコンの右側に投稿者名と日付の行を表示する が、ここに来て画像キャッシュの仕様が「sixelを保存」になっていてそこにはサイズの情報がないので、 「キャッシュされたアバターアイコンのサイズと現在のアバターアイコンのサイズとを照合して不一致ならキャッシュを破棄」 の処理を追加で入れないと不整合が発生してしまう(キャッシュされたアイコンは元のままのサイズで表示される)、 という状態になっています。 プルリクにしたのはリクエストというよりはコンセプトパッチとして見るためだったので、いったん破棄でもいいかも。

あー、老人なので記憶が飛んでました。 f48ce0f で「アバターアイコンサイズをフォント高さから算出する」はいったん無しにしてたんでしたっけ……。 アイコンサイズ可変は前述のとおりキャッシュもめんどくさいので、仕様としては 「アイコンの右側に投稿者名と日付の行を表示」だけにすべきですかね。

「アイコンの右側に投稿者名と日付の行を表示」だけ、ならばカーソル移動も `sixel.c` に持っていけばいいや、 と雑に書き直しました。 投稿者名が長いと80桁超える場合ありますが、それは元から同じということで……