SAKURA Development BBS (ANSI)

▼ 2008/6/6 (金) 21:46:12 なすこじ

Mozilla/4.8 (Macintosh; U; PPC)

[5331] Re2:修正版アップしました

▼ ryojiさん
> コードを眺めていて思ったんですが、今は「折り返さない」については最低限の実装（最大長の拡大方向のみ追従）に止めておきませんか？

私はそれでかまいません。
ただ、私の中では「最大長の拡大方向のみ追従」というのが「折り返さない」からほんの少し変更しただけの状態なので、いっその事出来の悪い「折り返さない」を削除してしまった方が良い気がしています。

以下>>dev:5326に対して
> １．
> テキスト最大幅の算出をテキスト変更直後ではなく CEditView::MoveCursor() で遅延実行するのはなぜですか？

元々は1回の操作に対して何度も全ライン走査が走らない様にする事を考えていたのですが、負荷軽減をした事で不要になったと思います。

> あと、以前の計算要求が実行される前に次のテキスト変更があった場合、新たな変更を加えてしまった後で以前の計算要求を実行しているように見えますが、これは何？

バグです (^^;
ループした時のために辻褄を合わせようとして変な事になってます。

> ２．
> テキスト最大幅の算出を CEditView::OnSize() で実行するのはなぜですか？
> テキスト最大幅は画面サイズには関係無いと思うのですが…

最大幅が未算出の時に算出されるようにしていますが、手抜き……かもしれない。
未算出の時だけなので、アプリ起動時／ファイル読み込み／設定変更などで最大幅がクリアされなければ殆ど実行されないです。
ただ、OnSize()を利用するのは確かに変ですね。
SetLayoutInfo()は印刷プレビューとの絡みで躊躇しました。

ということで、「折り返さない」を削除したものを作成しても良いでしょうか？

[▼次のスレッド]

INCM/CMT
Cyclamen v3.81

[ut:0.000][st:0.000]