Alexey I. Froloff wrote:

> * A.J.Mechelynck <antoine.mechelynck@> [060816 04:31]:
> > IIRC, in Windows percent signs should be doubled (which requires the same
> > kind of fancy-footwork as Vim's \ or ^V in the case of recursion), except
> > in the names of environment %STRINGS% to be resolved by the shell.
> > I'm not sure about how to quote double quotes.
> Well, I tried to create file
> 
> The 100% file with "quotes".txt
> 
> =66rom notepad and failed.
> 
> C:> type "The 100% file with quotes.txt"
> test
> C:> type "The 100% file with quotes.txt"
> The system cannot find the file specified.
> C:> type "The file with %percents%.txt"
> test2
> C:> type "The file with %%percents%%.txt"
> The system cannot find the file specified.
> C:> type "The file with %percents% and spaces.txt"
> test3
> C:> type "The file with %%percents%% and spaces.txt"
> The system cannot find the file specified.
> 
> And so on...
> 
> I guess stripping all quotes and leaving percents will be OK.

Are these characters allowed anyway?  In Unix any character except "/"
is allowed in a file name.  On MS-Windows certain characters are not
allowed.  The help mentions / \ : * ? " < > and |.  So percent needs to
be escaped, but double quote not, it can't be part of a file name.

> Is "defined(WIN16) || defined(WIN32) || defined(DOS)" will be
> enough for DOS/WIN platform?

I think it should be:

        #if defined(MSDOS) || defined(MSWIN)

We also need to check the value of 'shell'.  If it's a Unix-like shell
then % probably should not be doubled.

> > >A patch for the documentation is missing, this is required to see what
> > >the intention of shellescape() is.
> OK, I'll update patch.

Received it, thanks.

> P.S.  I don't have cproto and can't "make proto"...

I'll take care of that.

-- 
hundred-and-one symptoms of being an internet addict:
152. You find yourself falling for someone you've never seen or hardly
     know, but, boy can he/she TYPE!!!!!!

 /// Bram Moolenaar -- [EMAIL PROTECTED] -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Reply via email to