[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [Patch] org-display-inline-images: Add support for remote images
From: |
Michael Albinus |
Subject: |
Re: [O] [Patch] org-display-inline-images: Add support for remote images |
Date: |
Sat, 29 Nov 2014 11:50:50 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) |
Kit-Yan Choi <address@hidden> writes:
> But is it theoretically possible to have two remote file paths sharing
> the same local copy file name? i.e. `make-temp-file' generates the
> same temporary file name for two separate calls?
No, never ever. After choosing a random file name, make-temp-file checks
whether there exists already such a file. In case of yes, it chosses
another random file name, and so on.
> As far as org-mode is concerned, and assuming the current stable
> version of tramp, I think there are two options (1) the current
> approach which preserves the path of the remote file and therefore
> avoid unnecessary downloads; however we forgo the capability of
> compression that tramp's file-local-copy offers, (2) use
> `file-local-copy' but risk overwriting (albeit unlikely) downloaded
> local copy; this is possible with or with out a look-up table for
> reusing files.
Again, the risk of overwriting something else does not exist.
> Thanks,
> Kit
Best regards, Michael.
- [O] [Patch] org-display-inline-images: Add support for remote images, Kit-Yan Choi, 2014/11/21
- Re: [O] [Patch] org-display-inline-images: Add support for remote images, Nicolas Goaziou, 2014/11/25
- Re: [O] [Patch] org-display-inline-images: Add support for remote images, Kit-Yan Choi, 2014/11/25
- Re: [O] [Patch] org-display-inline-images: Add support for remote images, Rasmus, 2014/11/25
- Re: [O] [Patch] org-display-inline-images: Add support for remote images, Kit-Yan Choi, 2014/11/25
- Re: [O] [Patch] org-display-inline-images: Add support for remote images, Michael Albinus, 2014/11/25
- Re: [O] [Patch] org-display-inline-images: Add support for remote images, Kit-Yan Choi, 2014/11/25
- Re: [O] [Patch] org-display-inline-images: Add support for remote images, Michael Albinus, 2014/11/25
- Re: [O] [Patch] org-display-inline-images: Add support for remote images, Kit-Yan Choi, 2014/11/26
- Re: [O] [Patch] org-display-inline-images: Add support for remote images,
Michael Albinus <=