Michael Albinus <[email protected]> writes:
>
> (Somehow, I failed to read your second diff. Sorry for that)

The third part too below?  I don't think the defadvice should change the
return value for non-tramp names.

Actually "(list name)" seems doubtful in the tramp cases too.  If
there's no matches I think the return should be nil, not the pattern.
But I say that without trying it, and without knowing if it's a hack to
keep the pattern for later action ...


2009-11-15  Kevin Ryde  <[email protected]>

        * tramp.el (defadvice file-expand-wildcards): For non-tramp names
        don't change the return to (list name), leave per the original.

--- tramp.el.~2.748.~	2009-11-15 10:51:30.000000000 +1100
+++ tramp.el	2009-11-15 10:55:28.000000000 +1100
@@ -8376,9 +8376,7 @@
 		  (setq ad-return-value (list name))))
 	    (setq ad-return-value (list name)))
 	;; If it is not a Tramp file, just run the original function.
-	ad-do-it
-	(unless ad-return-value
-	  (setq ad-return-value (list name))))))
+	ad-do-it)))
   (add-hook
    'tramp-unload-hook
    (lambda ()
_______________________________________________
Tramp-devel mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/tramp-devel

Reply via email to