Login | Register
My pages Projects Community openCollabNet

Discussions > dev > Re: [T-Merge patch] syncing code for views

Project highlights: :. Download .: :. Support .: :. FAQ .: :. Translations .: :. Donate .: :. Report Bug .:

tortoisesvn
Discussion topic

Back to topic list

Re: [T-Merge patch] syncing code for views

Author otik
Full name Oto BREZINA
Date 2011-04-08 13:10:05 PDT
Message On 2011-04-08 21:00, Stefan Küng wrote:
> On 08.04.2011 00:23, Oto BREZINA wrote:
>> Part4:
>> Introduce and use ResetUndoStep, SaveUndoStep and m_Allstate to handle
>> undo step build time
>> BaseView: Use HasSelection() where applicable
> Some comments:
> * doesn't compile
I don't have VC, so that was bit expected. I thought to buy it, but it
is too expensive to use that occasionally. All I have is T-Merge :)
> * style is wrong ( "{" on same line instead of the next line, ...)
we use that one, so sorry, I know you have only few requerement for
patch and this is one of them :)
reference:
http://tortoisesvn.t​igris.org/ds/viewMes​sage.do?dsForumId=75​7&dsMessageId=25​87752

> Just when I thought you're ready for commit access :/
I thought to ask you for it, but only for poChecker - code changes
become quite big.
I like to use pair programing especially with code "owner"/main maintainer.
> I fixed, modified it slightly and committed it in r21127.
I hope I do more work then make(prepare for you) :)

I missed that dynamic_cast is needed, do you know about its overhead?
All (most of) that function can be moved to parent class, but it become
a big mess, so I like to keep their place just remove duplicates if
possible.

This also lead me to question:
CLeftView::UseFile handle DIFFSTATE_CONFLICTEMPTY as special state to be
resolved into DIFFSTATE_CONFLICTRE​SOLVEDEMPTY, unlike
CRightView::UseFile, CBottomView::UseTheirTextBlock,
CBottomView::UseMyTextBlock.
Also in two-panel view are BuildAllScreen2ViewVector()
RecalcAllVertScrollBars() calls missing.

> Stefan
>

--
Oto BREZINA, Printflow s.r.o., EU

« Previous message in topic | 14 of 27 | Next message in topic »

Messages

Show all messages in topic

[T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-04 13:57:21 PDT
     Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-04-05 11:03:39 PDT
         Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-05 14:10:14 PDT
             Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-04-06 09:26:00 PDT
                 Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-09 12:54:25 PDT
             Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-06 15:12:10 PDT
                 Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-04-07 10:16:41 PDT
                     Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-07 12:03:24 PDT
                         Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-04-07 12:42:35 PDT
                             Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-07 13:20:49 PDT
                                 Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-04-07 13:30:40 PDT
                                     Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-07 15:23:20 PDT
                                         Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-04-08 12:01:02 PDT
                                             Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-08 13:10:05 PDT
                                                 Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-04-09 00:17:09 PDT
                                                     Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-09 06:50:02 PDT
         Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-17 00:31:07 PDT
             Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-04-17 02:46:45 PDT
                 Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-05-11 13:54:50 PDT
                     Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-05-12 12:52:58 PDT
     Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-04-10 02:14:27 PDT
     Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-04-10 02:15:30 PDT
         Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-10 12:08:55 PDT
             Re: [T-Merge patch] syncing code for views steveking Stefan Küng 2011-04-10 12:14:39 PDT
                 Re: [T-Merge patch] syncing code for views otik Oto BREZINA 2011-04-10 12:22:04 PDT
Page: of 2 « Previous | Next »
Messages per page: