脳log[20131009]

2013年10月09日 (水) [SakuraEditor]「Sakura Editor / PatchUnicode / #682 GetActiveView()関数整理」■1.コードで保証されない明示されてもいない但し書き。2.ポインタサイズ(32ビット/64ビット)のさらなる肥大化。■配列とインデックスから有効なビューは求まるのだから有効なビューへのポインタを３番目のメンバ変数として新たに持つことはない。DRYじゃない。こんな、機能を追加するたびに肥大化してきたような多目的巨大クラスで、複数人がよってたかっていじくるソースで、いつまで整合性が維持できるものか。■ループの前に変数に受け取る最初のパッチの方がいいと思うなあ。■■■@2013-10-07「Sakura Editor / PatchUnicode / #680 色分け表示対象の事前判定」■非表示(=色分け不要)の場合を事前にチェックして色分けに参加させないのは賛成(自分も SHJS方式を実装したときそうした。Lua対応でコメントの色分け優先度を動的に決定するのと同時に)。だがその影響で「CheckColorMODEはレイアウト処理全体のボトルネックになるくらい頻繁に呼び出される(略)ここはエレガントさよりも性能優先で(略)」とコメントされてるようなホットスポットに NULLチェックを付け加えるというのは……。■というようなことをこの前からなんでここでつぶやいてるかというと、他人の足を引っ張るだけの人間はいらないよねっていう。コードレビューって言ったら聞こえがいいか？