On Thu, Dec 03, 2015 at 09:02:47PM +0100, Bram Moolenaar wrote: > Jörn Engel wrote: > > > But fundamentally this patch is still wrong. If I run > > export TMPDIR='$TMPDIR' vim > > then vim should very much try '$TMPDIR' because I explicitly asked it > > to do so. Might be a silly choice, but that is mine to make. > > The path should start with a slash. And if it doesn't, I don't think > any sane person would have a directory name starting with $. > Let's just say we don't support it.
Fair point. Having relative paths to TMPDIR is crazy enough to not support it. I like your patch, modulo the typo. > Well, we could make a version of expand_env() that returns an empty > string if the expansion fails. But I don't think it is worth it. Ack. Jörn -- Invincibility is in oneself, vulnerability is in the opponent. -- Sun Tzu -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
