From: Emanuel Berg <incal@dataswamp.org>
To: help-gnu-emacs@gnu.org
Subject: Re: Function to find symlink target
Date: Thu, 19 May 2022 01:00:34 +0200 [thread overview]
Message-ID: <87wneixx7x.fsf@dataswamp.org> (raw)
In-Reply-To: courier.0000000062855D51.00003459@stw1.rcdrun.com
Jean Louis wrote:
> (defun rcd-dired-find-symlink ()
> "Find target of a symlink"
> (interactive)
> (when (dired-get-marked-files)
> (let ((file (car (dired-get-marked-files))))
> (if (file-symlink-p file)
> (let* ((file (file-truename file))
> (directory (file-name-directory file))
> (file (file-name-nondirectory file))
> (back (length file)))
> (find-file directory)
> (search-forward file)
> (backward-char back))
> (message "Not a symlink: %s" file)))))
>
> Let me know if I should improve this by any means.
Hello again Jean!
1. Do auto-indentation ...
2. `require' is needed for `dired-get-marked-files', do
byte-compile to find out and always do that before
posting here.
3. The defun has a confusing name, you have already found the
symlink one would think?
4. Emacs thinks of the docstring that the "[f]irst sentence
should end with punctuation". Always do the style check
before posting here BTW, here's an example how. [1]
5. `dired-get-marked-files' is invoked twice, instead just do
it once and reuse the result with and in `let'.
6. "file" appears as a let-binding twice in `let*', confusing
and probably no good reason to do so either, right?
7. I'm unsure but if you `find-file' a directory that is
already open in a buffer, you still end up at `point-min'?
If you don't, you need to got there before the search,
(goto-char (point-min)) as you know.
8. Instead of doing `backward-char', do something like this:
[you can try it below as well]
(when (search-forward "key" (point-max) t)
(goto-char (match-beginning 0)) )
key
Wow, 8 paragraphs! That's what I call feedback ...
[1] https://dataswamp.org/~incal/emacs-init/ide/pack-style.el
--
underground experts united
https://dataswamp.org/~incal
next prev parent reply other threads:[~2022-05-18 23:00 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-05-18 20:55 Function to find symlink target Jean Louis
2022-05-18 23:00 ` Emanuel Berg [this message]
2022-05-22 13:52 ` Jean Louis
2022-05-22 18:02 ` Emanuel Berg
2022-05-23 9:05 ` Jean Louis
2022-05-23 20:46 ` Emanuel Berg
2022-05-24 0:59 ` Emanuel Berg
2022-05-24 1:09 ` Michael Heerdegen
2022-05-24 2:14 ` Emanuel Berg
2022-05-24 2:27 ` Michael Heerdegen
2022-05-24 8:04 ` Emanuel Berg
2022-05-24 8:52 ` Michael Heerdegen
2022-05-25 5:54 ` Michael Heerdegen
2022-05-27 2:25 ` Jean Louis
2022-05-27 3:31 ` Michael Heerdegen
2022-05-27 8:00 ` Jean Louis
2022-05-27 14:47 ` Emanuel Berg
2022-05-28 6:43 ` Jean Louis
2022-05-28 15:29 ` Emanuel Berg
2022-05-28 15:43 ` Emanuel Berg
2022-05-28 16:08 ` Eli Zaretskii
2022-05-28 16:46 ` Emanuel Berg
2022-05-28 16:55 ` Eli Zaretskii
2022-05-28 17:05 ` Emanuel Berg
2022-05-31 6:27 ` Jean Louis
2022-05-31 12:24 ` Emanuel Berg
2022-05-31 6:31 ` Jean Louis
2022-05-31 12:16 ` Emanuel Berg
2022-06-01 7:43 ` Jean Louis
2022-06-02 0:40 ` Emanuel Berg
2022-06-02 5:15 ` Jean Louis
2022-06-05 5:40 ` Emanuel Berg
2022-05-25 5:05 ` Jean Louis
2022-05-25 22:33 ` Emanuel Berg
2022-05-29 2:01 ` Emanuel Berg
2022-05-19 23:14 ` Nick Dokos
2022-05-20 0:42 ` Emanuel Berg
2022-05-23 8:17 ` Jean Louis
2022-05-23 22:49 ` Michael Heerdegen
2022-05-23 22:57 ` Emanuel Berg
2022-05-24 0:59 ` Michael Heerdegen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87wneixx7x.fsf@dataswamp.org \
--to=incal@dataswamp.org \
--cc=help-gnu-emacs@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.