Mattias Gaertner napisał(a):
On Mon, 13 Mar 2006 22:18:56 +0100
darekM <[EMAIL PROTECTED]> wrote:

  
Vincent Snijders napisa__(a):
    
darekm wrote:
      
Attached patch cause that OnSelChange for tComboBox is fired only one 
time per click

beside I enable some of trace event
Darek


Index: interfaces/gtk/gtkwinapi.inc
===================================================================
--- interfaces/gtk/gtkwinapi.inc    (wersja 8935)
+++ interfaces/gtk/gtkwinapi.inc    (kopia robocza)
@@ -2795,13 +2795,13 @@
       end;
       Self.WordWrap(DC, Str, MaxLength, Lines, NumLines);
 
-      LineWidth := 0;
-      If (Lines <> nil) then begin
+      If (Lines <> nil) and (NumLines>1) then begin
+        LineWidth := 0;
         For J := 0 to NumLines - 1 do begin
           GetTextExtentPoint(DC, Lines[J], StrLen(Lines[J]), AP);
           LineWidth := Max(LineWidth, AP.cX);
         end;
-      end;
+      end else lineWidth:=10000;
       LineWidth := Min(MaxLength, LineWidth);
 
       theRect.Right := theRect.Left + LineWidth;
        
Are you sure you want to revert this recent change? 
      
Why count ExtendPoint for  one line label text ?
    

The question is: Why not?
  
Speed?
If Str has only one line then
GetTextExtentPoint(DC, Lines[J], StrLen(Lines[J]), AP);
and
GetTextExtentPoint(DC, Str, Count, AP)
return the same AP


number 10000 may be greater
 
For example MessageDlg gets maximum size with the above 10000 instead of the
right width.

  
Darek

Reply via email to