Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add Fore button to bring the selected window to foreground #185

Merged

Conversation

sempreff
Copy link
Contributor

#83 の対応案です。

… to foreground in Broadcast command dialog
@sempreff
Copy link
Contributor Author

#188 の内容も盛り込みました。

@zmatsuo
Copy link
Member

zmatsuo commented Apr 3, 2024

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

コメントです。

  • 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
Copy link
Member

nmaya commented Apr 3, 2024

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

- remove unused variable ime_mode
- remove unused function BuildMulticastCopyData
- rename PreparePaste to CBPreparePaste
@sempreff
Copy link
Contributor Author

sempreff commented Apr 3, 2024

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

@zmatsuo
Copy link
Member

zmatsuo commented Apr 5, 2024

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

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

バイナリを作成しました。
https://ci.appveyor.com/project/teraterm/github-main/builds/49556019/artifacts

@sempreff
Copy link
Contributor Author

sempreff commented Apr 6, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants