darekm wrote:
Hi
attached patch added:
 ChecklistBox.onClickCheck (for GTK and GTK2)
publish property  TCheckBox.font
remove tCustomLabel.FontChange(Sender: TObject); (default is  fontchanged)



Thanks for the patch. I have the following questions:

Why is the following debugln commented out? Doesn't calling DestroyHandle, if the handle is not allocated, mean that there is a bug somewhere? Raising the exception is maybe too harsh, but silently ignoring it, doesn't seem a good idea to me.

@@ -3287,7 +3292,8 @@
   AWinControl: TWinControl;
 begin
   if not HandleAllocated then begin
-    DebugLn('Warning: TWinControl.DestroyHandle ',Name,':',ClassName,' Handle 
not Allocated');
+    //DebugLn('Warning: TWinControl.DestroyHandle ',Name,':',ClassName,' 
Handle not Allocated');
+    exit;
     //RaiseGDBException('');
   end;

<snip>

@@ -132,6 +139,12 @@
   FItemDataOffset := inherited GetCachedDataSize;
 end;
+
+procedure tCustomCheckListBox.DoChange(var Msg);

<snip>

+procedure tCustomCheckListBox.ClickChecked;

Why not TCustomCheckListBox.DoChange and TCustomCheckListBox.ClickChecked?

Vincent

_________________________________________________________________
    To unsubscribe: mail [EMAIL PROTECTED] with
               "unsubscribe" as the Subject
  archives at http://www.lazarus.freepascal.org/mailarchives

Reply via email to