Re: summary for the remaining byte compiler warnings

[Stefan Monnier]

> These probably need fixing:

> In toplevel form:
> simple.el:2564:2:Warning: `make-variable-frame-local' is an obsolete function
>     (as of Emacs 22.2); use a frame-parameter instead
> simple.el:2592:2:Warning: `make-variable-frame-local' is an obsolete function
>     (as of Emacs 22.2); use a frame-parameter instead

Yes, but it's not urgent.

> What's the best way to silence this one?

> In tex-mode:
> tex-mode.el:910:8:Warning: function `tex-mode' defined multiple times in this
>     file

No idea (I do know about it since I caused it and decided it was better
than other alternatives, and even convinced Richard of that as well).

> Not sure what to do about this:

> In cperl-mode:
> cperl-mode.el:1793:9:Warning: `vc-header-alist' is an obsolete variable; use
>     `vc-BACKEND-header' instead.
> cperl-mode.el:1805:25:Warning: reference to free variable
>     `compilation-error-regexp-alist'
> cperl-mode.el:1806:19:Warning: assignment to free variable
>     `compilation-error-regexp-alist'

Better leave cperl-mode.el alone unless it's a bug-fix.

> How about this?

> In toplevel form:
> python.el:2067:1:Warning: reference to free variable
>     `python-mode-abbrev-table'

Here the problem is that the byte-compiler doesn't understand that

   (define-abbrev-table 'python-mode-abbrev-table ()
     "Abbrev table for Python mode.
   The default contents correspond to the elements of `python-skeletons'."
     ;; Allow / in abbrevs.
     :regexp "\\<\\([[:word:]/]+\\)\\W*")

defines the `python-mode-abbrev-table' variable.  I believe I've just
fixed it.

> These probably need fixing:

> In toplevel form:
> viper-init.el:433:6:Warning: `make-variable-frame-local' is an obsolete
>     function (as of Emacs 22.2); use a frame-parameter instead
> viper-init.el:440:6:Warning: `make-variable-frame-local' is an obsolete
>     function (as of Emacs 22.2); use a frame-parameter instead
> viper-init.el:450:6:Warning: `make-variable-frame-local' is an obsolete
>     function (as of Emacs 22.2); use a frame-parameter instead
> viper-init.el:455:6:Warning: `make-variable-frame-local' is an obsolete
>     function (as of Emacs 22.2); use a frame-parameter instead

Even less urgent.


        Stefan