[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [TSVN] Improper quoting when running external diff/merge?

From: SteveKing <steveking_at_gmx.ch>
Date: 2005-03-03 17:57:43 CET

Brian Wong wrote:

> I have some odd quoting behavior to report.
[snip]
> The offending code seems to be in /trunk/src/TortoiseProc/Utils.cpp in
> CUtils::StartExtDiff() (and similar code in CUtils::StartExtMerge()) as
> follows:
>
> // is there an extension specific diff tool?
> CRegString difftool(_T("Software\\TortoiseSVN\\DiffTools\\") +
> file2.GetFileExtension().MakeLower());
> if (CString(difftool) != "")
> {
> viewer = difftool;
> viewer = _T("\"") + viewer + _T("\"");
> }
>
> If I'm reading this right, it reads the registry key which contains
> "C:\bin\externaldiff.exe %base %mine", then it quotes the entire string.
> However it looks like it should be quoting only the executable.
>
> If I may suggest a fix, either:
> * Separate the program name and parameters into separate fields, OR
> * Don't quote the string.

Thanks for the report - fixed in revision 2765.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Thu Mar 3 17:58:16 2005

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.