[PATCH xserver 8/8] Remove unused vtSysreq
Alexandr Shadchin
alexandr.shadchin at gmail.com
Sat Sep 17 04:35:08 PDT 2011
On Fri, Sep 16, 2011 at 04:03:44PM -0500, Jamey Sharp wrote:
> On Thu, Sep 15, 2011 at 09:13:09PM -0700, Alan Coopersmith wrote:
> > On 09/15/11 20:51, Jamey Sharp wrote:
> > >On Thu, Sep 15, 2011 at 06:42:59PM -0700, Alan Coopersmith wrote:
> > >>On 09/15/11 13:07, Jamey Sharp wrote:
> > >>>It looks to me like you should also delete VTSYSRQ from
> > >>>hw/xfree86/parser/xf86tokens.h.
> > >>
> > >>Please ensure that any changes result in the X server simply ignoring
> > >>the VTSysReq option in existing xorg.conf and not erroring out& refusing
> > >>to start on a file that happens to have a no-longer-useful option in.
> > >
> > >If I understand the parser correctly (and I probably don't), it's
> > >already too late for that: while there's a VTSYSRQ entry in the enum in
> > >xf86tokens.h, nothing generates that token type. Instead, there's a
> > >VTSysReq ServerFlag. So deleting the token from the enum is not a
> > >regression, I think.
> > >
> > >I can't figure out at a quick glance whether unknown ServerFlags are
> > >treated as an error, though. Does the patch need to somehow make this
> > >flag be explicitly ignored if present?
> >
> > Honestly, I don't remember - probably easiest to just test starting the
> > server (when built with this series) using an xorg.conf with the entry added.
>
> I just tested 1.11 with ServerFlags 'Option "NoSuchOption" "blag"' and
> didn't get any messages about it in the log. The server started up fine.
>
> > I just remember prior issues here (see commits e0a451eb7cc & d2cf562bbad
> > from prior obsoletion - though those seem to be keywords, not options)
>
> I saw commits along those lines, but I guess they aren't relevant here.
>
> Alexandr, my comment stands: if you delete VTSYSRQ from xf86tokens.h you
> can add my Reviewed-by tag to this patch.
>
> Jamey
I agree, delete VTSYSREQ from xf86tokens.h is correct. Option VTSysReq read only
in configServerFlags() and this place will be removed. Unknown options will be
ignored, as correctly noted Jamey. Also I tested on 1.11, started up fine.
--
Alexandr Shadchin
More information about the xorg-devel
mailing list