From: Mark D. <mar...@zn...> - 2007-03-22 02:08:19
Attachments:
listctrl.diff
|
Hi, Attached diff fixes documented problem with Wx::Perl::ListCtrl->SetItemData being called LONG_MAX or more times. It also includes tests too. Fix involves reseting index values for all items every 2147483640th time SetItemData is called. Knowing that if I had a Wx::Perl::ListCtrl open on my desktop which refreshed every minute with 3000 items, it would crash on the 497th day caused me great concern :-) Clearly I need to get out of the house more. Mark |
From: Mattia B. <mat...@li...> - 2007-03-31 10:25:49
|
On Thu, 22 Mar 2007 02:08:01 +0000 Mark Dootson <mar...@zn...> wrote: Hi, > Attached diff fixes documented problem with > Wx::Perl::ListCtrl->SetItemData being called LONG_MAX or more times. > > It also includes tests too. > > Fix involves reseting index values for all items every 2147483640th time > SetItemData is called. I applied a slightly modified form of your patch, the only substantial change is forcing integer arithmetic and using sprintf "%u", otherwise sometimes I obtained floating point values. > Clearly I need to get out of the house more. Me too... Anyway, 0.02 uploaded to CPAN. Thanks! Mattia |