On Mon, Feb 14, 2011 at 10:11:02AM +1000, Peter Hutterer wrote:
> On Fri, Feb 11, 2011 at 03:32:58PM +0500, Alexandr Shadchin wrote:
> > Also, do not print values of variables CustomKeycodes twice.
> >
> > Signed-off-by: Alexandr Shadchin <alexandr.shadc...@gmail.com>
> > ---
> > src/bsd_kbd.c | 1 -
> > src/hurd_kbd.c | 1 -
> > src/kbd.c | 13 ++-----------
> > src/sun_kbd.c | 2 --
> > 4 files changed, 2 insertions(+), 15 deletions(-)
> >
> > diff --git a/src/bsd_kbd.c b/src/bsd_kbd.c
> > index 127b6ab..1e432fd 100644
> > --- a/src/bsd_kbd.c
> > +++ b/src/bsd_kbd.c
> > @@ -444,7 +444,6 @@ xf86OSKbdPreInit(InputInfoPtr pInfo)
> > pKbd->RemapScanCode = NULL;
> >
> > pKbd->OpenKeyboard = OpenKeyboard;
> > - pKbd->CustomKeycodes = FALSE;
> >
> > pKbd->private = calloc(sizeof(BsdKbdPrivRec), 1);
> > if (pKbd->private == NULL) {
> > diff --git a/src/hurd_kbd.c b/src/hurd_kbd.c
> > index dde5fbb..8c0cd60 100644
> > --- a/src/hurd_kbd.c
> > +++ b/src/hurd_kbd.c
> > @@ -158,7 +158,6 @@ xf86OSKbdPreInit(InputInfoPtr pInfo)
> > pKbd->KbdGetMapping = KbdGetMapping;
> > pKbd->RemapScanCode = ATScancode;
> > pKbd->OpenKeyboard = OpenKeyboard;
> > - pKbd->CustomKeycodes = FALSE;
> > pKbd->private = NULL;
> > pInfo->read_input = ReadInput;
> > return TRUE;
> > diff --git a/src/kbd.c b/src/kbd.c
> > index f4b51ff..f5cecbb 100644
> > --- a/src/kbd.c
> > +++ b/src/kbd.c
> > @@ -151,7 +151,6 @@ KbdPreInit(InputDriverPtr drv, InputInfoPtr pInfo, int
> > flags)
> > #endif
> > {
> > KbdDevPtr pKbd;
> > - MessageType from = X_DEFAULT;
> > char *s;
> > const char **defaults;
> > int rc = Success;
> > @@ -220,16 +219,8 @@ KbdPreInit(InputDriverPtr drv, InputInfoPtr pInfo, int
> > flags)
> > xkb_variant = xf86SetStrOption(pInfo->options, "XkbVariant", NULL);
> > xkb_options = xf86SetStrOption(pInfo->options, "XkbOptions", NULL);
> >
> > - pKbd->CustomKeycodes = FALSE;
> > - from = X_DEFAULT;
> > - if (xf86FindOption(pInfo->options, "CustomKeycodes")) {
> > - pKbd->CustomKeycodes = xf86SetBoolOption(pInfo->options,
> > "CustomKeycodes",
> > - pKbd->CustomKeycodes);
> > - from = X_CONFIG;
> > - }
> > -
> > - xf86Msg(from, "%s: CustomKeycodes %s\n",
> > - pInfo->name, pKbd->CustomKeycodes ? "enabled" : "disabled");
> > + pKbd->CustomKeycodes = xf86SetBoolOption(pInfo->options,
> > "CustomKeycodes",
> > + NULL);
>
> NULL is a rather unusual choice for a boolean. ACK otherwise though.
>
> Cheers,
> Peter
Before calling xf86SetBoolOption(pInfo->options, "CustomKeycodes", NULL)
we've done xf86CollectInputOptions(pInfo, defaults).
defaults = kbdDefaults or kbd98Defaults
static const char *kbdDefaults[] = {
"Protocol", "standard",
"XkbRules", "base",
"XkbModel", "pc105",
"XkbLayout", "us",
"CustomKeycodes", "off", <-------- set by default
NULL
};
static const char *kbd98Defaults[] = {
"Protocol", "standard",
"XkbRules", "xfree98",
"XkbModel", "pc98",
"XkbLayout", "jp",
"CustomKeycodes", "off", <-------- set by default
NULL
};
Thus the value of "CustomKeycodes" when calling xf86SetBoolOption will already
be set
(by default or from the config) and xf86SetBoolOption can not return NULL.
Indeed replacement of NULL will add clarity, but functionally not change
anything.
What is your opinion on this issue?
--
Alexandr Shadchin
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
