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

RE: Re: Feature request: add a cancel button to RepoBrowser window

From: <webpost_at_tigris.org>
Date: Tue, 10 Jun 2014 00:56:35 -0700 (PDT)

Hello,

I gave a try to the version at http://nightlybuilds.tortoisesvn.net/1.8.x/win32/full/TortoiseSVN-1.8.7.25522-dev-win32-svn-1.8.9.msi
but it doesn't seem to work. Do you confirm it is supposed to?
I analyzed the code changes, but I didn't find any reason why the Cancel button remains not visible on my side (Spy++ confirmed that the Cancel button was well present but not having the WS_VISIBLE attribute, despite I started TortoiseProc.exe /command:repobrowser with the /outfile argument).

I know it has probably nothing to do with the non-functioning I observe, but I have a question/remark about the code:

I wondered what the code at http://code.google.com/p/tortoisesvn/source/browse/trunk/src/TortoiseProc/RepositoryBrowser.cpp?spec=svn25510&r=25510#725 (lines 725 & 726) is for? I don't manage to see an execution path leading to the Cancel button being still visible at this step in stand-alone mode. Am I wrong?

If the code is actually needed, it seems that there is a small typo error on the argument provided to the ShowWindow() function (probably without consequence on the behavior, because it turns that FALSE == SW_HIDE).

Best regards.

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=3083004

To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2014-06-10 09:58:53 CEST

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