もずはっく日記(2015年8月)
2015年8月30日
Bug-org 895274 Change GUI MESSAGES to enum
初回投稿日時: 2015年08月30日10時49分00秒
カテゴリ: Events Mozilla Core バグ修正
SNS:
Tweet (list)
まだ作業半ばですが、しばらく、他の人のパッチを壊す可能性のあるバグなので紹介しておきます。
これまで、WidgetEvent::messageはuint32_tとして定義されていて、その値は単純にマクロを使って、整数値に名前を付けていただけでした。
このバグではソースコードをより安全なものにするため、これを名前付きenumに変更しました。その名前はmozilla::EventMessageです。これにより、整数値を単純にWidgetEvent::mMessage(リネームしました)に代入しようとしたり、比較しようとするとコンパイルエラーになるようになり、分かり難いバグの発生を抑えています。
そして現在はコーディングルールに従い、全てのイベントのメッセージをeプレフィックス付きの短い名前に置換していっています。影響範囲の大きいであろうキーイベントとマウスイベントのメッセージは昨日、mozilla-inboundに既に投入しました。明日にあると思われるマージでmozilla-centralに入ると思いますが、その近辺をいじっていたパッチはrebaseが必要になります。
そして、来週も時間をかけて地道な置換作業が続いていく予定です。