lgtm with a review. https://codereview.appspot.com/6775097/diff/1/ui/gtk3/keybindingmanager.vala File ui/gtk3/keybindingmanager.vala (right): https://codereview.appspot.com/6775097/diff/1/ui/gtk3/keybindingmanager.vala#... ui/gtk3/keybindingmanager.vala:231: foreach (var mask in masks) { This also can be fixed.
https://codereview.appspot.com/6775097/diff/1/ui/gtk3/keybindingmanager.vala File ui/gtk3/keybindingmanager.vala (right): https://codereview.appspot.com/6775097/diff/1/ui/gtk3/keybindingmanager.vala#... ui/gtk3/keybindingmanager.vala:231: foreach (var mask in masks) { On 2012/11/05 05:26:06, fujiwara wrote: > This also can be fixed. This one is not necessary. Because masks[] is not a const value. Only const array has the problem. Because vala does not generate code for const array correctly. :(