[2893] そろそろ取り込みすい 2003年06月26日 02:27

そろそろ取り込んでみてはどうでしょうか、と言ってみる。
大分落ち着きましたし。

◎ みくさん http://www.geocities.co.jp/SiliconValley-SanJose/3299/

★ タブ(もどき)
　正直、なんか初めのうちは実験だけかな？とか思っていたのですけど、
　動作がちょっとおかしい所とかをちょこちょこ修正していくうちに、
　なんだかとっても本格的にそれっぽい雰囲気になってきて、、、
　万一、このまま消えちゃったりしたら勿体無いな～、と。

◎ もかさん http://www.geocities.co.jp/SiliconValley-Oakland/6358/

★ 動的補完機能
　編集中のファイル中の単語も入力補完の候補として現れるように設定できる。
　プログラミング時に自分で定義した変数名/関数名の補完などに．．．

★ 持ち歩き向け
　各設定中で相対パス指定 *も* 可能になる。

★ 入力改行コード指定
　改行時の改行コードをメニューやツールバーから設定可能に。

★ マクロ拡張
　マクロの機能追加。

#「Grepプレビュー版」は、もう取り込み済みなんですよね。

「持ち歩き向け」だけ最新版はまだバイナリが入手できないので未確認ですが、
その前の版では特に問題は見つからなかったですし。。。
最後の修正でチョンボが無い事を祈りつつ(笑)

変更箇所の衝突は、、、私が知っている範囲では、「タブ」と「入力改行コード指定」が
共にツールバー（画像）の変更あり、くらいで、後は．．．(;^^Ａ

[2895] Re: そろそろ取り込みげんた 2003年06月26日 02:47

>そろそろ取り込んでみてはどうでしょうか、と言ってみる。
>大分落ち着きましたし。
そのようですね．といいつつ，全然中身見てなかったりして...
でも，すいさんが大丈夫って言うなら大丈夫でしょう:D

最近私の周りではWindows 95/98/NTあたりが死滅しかかっているので複数環境で丁寧に動作確認して頂ける方は非常に貴重です．はい．
どこかからPC調達して初代Win95やWin98を動作確認用にインストールとか本気でやりたいくらい．

--
ところで両方いっぺんに取り込んでうまくいくだろうか...

>★ タブ(もどき)

>◎ もかさん http://www.geocities.co.jp/SiliconValley-Oakland/6358/
こっちはMix版が出るんでしょうか？

>#「Grepプレビュー版」は、もう取り込み済みなんですよね。
ええっ？入れた記憶無いですけど...

>変更箇所の衝突は、、、私が知っている範囲では、「タブ」と「入力改行コード指定」が
>共にツールバー（画像）の変更あり、くらいで、後は．．．(;^^Ａ
イメージのマージ...やったこと無いな...
- [2897] Re2: そろそろ取り込みすい 2003年06月26日 04:02
  
  >ところで両方いっぺんに取り込んでうまくいくだろうか...
  げんたさんの愛と人徳があれば大丈夫です、きっと。(笑)
  # そんな物がプログラミングに何の関係あるんだ？＞私
  さぁ、レッツごちゃ混ぜ。
  だめだったらげんたさんに愛と人徳が無かったという事で．．．(をぃ＞私)
  
  >>#「Grepプレビュー版」は、もう取り込み済みなんですよね。
  >ええっ？入れた記憶無いですけど...
  
  あ、イカン、「Grepプレビュー版」って2831の修正の事じゃぁなかったのですね。(オオボケ) (;^^ゞ
  簡単に動かしてみたけど、どこがどう変わっているのかよく分からず、
  Grep 対象のファイル欄が空の時に *.* になってくれていたから
  てっきり2831の一連の修正のバイナリかと。
  プレビュー機能、見てないです。 m(_ _)m
[2901] RE: そろそろ取り込みみく 2003年06月26日 19:01

>変更箇所の衝突は、、、私が知っている範囲では、「タブ」と「入力改行コード指定」が
>共にツールバー（画像）の変更あり、くらいで、後は．．．(;^^Ａ

タブウインドウ関係で追加した画像は、
270 タブの表示・非表示　（270にあった作業画像は271に移動）
313 タブ右クリックメニュー（カスタムメニュー24）（312,313にあった作業画像は317,318に移動）
です。
[2904] Re: そろそろ取り込みげんた 2003年06月26日 23:29

せっかくなので，ヘルプも含めてパッケージとして出したいですね．
最近書籍収録のタイミングでしかパッケージ作ってないなぁ...

ただ，かろとさんの印刷問題修正を先に出した方がよさそう．
[2907] マージの前に: げんた 2003年06月29日 05:59

>◎ みくさん http://www.geocities.co.jp/SiliconValley-SanJose/3299/
>
>★ タブ(もどき)
落ち着いたと思ったらまた更新されてますね．焦らなくて良かった(^^)

--
TABありとなしの切り替えをしてウィンドウを開くと両者が混在して変なことになるようですが，どちらかに限れば問題ないのではないでしょうか．混在すると手順によってはウィンドウが画面上から消えて無くなったりするようですが．

---
私はTABブラウザでTAB上の右クリックはウィンドウを閉じるに割り当てているので，メニュー無しでいきなりアクションを登録することもできた方がいいと思います．カスタムメニューに登録されている機能が1つだけならメニューを開かずに登録された機能を実行というのを付ける方法が簡単かと思います．

右クリックしたときに当該ウィンドウに切り替わってしまうのは現状ではいかんともしがたいのでしょうね．

--
TABを付けてウィンドウをまとめないモードって使いにくいような気がしますが...

--
sakura_rc.rcでフォント指定がよく

-FONT 9, "ＭＳＰゴシック"
+FONT 9, "ＭＳＰゴシック", 0, 0, 0x1

となっているのですが，これはツールの違いですかね？なぜ後ろのパラメータが増えてしまうのか．
- [2912] RE: マージの前に: みく 2003年06月30日 19:36
  
  >TABありとなしの切り替えをしてウィンドウを開くと両者が混在して変なことになるようですが，どちらかに限れば問題ないのではないでしょうか．混在すると手順によってはウィンドウが画面上から消えて無くなったりするようですが．
  
  ウインドウメニューから切り替えできませんか？
  
  >右クリックしたときに当該ウィンドウに切り替わってしまうのは現状ではいかんともしがたいのでしょうね．
  
  対象のウインドウが非表示の状態なのに、
  （検索などの）ダイアログが出るようなパターン
  があったので、先に切り替えるようにしました。
  
  >sakura_rc.rcでフォント指定がよく
  >
  >-FONT 9, "ＭＳＰゴシック"
  >+FONT 9, "ＭＳＰゴシック", 0, 0, 0x1
  
  多分イタリック体とかの情報だと思うのですが、
  逆に消える場合もあるので、私もよくわかりません。
  - [2928] 仮マージ完了げんた 2003年07月04日 01:47
    
    とりあえずみくさんのTABのみ先行してマージしました．
    
    マージ済みソースをご確認ください．
    http://sakura-editor.sourceforge.net/snapshot/ssrc_tab_0628.tar.bz2
    
    ・コメント追加以外の変更点
    共通設定ダイアログで，追加されたTAB表示とTABをまとめない設定に同一のIDが振られて誤動作していたので，まとめない設定の番号を変更しました．sakura_rc.h
    
    一般掲示板で書きましたが，外部プロセスを監視する時に休みが全然無いと外部プロセス実行中にsakura.exeがCPUを100%使ってしまうので，パイプのチェック間隔を200msecおきにしました．
    
    メッセージ処理をブロックしないためにはMsgWaitForMultipleObjects()を使ってメッセージを待たせないようにするのがbestですけどね．
    
    ・質問
    CFuncLookupで機能名を引く関数が追加されて全部コメントアウトされているのは何なんですか？
    
    TABの追加以外で何が直ったのか全然追えていません．
    - [2934] RE: 仮マージ完了みく 2003年07月04日 18:56
      
      >マージ済みソースをご確認ください．
      ファイルが壊れてる模様
      
      >・質問
      >CFuncLookupで機能名を引く関数が追加されて全部コメントアウトされているのは何なんですか？
      追加したのにコメントになってるソースは実装時のごみなので、
      ばっさり削除してかまいません。
      
      CFuncLookupの関数は、
      メニューの文字列が統一されてないという指摘があったので、
      文字列の出所を1箇所（リソースからLoadString）にまとめようとした跡です。
      メニューにはショートカットがありますが（例：新規作成(&N)）、
      リソースにはショートカットがありません（例：新規作成）。
      そこで、メニューを生成するときにリソースを元にショートカット付きの文字列を生成するものです。
      ＃意欲のある方に実装してもらって...
      - [2936] Re2: 仮マージ完了げんた 2003年07月04日 23:19
        
        >>マージ済みソースをご確認ください．
        >ファイルが壊れてる模様
        失礼致しました．ただいま差し替えました．
        
        >追加したのにコメントになってるソースは実装時のごみなので、
        >ばっさり削除してかまいません。
        こちらの返事をいただけたので，こっちで進めます．
[2908] マージの前に:1 げんた 2003年06月29日 05:59

遅くなりましたが変更も落ち着いてきたようなのでマージに取りかかろうと思い差分ファイルを一通り見てみました．マージの後にまた変更されると手間がかかるので一応反応を聞いてからということで．ほとんど独り言ですので，今のままで良ければコメントだけ加えてマージします．

試しにまとめてパッチ当てると結構ぶつかりますねぇ．^^;

>◎ もかさん
変更したのに何も記述していないところが多いです．
そのような箇所はdiffを見ながら日付と名前を入れています．

以下，コメント＋質問です．今回の変更と直接関係ないものもありますがご容赦ください．

>★ 動的補完機能
>　編集中のファイル中の単語も入力補完の候補として現れるように設定できる。
>　プログラミング時に自分で定義した変数名/関数名の補完などに．．．

補完を開始したとき＋文字入力するたびにファイル全体を走査するのはパフォーマンスが悪いように思いましたが，1万行程度のファイルなら問題なく使えますね．(Athlon 1.4Gの場合) CPU負荷も全然増えないですし．
50万行×4wordの200万語のすべて長さ5byteの単語かつ補完候補が1024以上という環境でも0.5秒くらいの遅延ですね．単語の長さが異なる場合には比較は行われないので，最悪のケースでこのぐらいと考えて良さそうです．

>★ 持ち歩き向け
>　各設定中で相対パス指定 *も* 可能になる。

_IS_ABS_PATH()という名前で，絶対パスの時falseを返す仕様は間違いの元になりそう．

--
相対パス指定が可能となってはいますが，ファイルを開くダイアログボックスを使って設定すると絶対パスになりますよね．
ここも自動的に相対パスになるとbetterな気がしますが．

>★ 入力改行コード指定
>　改行時の改行コードをメニューやツールバーから設定可能に。

特に気になった点はありません．

>★ マクロ拡張
>　マクロの機能追加。
CPPAのインスタンスはPPAが実行されている間は存在しているので，追加された
CMemory CPPA::m_cMemRet;
CMemory CPPA::m_cMemDebug;
bool CPPA::m_bError = false;
と元からある
CEditView* CPPA::m_pcEditView = NULL;
DLLSHAREDATA* CPPA::m_pShareData = NULL;
がstaticである必要は無いのでは？

---
PPAのバージョンによる条件コンパイルが（やざきさんの初期版から）入っていますが，よく考えてみるとこの条件は実行時ではなくコンパイル時に判定するものなので，あまり意味はないのかなぁと．実行時にDLLからバージョンを取得して使い分けるのであれば意味があるでしょうけど．

よく見ると，PPA付属のヘッダファイルは使っていないから自分で定義しているのね(笑)

---
CDocLineMgr::GetLineStr() は連続したアクセスが近い位置にあることを前提に内部のポインタを使ってシーケンシャルにスキャンしているので，ランダムアクセスは時間がかかりますね．これのポインタを外に出せればアウトライン解析の結果を自動追従させたりできそうなのになぁ．(関係ない話になってしまった...)
- [2922] Re: マージの前に:1 もか 2003年07月02日 11:02
  
  ▼げんたさん
  >遅くなりましたが変更も落ち着いてきたようなのでマージに取りかかろうと思い差分ファイルを一通り見てみました．
  >マージの後にまた変更されると手間がかかるので一応反応を聞いてからということで．
  
  *予定と確認
  _IS_ABS_PATHの修正とコメントを追加して、おそらく金曜の夕方までにミックス修正版を作るつもりでしたが、
  本業に時間をとられそうなので、さらに遅くなるかもしれません。
  すでに取り込み中か、それより早くにマージするのでしたら、申し訳ありませんが、げんたさん側で修正お願いします。
  先にタブモードを取り込むのでしたら、そっちだけでリリースして、それを元に再作成でもかまいませんけど。
  
  >>◎ もかさん
  >変更したのに何も記述していないところが多いです．
  >そのような箇所はdiffを見ながら日付と名前を入れています．
  良くないと思いつつ、相変わらずコメント少ないです。
  日付と変更内容・仕様の記述は意味があることは理解しています。言い訳を書くと...(前も書いたような）
  ・ソースコードが、日付と名前（と未使用になった過去のソース）で埋まることを暗に恐れていたり
  ・元々統合する予定でなくコメントなしのままになっている
  ・急ごしらえで作った
  ・投げやり実装。←何とかしろ自分
  
  >>★ 持ち歩き向け
  >_IS_ABS_PATH()という名前で，絶対パスの時falseを返す仕様は間違いの元になりそう．
  意味が正反対ですね。突っ込まれるまで気が付かないとは、英語力の無さを痛感。失礼しました。
  _IS_REL_PATHかな。*間違いの元*ではなく、間違いそのものですので修正願います。
  
  >相対パス指定が可能となってはいますが，ファイルを開くダイアログボックスを使って設定すると絶対パスになりますよね．
  >ここも自動的に相対パスになるとbetterな気がしますが．
  実装自体は簡単そうです。
  自己メモ：「実行ファイルディレクトリ\」を「\」に大小英文字を区別せずに置換するだけ
  
  >>★ マクロ拡張
  >CPPAのインスタンスはPPAが実行されている間は存在しているので，
  >CPPA::m_cMemRet等はstaticである必要は無いのでは？
  コールバック関数からアクセスするのに必要です。
  普通のメンバ変数にしてCPPA*をstatic(global)で保持すれば、確かに必要ないです。
  
  >PPAのバージョンによる条件コンパイルが，あまり意味はないのかなぁと．
  ソースを読むとPPAの必須バージョンが分かる程度ですね。
  そういえば、PPA.DLLの必須バージョンが1.3以上になりました。1.2以下の人は更新してください。
  >実行時にDLLからバージョンを取得して使い分けるのであれば意味があるでしょうけど．
  1.2以下でも動くようになる以外、さほど実装する価値はなさそう
  #Real型を扱う関数は将来も必要にならないような...
  - [2951] Re2: マージの前に:1 もか 2003年07月14日 17:19
    
    お久しぶりです。
    またしても、HDDが壊れて現在、修理に出している最中です。
    当分作業できそうにないです。（泣

[2893] そろそろ取り込み すい 2003年06月26日 02:27

[2895] Re: そろそろ取り込み げんた 2003年06月26日 02:47

[2897] Re2: そろそろ取り込み すい 2003年06月26日 04:02

[2901] RE: そろそろ取り込み みく 2003年06月26日 19:01

[2904] Re: そろそろ取り込み げんた 2003年06月26日 23:29

[2907] マージの前に: げんた 2003年06月29日 05:59

[2912] RE: マージの前に: みく 2003年06月30日 19:36

[2928] 仮マージ完了 げんた 2003年07月04日 01:47

[2934] RE: 仮マージ完了 みく 2003年07月04日 18:56

[2936] Re2: 仮マージ完了 げんた 2003年07月04日 23:19

[2908] マージの前に:1 げんた 2003年06月29日 05:59

[2922] Re: マージの前に:1 もか 2003年07月02日 11:02

[2951] Re2: マージの前に:1 もか 2003年07月14日 17:19

[2893] そろそろ取り込みすい 2003年06月26日 02:27

[2895] Re: そろそろ取り込みげんた 2003年06月26日 02:47

[2897] Re2: そろそろ取り込みすい 2003年06月26日 04:02

[2901] RE: そろそろ取り込みみく 2003年06月26日 19:01

[2904] Re: そろそろ取り込みげんた 2003年06月26日 23:29

[2928] 仮マージ完了げんた 2003年07月04日 01:47

[2934] RE: 仮マージ完了みく 2003年07月04日 18:56

[2936] Re2: 仮マージ完了げんた 2003年07月04日 23:19