|
From: | Dmitry Gutov |
Subject: | Re: [SUSPECTED SPAM] Re: [Emacs-diffs] scratch/widen-less a4ba846: Replace prog-widen with consolidating widen calls |
Date: | Fri, 22 Dec 2017 00:07:28 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:58.0) Gecko/20100101 Thunderbird/58.0 |
On 12/21/17 3:33 AM, John Wiegley wrote:
And I wouldn't really call using define-inline instead of defun for tiny functions "premature optimization" in cases like these. You're not changing the semantics of the function, just indicating it's small enough that it should be inlined when byte-compiling. And you know the number of call sites too, so it won't cause code bloat. When it's that harmless, I usually do it as a matter of convention in my own code.
It's simply extra work. Also see what the manual says: https://www.gnu.org/software/emacs/manual/html_node/elisp/Inline-Functions.html, "you should not make a function inline...".
[Prev in Thread] | Current Thread | [Next in Thread] |