|
From: | Vladimir 'φ-coder/phcoder' Serbinenko |
Subject: | Re: Device instability, gettext and default |
Date: | Mon, 05 Mar 2012 01:03:57 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20120216 Icedove/8.0 |
On 05.03.2012 00:51, Andreas Vogel wrote:
Am 03.03.2012 14:37, schrieb Vladimir 'φ-coder/phcoder' Serbinenko:=== modified file 'grub-core/commands/menuentry.c' --- grub-core/commands/menuentry.c 2012-02-26 16:28:05 +0000 +++ grub-core/commands/menuentry.c 2012-03-03 12:11:30 +0000 @@ -36,6 +36,8 @@ N_("Keyboard key to quickly boot this entry."), N_("KEYBOARD_KEY"), ARG_TYPE_STRING}, {"source", 4, 0, N_("Use STRING as menu entry body."), N_("STRING"), ARG_TYPE_STRING}, + {"id", 1, GRUB_ARG_OPTION_REPEATABLE, + N_("Menu entry identifier."), N_("STRING"), ARG_TYPE_STRING}, {0, 0, 0, 0, 0, 0} };1) After bzr-pulling your latest changes and reviewing that source again, i noticed that you are using GRUB_ARG_OPTION_REPEATABLE for the new id option but you are not handling multiple ids (as far as i can see).
It wasn't intentional. Fixed now.
2) Why do you refuse to allow short options for all of the menuentry options? Any special reason?
Because it shares the space with options to menuentry.
No. Thinking like this is a slippery slope. Such patches may also contain bugs (if you confuse 2 numbers).3) Wouldn't it be a good chance to use my patch which uses an anonymous enum for indexing the options array?
Your patch could be committed into experimental but not trunk. Life doesn't end at 2.00.
-- Regards Vladimir 'φ-coder/phcoder' Serbinenko
[Prev in Thread] | Current Thread | [Next in Thread] |