Re: tramp-compat-funcall -> compat-funcall?

[Stefan Monnier]

> ;; For not existing functions, obsolete functions, or functions with a
> ;; changed argument list, there are compiler warnings.  We want to
> ;; avoid them in cases we know what we do.
> (defmacro tramp-compat-funcall (function &rest arguments)
>   "Call FUNCTION if it exists.  Do not raise compiler warnings."
>   `(when (or (subrp ,function) (functionp ,function))
>      (with-no-warnings (funcall ,function ,@arguments))))

[ The `subrp' check looks wrong/redundant.  `functionp' should already return
  non-nil if `function` is a subr (unless it's a special form, in which
  case using `funcall` would be wrong anyway).  ]

FWIW, using tramp-compat-funcall "for not existing functions" is a bad
idea: better use (if (fboundp <foo>) (<foo> ...)).

I think the same kind of consideration should hold for the other two
cases, tho maybe currently the byte-compiler does not offer/detect any
syntax for that.

I think hiding this behind (tramp-)compat-funcall would be better
avoided (just like using with-no-warnings should be avoided whenever
possible).

IOW, if there's a kind of situation that recurs often enough to warrant
something like (tramp-)compat-funcall you should report this as a bug.


        Stefan