◀Unicode版開発トップへ
  • 543 Commit報告:(Keep) リソース関連のリファクタリング
    • 586 Re:Commit報告:(Keep) リソース関連のリファクタリング

  • [543] Commit報告:(Keep) リソース関連のリファクタリング Uchi 2008年07月04日 22:42

    リビジョン：
    rev1406

    変更種別：
    リファクタリング

    内容：
    リソース関連のリファクタリング
    sakura_rc.hm の削除
    sakura_rc.hからsakura_ra.rcで使用していないシンボルの削除
    上記の伴うソースの変更（存在していないコントロールの削除）
    リーソースのシンボル名の変更（デフォルト名のままの物に名前を付ける）
    いくつかのシンボル値の変更（IDC_BUTTON_PRINTERSELECT）
    IDS_AUTHOR_PAGE を http://sakura-editor.sourceforge.net/ に変更
    IDS_AUTHOR_MAIL の削除
    IDR_TRAYMENU_Lの中身の削除（プログラムで作成しているので）

    • [586] Re:Commit報告:(Keep) リソース関連のリファクタリング kobake 2008年08月26日 23:30

      rev1406
      ソースコードさらっと確認しました。
      ・define変更部分は流し読み
      ・リソース変更部分は流し読み
      ・define以外のソース部分に問題ないことを確認

      動作さらっと確認しました。
      ・無作為に選んだヘルプ参照が正常に動作することを確認
      ・それ以外の変更箇所の動作で問題ないことを確認

      問題はないのですが、ひとつコメントしますと、
      CDlgProperty::OnBnClickedで
      「case IDCANCEL: // 未使用 del 2008/7/4 Uchi」
      とありますが、IDCANCELは未使用ではないです。
      ダイアログの×ボタンを押したときと、キーボードのEscを押したときに、IDCANCELのWM_COMMANDが発生します。

      今回の場合は、それ以降の CDialog::OnBnClicked が IDCANCEL の処理をしてくれているので、動作上の違いは出ていません。