[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: SimpleList questions from a beginner



Bjarne Steinsbø said:
> What I did was to add another button (called it "Many"), since I wanted
> to test the performance of this widget when handling large amounts of
> data.  Adding the button was just a question of adding it to the list of
> buttons to make.  In "btn_clicked" I then added a case to the switch for
> my new button "Many".  I first tried the code
>

i have now done the same.

> ...
> elsif( $op eq 'Many' )
> {
>     my @more;
>     for (my $i = 0; $i < 10000; $i++)
>     {
>        push @more, [
>           'many', $i, $i + $i/100, 1, 'many', undef, 'many', []
>        ];
>     }
>     push @{$slist->{data}}, @more;
> }
> ...

should be perfectly legal, and will be.

>
> This is probably closest to the way I want to add data to the model,
> i.e. the data already prepared in an array for adding, and then added to
> the model in one go.  At the time I hadn't looked at the implementation
> of the SimpleList, so I also thought that this would have a better
> performance than adding the data one row at a time.  I know see that it
> doesn't really matter that much.
>
>
> However, it didn't work as expected.  Only one row (the first one) was
> added to the model.  I had expected the call signature of the "push"
> function to be honored, but it's not.  From the "SimpleList.pm". the
> problem seems to be in the PUSH (and SHIFT) function in ...::TiedList.
> It's not expecting more than one parameter in addtion to the object
> itself.  This making any sense to you?
>

good spot, that was the problem. what's worse is that i ran into it the other
night playing around but got interuped before i dug into it and never went
back to play with what i was then.

> It is of course no big deal to change the code to add the data one row
> at the time.

shouldn't be nec, and in the future will not be.

changes/fixes will be commited to cvs pretty soon, i need to do a little
thinking, verification, and testing first. it will be included in any further
releases.

that was both muppet's and my first expirence with tie'ing so we didn't really
have a clue what was going on in the begining, which may be evident to people
who do.

-rm



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]