Subject: Re: bug#28227: 26.0.50; Tramp tests are slow

From: [email-masked]
Date: Wed, 30 Aug 2017 22:26:33 -0400

Gemini Lasswell <[email-masked]> writes:

> +(defmacro tramp-lookup-syntax (alist)
> +  "Look up a syntax string in ALIST according to `tramp-compat-tramp-syntax.'
> +Raise an error if `tramp-syntax' is invalid."
> +  `(let ((result (cdr (assq (tramp-compat-tramp-syntax) ,alist))))
> +     (or result
> +         (error "Wrong `tramp-syntax' %s" tramp-syntax))))

Was there a reason to defmacro instead of defun here?  If it's
performance, perhaps defsubst could work instead?

(also, the let-binding seems redundant)