Ship Mints <shipmints@gmail.com> writes:
> Latest patch attached.
And still nitpicks :-)
They're all thoughtful and welcome comments. Good engineers should be nitpicky.
non-@code{nil}
Done.
> +@xref{Top, The Tramp Manual,, tramp, The Tramp Manual}. Also see the
> +more detailed documentation available here
> +@url{https://www.gnu.org/software/tramp/}.
??? Both the info manual and the HTML page are the same text.
Done. This was just ignorance on my part, not knowing the tramp manual was in a different doc subdirectory.
My previous comment still stands: Use `connection-local-value'. Did you check?
Got it. It was an oversight.
`shell-mode'
Done.
> +For a remote shell `default-directory' will be the remote file name.
> +Remote shell buffers reuse existing connections that match the remote
> +file name, or may prompt you to create a new connection. Bind
> +`tramp-show-ad-hoc-proxies' to non-nil to ensure multi-hop remote
> +connections are fully qualified.
non-@code{nil}
I see references to non-nil without quotes all over the place in docstrings. Further, checkdoc warns about not putting t or nil in quotes.
Latest revision attached.
-Stephane