Re: trunk r114123: * lisp/net/tramp-gvfs.el (tramp-gvfs-mount-spec, tramp-synce-list-devices):

[Michael Albinus]

Stefan Monnier <address@hidden> writes:

Hi Stefan,

> message:
>   * lisp/net/tramp-gvfs.el (tramp-gvfs-mount-spec, tramp-synce-list-devices): 
>   * lisp/net/tramp-smb.el (tramp-smb-get-file-entries):
>   * lisp/net/tramp-sh.el (tramp-sh-handle-insert-directory)
>   (tramp-compute-multi-hops): Fix misuses of `add-to-list'.

With this change (replacing `add-to-list' by `pushnew') I get compiler
warnings like this:

--8<---------------cut here---------------start------------->8---
emacs24 --no-site-file -batch -l bytecomp \
        --eval "(add-to-list 'load-path default-directory)" \
        -f batch-byte-compile tramp-sh.el

In tramp-compute-multi-hops:
tramp-sh.el:4277:31:Warning: function `adjoin' from cl package called at
    runtime
emacs24 --no-site-file -batch -l bytecomp \
        --eval "(add-to-list 'load-path default-directory)" \
        -f batch-byte-compile tramp-gvfs.el

In tramp-synce-list-devices:
tramp-gvfs.el:1718:38:Warning: function `adjoin' from cl package called at
    runtime
--8<---------------cut here---------------end--------------->8---

This does not happen for the trunk version, but for older Emacsen (like
24.2, used in this example).

Is your change worth the trouble? And if yes, how to handle those warnings?

Best regards, Michael.