Hi Bastien,

Bastien wrote:
> "Sebastien Vauban" writes:
>
>> (if (locate-library "org-loaddefs")
>>     (require 'org-loaddefs)
>                ^^^^^^^^^^^^^
>
> You don't need this, there is (load "org-loaddefs.el" t t) in org.el.

OK. And `org.el' is loaded[1] because of the major-mode association to the
`.org' files, then, right?

Anyway, that does not have impacts here.

> Are your org elisp files compiled?

Nope. Never.

> What results do you have without the advice on `require'?

I've now ran 5 tests with each config:

    | without advice | with advice |
    |----------------+-------------|
    |           3.78 |        3.95 |
    |           3.73 |        3.86 |
    |           3.76 |        3.83 |
    |           3.80 |        3.83 |
    |           3.81 |        3.94 |
    |----------------+-------------|
    |           3.78 |        3.88 |
    #+TBLFM: @7$1..@7$2=vmean(@I..@-I);%.2f

That is less than 3% time penalty for the logs done during the execution of
all the `require'.

Best regards,
  Seb

[1] BTW, why not a `require' instead?

-- 
Sebastien Vauban


Reply via email to