もずはっく日記(2014年2月)
2014年2月22日
Bug-org 969219 Simplify constructor of InternalTransitionEvent
初回投稿日時: 2014年02月22日13時59分57秒
最終更新日時: 2014年02月22日14時00分18秒
カテゴリ: Mozilla Core Mozilla30 バグ修正
SNS:
Tweet (list)
全イベントのコンストラクタをできるだけ同じ形にして、各メンバは明示的に初期化するようにしよう、というバグの、InternalTransitionEvent版です。
InternalTransitionEvent(bool aIsTrusted, uint32_t aMessage,
const nsAString& aPropertyName,
float aElapsedTime,
const nsAString& aPseudoElement);
を、
InternalTransitionEvent(bool aIsTrusted, uint32_t aMessage);
に変更しています。これ、何が問題化というと、次のようなコードを書いた時、何が何だか分からないんですよね。
InternalTransitionEvent event(false, 0, EmptyString(), 0.0f, EmptyString());
メンバに直接代入するようにすれば、無駄を省けるし、何に対して値を設定しているのか明瞭になるので、コードが読みやすくなり、イージーミスも減るという訳です。