[7870] プライベートブランチでの変数の初期化について返信 削除
2014/7/22 (火) 01:35:26 ねこ
Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/36.0.1985.125 Safari/537.36
ローカルでカスタマイズを行っています。
タイプ別設定構造体(CTypr.h の STypeConfig)にint型の変数を追加しました。
基本動作は完成したのですが、変数の初期化タイミングが分かりません。

カスタマイズ内容
・行番号の最小幅を可変にする
・主な変更箇所はCTextArea::DetectWidthOfLineNumberArea_c
alculate()
・タイプ別設定ダイアログに設定項目を一つ追加する(3〜12の整数)

今問題になっていること
・変数の適切な初期化タイミングが分からず、追加した変数の初期化を行っていない。
・エディタを起動すると、行番号が表示される幅が0になっている。
・タイプ別設定を開くと、追加したEDITボックスコントロールが0となっている。
・ダイアログをOKで閉じた後は補正処理が効くため期待通り操作する。

期待する動作
・iniファイルに当該項目がない場合は初期値3で初期化する(現状の動作に合わせる)


どの変数を参考にすればよいなどのアドバイスがありましたらよろしくお願いします。


[7871] Re: プライベートブランチでの変数の初期化について返信 削除
2014/7/23 (水) 22:08:29 もか
INCM1.23a
タイプ別設定の初期化コードは CType.cpp _DefaultConfig()にあります。


[7872] Re2: プライベートブランチでの変数の初期化について返信 削除
2014/7/26 (土) 16:41:53 ねこ
Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/36.0.1985.125 Safari/537.36
ありがとうございます。
期待通り初期化ができるようになりました。


[7873] Re2: プライベートブランチでの変数の初期化について返信 削除
2014/7/26 (土) 17:19:26 ねこ
Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/36.0.1985.125 Safari/537.36
行番号の幅(桁数)を可変にするパッチをwikiに添付しました。
ダイアログの修正が仮なのでご注意願います。
ご確認いただけますと幸いです。
http://sakura.qp.land.to/?Develop%2F13


[7874] Re3: プライベートブランチでの変数の初期化について返信 削除
2014/7/26 (土) 17:28:03 ねこ
Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/36.0.1985.125 Safari/537.36
裁量権のある方に確認いたします。

この修正を適用するにあたり
・設定ダイアログを大きくしてよいか。
・math.hをincludeしてよいか。

以上二点、よろしくお願いします。
(最終的には本流へのcommitを目標としています)


[7877] Re4: プライベートブランチでの変数の初期化について返信 削除
2014/7/27 (日) 19:00:35 もか
INCM1.23a
#裁量権はわかりませんがcommit権限もちです。

>・設定ダイアログを大きくしてよいか。
行番号関連はタイプ別設定-ウィンドウにあるので、そちらに置いたらどうでしょうか。
それなら大きくしなくてもよさそうです。
>・math.hをincludeしてよいか。
個人的には初心者向きのfor版のままのほうがいいかなと思います。
_で始まるキーワードは既存コードでも使われていますが、コンパイラ等の予約なので、普通の文字で始めるようにしたほうがいいです。


[7879] Re5: プライベートブランチでの変数の初期化について返信 削除
2014/7/28 (月) 23:43:57 ねこ
Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/36.0.1985.125 Safari/537.36
もかさん、ありがとうございます。
ご提案いただいた方針で修正します。

[▼次のスレッド]
INCM/CMT
Cyclamen v3.81