[Emacs-diffs] /srv/bzr/emacs/trunk r111225: Explain, why `tramp-*-file-name-p' is a defsubst.

[Michael Albinus]

------------------------------------------------------------
revno: 111225
committer: Michael Albinus <address@hidden
branch nick: trunk
timestamp: Fri 2012-12-14 10:02:34 +0100
message:
  Explain, why `tramp-*-file-name-p' is a defsubst.
modified:
  lisp/net/tramp-adb.el
  lisp/net/tramp-ftp.el
  lisp/net/tramp-gvfs.el
  lisp/net/tramp-smb.el

=== modified file 'lisp/net/tramp-adb.el'
--- a/lisp/net/tramp-adb.el     2012-12-13 08:42:25 +0000
+++ b/lisp/net/tramp-adb.el     2012-12-14 09:02:34 +0000
@@ -111,6 +111,8 @@
     (start-file-process . tramp-adb-handle-start-file-process))
   "Alist of handler functions for Tramp ADB method.")
 
+;; It must be a `defsubst' in order to push the whole code into
+;; tramp-loaddefs.el.  Otherwise, there would be recursive autoloading.
 ;;;###tramp-autoload
 (defsubst tramp-adb-file-name-p (filename)
   "Check if it's a filename for ADB."

=== modified file 'lisp/net/tramp-ftp.el'
--- a/lisp/net/tramp-ftp.el     2012-06-11 10:30:07 +0000
+++ b/lisp/net/tramp-ftp.el     2012-12-14 09:02:34 +0000
@@ -200,6 +200,8 @@
                 (inhibit-file-name-operation operation))
            (apply 'ange-ftp-hook-function operation args)))))))
 
+;; It must be a `defsubst' in order to push the whole code into
+;; tramp-loaddefs.el.  Otherwise, there would be recursive autoloading.
 ;;;###tramp-autoload
 (defsubst tramp-ftp-file-name-p (filename)
   "Check if it's a filename that should be forwarded to Ange-FTP."

=== modified file 'lisp/net/tramp-gvfs.el'
--- a/lisp/net/tramp-gvfs.el    2012-12-06 09:15:27 +0000
+++ b/lisp/net/tramp-gvfs.el    2012-12-14 09:02:34 +0000
@@ -432,6 +432,8 @@
   "Alist of handler functions for Tramp GVFS method.
 Operations not mentioned here will be handled by the default Emacs 
primitives.")
 
+;; It must be a `defsubst' in order to push the whole code into
+;; tramp-loaddefs.el.  Otherwise, there would be recursive autoloading.
 ;;;###tramp-autoload
 (defsubst tramp-gvfs-file-name-p (filename)
   "Check if it's a filename handled by the GVFS daemon."

=== modified file 'lisp/net/tramp-smb.el'
--- a/lisp/net/tramp-smb.el     2012-10-23 14:26:30 +0000
+++ b/lisp/net/tramp-smb.el     2012-12-14 09:02:34 +0000
@@ -265,6 +265,8 @@
   :type 'string
   :version "24.3")
 
+;; It must be a `defsubst' in order to push the whole code into
+;; tramp-loaddefs.el.  Otherwise, there would be recursive autoloading.
 ;;;###tramp-autoload
 (defsubst tramp-smb-file-name-p (filename)
   "Check if it's a filename for SMB servers."