Fix/#1463 dup session - #1466
Merged
Merged
Conversation
- GetParamAlloc() を common/ttlib_static_cpp.cpp に追加
- GetParam() の動的バッファ版
- 入力文字列と戻り値を const 化
- GetParam() は GetParamAlloc() を使用する wrapper に変更
- GetFilePath() を ttset.c から common/ttlib_static_cpp.cpp へ移動
- common/ttlib.h で参照できるようにした
- ParseFOption() を ttset.c から vtwin.cpp へ移動して static 化
- GetParamAlloc() の使用でコマンドライン文字列を破壊しなくなり、
GetCommandLineW() の戻り値を複製せずに渡せるようになった
- 使用していなかった BOOL の戻り値を廃止
- CVTWindow コンストラクタの初期化処理を InitSettings() として切り出し
- TTXInit() によるフック設定と ini ファイル読み込みの順序についてコメントを追加
- wchar_t 配列のサイズを sizeof()(バイト数)で渡していた箇所を
_countof()(文字数)に修正
- 実際のバッファの2倍のサイズが渡り、長いパラメータで
バッファオーバーランする可能性があった
- GetParam() に引数のコメントを追加
szOption はモードによって connect コマンドの引数になったり
ttermpro.exe の引数になったりしていたため、役割ごとに分離した。
あわせてセッション複製の設定引き継ぎを起動時に行うようにした。
- szConnectOption は常に TTL の connect コマンドの引数に追加
- 全モードで connect 行末尾への追記に対応
- /nossh は接続オプションなので TTDUP_TELNET の exe 引数から
connect コマンド側に移動
- szExecOption は常に ttermpro.exe のコマンドライン引数に追加
- セッション複製時、/DUPLICATE, /K=, /F= を szExecOption で
起動時オプションとして渡すようにした
- 起動時の ParseParam で処理され、共有メモリからの設定引き継ぎが
ウィンドウ作成前に行われる(従来は connect 経由の DDE 受信時まで
処理されなかった)
- connect のオプションは TTXSetCommandLine() 結果の実行ファイル名
以降を使用(プラグインは末尾追記だけでなく挿入も行うため、
呼び出し前後の差分では切り出せない)
- /DUPLICATE 付き起動時の設定リストアを修正
- 共有メモリに複製元の設定がないときは何もしない
- コマンドラインで指定したマクロ(/M=)を上書きしないよう待避・復元
- ホスト名は connect コマンドで渡されるためクリア
- CopyShmemToTTSet() は既存の共有メモリを OpenFileMapping() で開くよう修正
- セッション複製時の設定引き継ぎとキーボード初期化を CVTWindow コンストラクタから InitSettings() へ移動 - コマンドライン文字列の複製を1つにまとめ、 ParseFOption() と ParseParam() で共用するようにした - セッション複製時、コマンドラインで指定したマクロ (/M=) の 待避・復元でメモリリークしないよう修正
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.