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

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

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Tue, 05 Apr 2011 20:03:30 +0200

On 04.04.2011 22:57, Oto BREZINA wrote:
> Theme of patch: syncing code of BottomView.cpp, LeftView.cpp,
> RightView.cpp for later refactor common part to (BaseView.cpp)
> popup.AppendMenu(
> UINT nFlags,
> UINT_PTR nIDNewItem = 0,
> LPCTSTR lpszNewItem = NULL )
> 1. Is there any real gain to do not use nIDNewItem same value as IDS ?

That depends on whether we trust the optimizations of the compiler :)
If we use the IDS values, then the switch() could take up a lot of space
since those values are not continuous or even in the same value range.
Sure, we could edit the resource.h file manually to make sure those IDS
values are as we would like them, but I don't like messing with
generated files.

"To optimize a switch statement, the programmer must use a very compact
range of possible values to test."

> 2. Is there plan to support icons there? let say, same like in main
> menu? not worth much but nice :)

Good idea.

> 3. hope you are fine with virtual functions

Sure, as you might have seen, we use those in some places already :)


   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-04-05 20:03:40 CEST

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.