GNU bug report logs - #15447
(quail-help "korean-hangul") generates errors.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 15447 in the body.
You can then email your comments to 15447 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: crocket <crockabiscuit <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: (quail-help "korean-hangul") generates errors.
Date: Sun, 22 Sep 2013 18:53:25 +0900

[Message part 1 (text/plain, inline)]

The error trace is shown below.

Debugger entered--Lisp error: (wrong-type-argument char-or-string-p nil)
  quail-help("korean-hangul")
  eval((quail-help "korean-hangul") nil)
  eval-last-sexp-1(t)
  eval-last-sexp(t)
  eval-print-last-sexp()
  call-interactively(eval-print-last-sexp nil nil)
  recursive-edit()
  debug(error (wrong-type-argument char-or-string-p nil))
  quail-help("korean-hangul")
  eval((quail-help "korean-hangul") nil)
  eval-last-sexp-1(t)
  eval-last-sexp(t)
  eval-print-last-sexp()
  call-interactively(eval-print-last-sexp nil nil)
  recursive-edit()
  debug(error (void-function quail-help))
  (quail-help "korean-hangul")
  eval((quail-help "korean-hangul") nil)
  eval-last-sexp-1(t)
  eval-last-sexp(t)
  eval-print-last-sexp()
  call-interactively(eval-print-last-sexp nil nil)
  recursive-edit()
  debug(error (void-function quail-help))
  (quail-help (quote korean-hangul))
  eval((quail-help (quote korean-hangul)) nil)
  eval-last-sexp-1(t)
  eval-last-sexp(t)
  eval-print-last-sexp()
  call-interactively(eval-print-last-sexp nil nil)

[Message part 2 (text/html, inline)]

Message #8 received at 15447 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: crocket <crockabiscuit <at> gmail.com>
Cc: 15447 <at> debbugs.gnu.org
Subject: Re: bug#15447: (quail-help "korean-hangul") generates errors.
Date: Tue, 24 Sep 2013 09:25:40 +0300

> Date: Sun, 22 Sep 2013 18:53:25 +0900
> From: crocket <crockabiscuit <at> gmail.com>
> 
> The error trace is shown below.
> 
> Debugger entered--Lisp error: (wrong-type-argument char-or-string-p nil)
>   quail-help("korean-hangul")

quail-help gives help on quail _packages_, and korean-hangul is not a
package, it's an input method.

Try this instead:

  C-u C-\ korean-hanja RET
  C-\
  M-: (quail-help "korean-hanja") RET

This works because korean-hanja is a Leim package.

Message #11 received at 15447 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: crocket <crockabiscuit <at> gmail.com>
Cc: 15447 <at> debbugs.gnu.org
Subject: Re: bug#15447: (quail-help "korean-hangul") generates errors.
Date: Tue, 24 Sep 2013 23:32:51 +0300

[Please keep the bug address on the CC list.]

> Date: Wed, 25 Sep 2013 05:18:49 +0900
> From: crocket <crockabiscuit <at> gmail.com>
> 
> So, is it not a bug?

I don't think it is a bug, although it would be nicer if instead of
signaling an error Emacs would say something like "unknown package:
FOO".

Message #14 received at 15447 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 15447 <at> debbugs.gnu.org, crocket <crockabiscuit <at> gmail.com>
Subject: Re: bug#15447: (quail-help "korean-hangul") generates errors.
Date: Tue, 24 Sep 2013 18:18:44 -0400

>> So, is it not a bug?
> I don't think it is a bug, although it would be nicer if instead of
> signaling an error Emacs would say something like "unknown package:
> FOO".

I guess it also depends on how we end up calling quail-help.
After all, the user is not expected to type M-: (quail-help "foobar") RET.


        Stefan

Message #17 received at 15447 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: 15447 <at> debbugs.gnu.org, crockabiscuit <at> gmail.com
Subject: Re: bug#15447: (quail-help "korean-hangul") generates errors.
Date: Wed, 25 Sep 2013 08:43:58 +0300

> From: Stefan Monnier <monnier <at> iro.umontreal.ca>
> Cc: crocket <crockabiscuit <at> gmail.com>, 15447 <at> debbugs.gnu.org
> Date: Tue, 24 Sep 2013 18:18:44 -0400
> 
> >> So, is it not a bug?
> > I don't think it is a bug, although it would be nicer if instead of
> > signaling an error Emacs would say something like "unknown package:
> > FOO".
> 
> I guess it also depends on how we end up calling quail-help.
> After all, the user is not expected to type M-: (quail-help "foobar") RET.

It's a command, but invoking it interactively yields the same
behavior.  You need to have activated an input method that loads some
quail package, before invoking this command, in order to have a help.
korean-hangul doesn't load any quail package, so the command throws an
error.

So I still think a more human-readable error message is in order,
although when invoked interactively the package name is not given, so
some other wording is in order.

It looks like quail-help was designed to be called only by "C-h C-\",
where it behaves reasonably, but since it's a command, it perhaps
should have some minimal harness for when it is invoked by users
directly.

Message #20 received at 15447 <at> debbugs.gnu.org (full text, mbox):

From: Kenichi Handa <handa <at> gnu.org>
To: crocket <crockabiscuit <at> gmail.com>
Cc: 15447 <at> debbugs.gnu.org
Subject: Re: bug#15447: (quail-help "korean-hangul") generates errors.
Date: Wed, 25 Sep 2013 22:29:00 +0900

In article <CAL0=0H3qh1xfncjnfH92TZm5UCXVCSBouyaVe1E=QciLLD-Z-Q <at> mail.gmail.com>, crocket <crockabiscuit <at> gmail.com> writes:

> The error trace is shown below.

> Debugger entered--Lisp error: (wrong-type-argument char-or-string-p nil)
>   quail-help("korean-hangul")

As these input methods are not "quail" input methods:
  korean-hangul, korean-hangul3f, korean-hangul390, korean-hangul3,
  ucs
quail-help doesn't work for them.

If you want to describe those input methods, do:
  (describe-input-method "korean-hangul")
or, type "C-h C-\ korean hangul RET".

---
Kenichi Handa
handa <at> gnu.org

Message #23 received at 15447 <at> debbugs.gnu.org (full text, mbox):

From: Kenichi Handa <handa <at> gnu.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 15447 <at> debbugs.gnu.org, monnier <at> iro.umontreal.ca, crockabiscuit <at> gmail.com
Subject: Re: bug#15447: (quail-help "korean-hangul") generates errors.
Date: Wed, 25 Sep 2013 23:09:06 +0900

In article <83li2lh2w1.fsf <at> gnu.org>, Eli Zaretskii <eliz <at> gnu.org> writes:
> It looks like quail-help was designed to be called only by "C-h C-\",
> where it behaves reasonably, but since it's a command, it perhaps
> should have some minimal harness for when it is invoked by users
> directly.

If it is ok to break a subtle backward compatibility here,
let's make quail-help not a command.  It had to be a
command when I first implemented it, but now it doesn't have
to be.  It is now called from describe-current-input-method
by funcall.

---
Kenichi Handa
handa <at> gnu.org

Message #26 received at 15447 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: Kenichi Handa <handa <at> gnu.org>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 15447 <at> debbugs.gnu.org,
 crockabiscuit <at> gmail.com
Subject: Re: bug#15447: (quail-help "korean-hangul") generates errors.
Date: Wed, 25 Sep 2013 13:07:41 -0400

> If it is ok to break a subtle backward compatibility here,
> let's make quail-help not a command.

Fine by me,


        Stefan

Message #29 received at 15447 <at> debbugs.gnu.org (full text, mbox):

From: Kenichi Handa <handa <at> gnu.org>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Cc: eliz <at> gnu.org, 15447 <at> debbugs.gnu.org, crockabiscuit <at> gmail.com
Subject: Re: bug#15447: (quail-help "korean-hangul") generates errors.
Date: Thu, 26 Sep 2013 23:00:15 +0900

In article <jwv7ge4x229.fsf-monnier+emacsbugs <at> gnu.org>, Stefan Monnier <monnier <at> IRO.UMontreal.CA> writes:

> > If it is ok to break a subtle backward compatibility here,
> > let's make quail-help not a command.

> Fine by me,

I've just committed that change.

---
Kenichi Handa
handa <at> gnu.org

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.