|
From: | Zhongwei Yao |
Subject: | Re: Enable adb port in tramp adb |
Date: | Fri, 16 Jan 2015 00:58:09 +0800 |
Ashi <address@hidden> writes:
> Hi, Michael,
Hi Zhongwei,
> This update includes following change:
> - Fix the auto completion bug.
Now that your FSF assignment is complete (congratulations!), I did a
more granular review + test of your patch. Some remarks:
- I still would like to call the entry in tramp-methods tramp-default-port.
We shall not introduce different entries for the same thing, and
tramp-default-port does exist already.
- I've added traces to tramp-adb-parse-device-names, and now it's
visible that this is called very often. Not so good with performance
in mind; maybe we could do something in the future to change it. Call
it less often, cache the result, whatever.
I have also changed it to return "host#port" entries. So we could work
with this syntax internally. By this, I could remove
tramp-adb-parse-device-names-for-completion.
- I have rewritten tramp-adb-get-host-for-execution (new name is
tramp-adb-get-device) in order to simplify it. Works for me, but I'm
not sure whether it is OK for all use cases. Please check!
It uses now also caching. tramp-adb-search-host-in-devices is not
needed anymore.
- In tramp-adb-maybe-open-connection I have applied changes resulting
from the previous ones. I have also removed some checks for devices;
hopefully I haven't removed too much.
Could you, please, test the updated patch? If it is fine with you, I
would commit it in your name. I would need ChangeLog entries from you.
> Best regards,
> Zhongwei
Best regards, Michael.
[Prev in Thread] | Current Thread | [Next in Thread] |