[webkit-reviews] review granted: [Bug 18363] [GTK] Combo boxes cannot be opened pressing space : [Attachment 33001] Make it if-elif-else-endif as suggested by Holger

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jul 17 19:41:55 PDT 2009


Holger Freyther <zecke at selfish.org> has granted Jan Alonzo
<jmalonzo at gmail.com>'s request for review:
Bug 18363: [GTK] Combo boxes cannot be opened pressing space
https://bugs.webkit.org/show_bug.cgi?id=18363

Attachment 33001: Make it if-elif-else-endif as suggested by Holger
https://bugs.webkit.org/attachment.cgi?id=33001&action=review

------- Additional Comments from Holger Freyther <zecke at selfish.org>
OKay, the next thing boils down to taste. To reduce code duplication one could
do

if (
#if SPACE_OR_..
keyCode == ' ' || keyCode == '\r'
#elif
keyCode == ' '
#endif
) {
  common Code
}


More information about the webkit-reviews mailing list