TeraTermProject / teraterm

Other

392 stars 33 forks source link

Closed sempreff closed 3 months ago

sempreff commented 3 months ago

83 の対応案です。

sempreff commented 3 months ago

zmatsuo commented 3 months ago

ざっと動かしてみてよさそうな感じです。

コメントです。

selected[] 変数
- ファイル内だけなので static を付けたい
PreparePaste()
- PreparePaste() にしたい
- clipboar.c,h の関数には CBを付けたい
TTTSet ts;
- ttwinman.h を include してあるので不要
- multiple definition of `ts'; CMakeFiles/teraterm.dir/objects.a(broadcast.cpp.obj):broadcast.cpp:(.data$ts+0x0): first defined here
broadcast.cpp:184:21: warning: variable 'ime_mode' set but not used [-Wunused-but-set-variable]
warning: 'COPYDATASTRUCT BuildMulticastCopyData(const char, const char*)' defined but not used [-Wunused-function]
- この関数使っていないですね。削除しておいてください。

nmaya commented 3 months ago

en と ja は対称になっているはずなので、history.html が en だけ conflict しているのはおかしい気がします。 conflict するとしたら ja 側もすると思いますが、この修正とマージ先のどちらが異なっているでしょうか？

sempreff commented 3 months ago

@zmatsuo レビューありがとうございます、修正しました。 @nmaya PRブランチに upstream の main を取り込む際に誤ったと思われます、修正しました。

zmatsuo commented 3 months ago

何度も修正いただきありがとうございます。

mainにマージしました。でもマージ時にミスして修正しつつマージしました。まだミスが残っていたら指摘ください。

sempreff commented 3 months ago

日英以外の言語ファイルについてこれから作業するつもりでしたが、手当ていただきましてありがとうございます。お手数おかけしました。マージありがとうございました。