utf-translate-cjk only settable via Customize

utf-translate-cjk only settable via Customize

[Kai Großjohann]

Should a function for setting the variable be created?

My suggestion would be to create a function utf-translate-cjk-mode
based on the :set expression for utf-translate-cjk and then change
the :set expression to invoke that function.

OK to fix?

(There is a relation to the custom-setq discussion currently going
on -- when custom-setq is added to Emacs, the previously described
change might not be necessary anymore.)
-- 
A preposition is not a good thing to end a sentence with.

Re: utf-translate-cjk only settable via Customize

[Stefan Monnier]

> My suggestion would be to create a function utf-translate-cjk-mode
> based on the :set expression for utf-translate-cjk and then change
> the :set expression to invoke that function.

Why not make it into a minor mode ?


	Stefan

Re: utf-translate-cjk only settable via Customize

[Kai Großjohann]

"Stefan Monnier" <monnier+gnu/emacs@rum.cs.yale.edu> writes:

>> My suggestion would be to create a function utf-translate-cjk-mode
>> based on the :set expression for utf-translate-cjk and then change
>> the :set expression to invoke that function.
>
> Why not make it into a minor mode ?

That's a good idea in principle, and I've now started on it.
However, the original defcustom contains a :set-after keyword and a
:version keyword.  Can these be put into the define-minor-mode?

[time passes]

The keyword :extra-args for define-minor-mode is not documented.

[pondering]

It ought to be easy to augment define-minor-mode to grok more keyword
args.  But is this the right approach?  Do we want a
:extra-defcustom-args thing that contains a list of those additional
args to defcustom?
-- 
A preposition is not a good thing to end a sentence with.

Re: utf-translate-cjk only settable via Customize

[Stefan Monnier]

> >> My suggestion would be to create a function utf-translate-cjk-mode
> >> based on the :set expression for utf-translate-cjk and then change
> >> the :set expression to invoke that function.
> >
> > Why not make it into a minor mode ?
> 
> That's a good idea in principle, and I've now started on it.
> However, the original defcustom contains a :set-after keyword and a
> :version keyword.  Can these be put into the define-minor-mode?

IIRC, additional keyword args are just passed on to defcustom.
I did this specifically for :version.


	Stefan

Re: utf-translate-cjk only settable via Customize

[Kai Großjohann]

"Stefan Monnier" <monnier+gnu/emacs@rum.cs.yale.edu> writes:

>> That's a good idea in principle, and I've now started on it.
>> However, the original defcustom contains a :set-after keyword and a
>> :version keyword.  Can these be put into the define-minor-mode?
>
> IIRC, additional keyword args are just passed on to defcustom.
> I did this specifically for :version.

I was confused by the following bit of code in define-minor-mode:

    ;; Check keys.
    (while (keywordp (car body))
      (case (pop body)
	(:init-value (setq init-value (pop body)))
	(:lighter (setq lighter (pop body)))
	(:global (setq globalp (pop body)))
	(:extra-args (setq extra-args (pop body)))
	(:group (setq group (nconc group (list :group (pop body)))))
	(:require (setq require (pop body)))
	(t (pop body))))

It seemed to me that the `t' case just throws away the keyword and its
arg.

Hm.  Now I'm looking again, at the generated defcustom statement, and
I really can't see the extra args being passed :-/  Is it just me
being stupid?
-- 
A preposition is not a good thing to end a sentence with.

Re: utf-translate-cjk only settable via Customize

[Stefan Monnier]

> I was confused by the following bit of code in define-minor-mode:
> 
>     ;; Check keys.
>     (while (keywordp (car body))
>       (case (pop body)
> 	(:init-value (setq init-value (pop body)))
> 	(:lighter (setq lighter (pop body)))
> 	(:global (setq globalp (pop body)))
> 	(:extra-args (setq extra-args (pop body)))
> 	(:group (setq group (nconc group (list :group (pop body)))))
> 	(:require (setq require (pop body)))
> 	(t (pop body))))
> 
> It seemed to me that the `t' case just throws away the keyword and its arg.

That's the RC code.  The trunk code is different.


	Stefan

Re: utf-translate-cjk only settable via Customize

[Richard Stallman]

    It ought to be easy to augment define-minor-mode to grok more keyword
    args.  But is this the right approach?  Do we want a
    :extra-defcustom-args thing that contains a list of those additional
    args to defcustom?

Would you please explain what you are trying to achieve?
I don't see what purpose any of this would serve.

Re: utf-translate-cjk only settable via Customize

[Kai Großjohann]

Richard Stallman <rms@gnu.org> writes:

>     It ought to be easy to augment define-minor-mode to grok more keyword
>     args.  But is this the right approach?  Do we want a
>     :extra-defcustom-args thing that contains a list of those additional
>     args to defcustom?
>
> Would you please explain what you are trying to achieve?
> I don't see what purpose any of this would serve.

The variable utf-translate-cjk must be set via Customize to achieve
the desired effect.  Stefan suggested to make a minor mode such that
people can just invoke that via Lisp and don't have to use Customize.
I was following this discussion and looked at define-minor-mode to do
that.  So I wanted to replace the current (defcustom utf-translate-cjk
...) statement with a (define-minor-mode utf-translate-cjk-mode ...)
statement.

The defcustom for utf-translate-cjk contains several keyword args:
:set, :version, :type, :set-after, and :group.  define-minor-mode is
a macro that expands to a defcustom statement.  I couldn't see how to
inject the :version and :set-after keywords into the defcustom
statement generated by define-minor-mode.

Of course, I could also do it manually, as follows:

* Change the :set arg for the current defcustom utf-translate-cjk to
  invoke utf-translate-cjk-mode, instead of the current code.

* Write a new function utf-translate-cjk-mode that does what the
  current lambda expression for :set does.

But what's the point of a convenience macro like define-minor-mode
when I circumvent it?
-- 
A preposition is not a good thing to end a sentence with.

Re: utf-translate-cjk only settable via Customize

[Richard Stallman]

    The defcustom for utf-translate-cjk contains several keyword args:
    :set, :version, :type, :set-after, and :group.  define-minor-mode is
    a macro that expands to a defcustom statement.  I couldn't see how to
    inject the :version and :set-after keywords into the defcustom
    statement generated by define-minor-mode.

I see now.  I gues adding them to define-minor-mode is ok.

Re: utf-translate-cjk only settable via Customize

[Kai Großjohann]

kai.grossjohann@uni-duisburg.de (Kai Großjohann) writes:

> Should a function for setting the variable be created?

As Stefan suggested, I've now made it into a (global) minor mode.
-- 
A preposition is not a good thing to end a sentence with.