Re: [TSVN][PATCH]ResizableLib C++ standard and WinAPI compliance

Stefan Fuhrmann wrote:
> Hi dev,
>
> I probably forgot this one when posting the
> initial set of TSVN patches.
>
> [[[
> Improve ResizableLib
> * C++:
> * ResizableComboBox.cpp: declare loop variables in front
> of the loop when they are also accessed after the loop block
> * ResizableFrame.cpp: resolve evaluation ambiguity
> * WinAPI (32 / 64 bit)
> * ResizableLayout.cpp: use target-aware types where appropriate
> ]]]

Committed in revision 4229, except the change to ResizableComboBox.cpp.

> Index: ResizableComboBox.cpp
> ===================================================================
> --- ResizableComboBox.cpp (revision 4223)
> +++ ResizableComboBox.cpp (working copy)
> @@ -182,6 +182,7 @@
> availh -= h;
> }
> // to fill the remaining height, use items above
> + int i = 0;
> for (i=GetTopIndex()-1; availh>0 && i>=0; i--)
> {

Have a look a few lines above this one: there's already an 'int i = 0;'
there.

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