もずはっく日記(2015年9月)
2015年9月17日
Bug-org 895274 Change GUI MESSAGES to enum #2
初回投稿日時: 2015-09-17 21:59:33
カテゴリ: Events Mozilla Core Mozilla43 バグ修正
SNS:
Tweet (list)
大きな変更になるので、予め記事を書いておきましたが、その修正がようやく完了しました。その結果、
WidgetEvent::messageはWidgetEvent::mMessageに改名WidgetEvent::mMessageは名前付きのenumである、mozilla::EventMessageになったEventMessageを格納している変数名は、基本的にはmessageかmsgを含むものとなったEventMessageの各項目の名前は、eプリフィックス付きのできるかぎり短い名前になった- DOMイベントの元となるイベントの場合、その名前を利用するようになった
- 名前があまりにも一般的すぎる場合は
eの直後にイベントのグループ名等を含むようになった - 標準仕様策定前に使われていたイベントや、現在廃止予定のイベントは
eLegacyで始まる名前になった
という形になっています。内部の名前とDOMイベントの名前のズレや、歴史的経緯を知らない開発者向けにレガシーイベントであることを名前で示せるようになったのは大きかったです。
また、VisualStudio、gcc、lldb、Xcode全てで変数の内容を見ると数値ではなく、enumで付けられている名前が表示されますのでデバッグしやすくなっています。
ちなみに安全のため、ひとつの置換をひとつのパッチで行ったため、最終的には270個のパッチに分割され、関係者からも単独のバグではさすがに過去最多のパッチ数じゃないかと言われています。