bug#5392: 23.1.91; -Q becomes unknown if -displat is given.

bug#5392: 23.1.91; -Q becomes unknown if -displat is given.

[Jan =?UTF-8?Q?Dj=C3=A4rv]

Start emacs:
% emacs -Q -display :0

A message appears in the minibuffer: Unknown option -Q.
This is from command-line-1.  -Q has not been handeled since
sort_args seems to always put -display before -Q (and others) handled
in startup.el.  Thus the handling of option stops when -display is seen
and -Q gets passed to command-line-1, which chould not happen.

I don't understand the comment in emacs.c:
  /* -d must come last before the options handled in startup.el.  */
  { "-d", "--display", 60, 1 },
  { "-display", 0, 60, 1 },

Changing the prio to 6 cures this bug, but the comment seems to indicate
this is the wrong thing to do.


In GNU Emacs 23.1.91.1 (x86_64-unknown-linux-gnu, GTK+ Version 2.18.3)
 of 2010-01-15 on gaffa
Windowing system distributor `The X.Org Foundation', version 11.0.10604000
configured using `configure  '--prefix=/opt/emacs-cvs' '--verbose' '--enable-asserts' 'CFLAGS=-g''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: C
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: sv_SE.UTF-8
  value of $XMODIFIERS: nil
  locale-coding-system: utf-8-unix
  default enable-multibyte-characters: t

Major mode: Lisp Interaction

Minor modes in effect:
  delete-selection-mode: t
  icomplete-mode: t
  display-time-mode: t
  tooltip-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
M-x r e p o r t - e m <tab> <return>

Recent messages:
Loading /home/jhd/lib/elisp/BAK-file.el (source)...done
Loading /home/jhd/lib/elisp/ccsetup.el (source)...done
Loading icomplete...done
Loading /home/jhd/lib/elisp/saveframe.el (source)...done
For information about GNU Emacs and the GNU system, type C-h C-a.
command-line-1: Unknown option `-Q'

Load-path shadows:
None found.

Features:
(shadow sort mail-extr message sendmail ecomplete rfc822 mml mml-sec
password-cache mm-decode mm-bodies mm-encode mailcap mail-parse rfc2231
rfc2047 rfc2045 qp ietf-drums mailabbrev nnheader gnus-util netrc
time-date mm-util mail-prsvr gmm-utils wid-edit mailheader canlock sha1
hex-util hashcash mail-utils emacsbug desktop msb delsel icomplete
cc-mode cc-fonts easymenu cc-menus cc-cmds cc-styles cc-align cc-engine
cc-vars cc-defs regexp-opt time tooltip ediff-hook vc-hooks
lisp-float-type mwheel x-win x-dnd font-setting tool-bar dnd fontset
image fringe lisp-mode register page menu-bar rfn-eshadow timer select
scroll-bar mldrag mouse jit-lock font-lock syntax facemenu font-core
frame cham georgian utf-8-lang misc-lang vietnamese tibetan thai
tai-viet lao korean japanese hebrew greek romanian slovak czech european
ethiopic indian cyrillic chinese case-table epa-hook jka-cmpr-hook help
simple abbrev loaddefs button minibuffer faces cus-face files
text-properties overlay md5 base64 format env code-pages mule custom
widget hashtable-print-readable backquote make-network-process dbusbind
system-font-setting font-render-setting gtk x-toolkit x multi-tty emacs)

bug#5392: 23.1.91; -Q becomes unknown if -displat is given.

[Chong Yidong]

> % emacs -Q -display :0
>
> A message appears in the minibuffer: Unknown option -Q.
> This is from command-line-1.  -Q has not been handeled since
> sort_args seems to always put -display before -Q (and others) handled
> in startup.el.  Thus the handling of option stops when -display is seen
> and -Q gets passed to command-line-1, which chould not happen.
>
> I don't understand the comment in emacs.c:
>   /* -d must come last before the options handled in startup.el.  */
>   { "-d", "--display", 60, 1 },
>   { "-display", 0, 60, 1 },
>
> Changing the prio to 6 cures this bug, but the comment seems to indicate
> this is the wrong thing to do.

The --display arg is a little funny, because it is handled partly in C
and partly in Lisp.  That's why it needs to have high priority.  I've
tweaked `command-line' to handle it specially.  Thanks for noticing
this.