Re: master 8fd97b1 1/2: Fix warning generated by indian.el + quail.el

[Stefan Monnier <monnier@iro.umontreal.ca>]

>> Would the patch below fix this problem more directly?
>
> [...]
>
>> -	    (while (re-search-forward "^[ \t]*(quail-define-package" nil t)
>> +	    (while (re-search-forward "^[ \t]*(quail-define-package[ \t\n]*\"" nil t)
>
> I'm not sure?  This all ends up as a (register-input-method ...) call,
> which is a function, from which I inferred that literal strings aren't
> necessarily required here (i.e., that you can quail-define-package with
> a variable name as the first element).  

But IIRC those that we want to extract are specifically those that
are already "fully computed".  In theory we could miss some things like

    (let ((name "foo"))
      (quail-define-package name ...))

but that seems very unlikely.


        Stefan