Sign in Join

Live chat for Free Pascal, Lazarus, Delphi and every Pascal programmer.

Mailing lists / fpc-devel / 2005-05

Some thoughts about TFPList

5 messages · 4 people · –

Reply by email Subscribe to this list Your subscription

Read-only archive. Use j and k to move between messages.

pascalive

I noticed that TFPList class was added in cvs version as a way to get rid of Notify performance penalty. I suggest to remove TFPList.Grow method since its only for Delphi compability (but Delphi doesnt have TFPlist) and AFAIK virtual methods add some degree of performance hit.

Luiz

__________________________________________________________________________ Acabe com aquelas janelinhas que pulam na sua tela. AntiPop-up UOL - É grátis!
http://antipopup.uol.com.br/

Reply by email

Michael Van Canneyt
to pascalive
pascalive wrote
I noticed that TFPList class was added in cvs version as a way to get rid of Notify performance penalty. I suggest to remove TFPList.Grow method since its only for Delphi compability (but Delphi doesnt have TFPlist) and AFAIK virtual methods add some degree of performance hit.

I'd prefer to make it a normal static routine.

Michael

Reply by email

Luiz Américo
to Michael Van Canneyt
Michael Van Canneyt wrote 11 lines
On Mon, 2 May 2005, pascalive wrote: > I noticed that TFPList class was added in cvs version as a way to get > rid of Notify performance penalty. I suggest to remove TFPList.Grow > method since its only for Delphi compability (but Delphi doesnt have > TFPlist) and AFAIK virtual methods add some degree of performance hit. I'd prefer to make it a normal static routine.

For me is good.

Luiz

Reply by email

Dean Zobec
to Michael Van Canneyt: I'd prefer to make it a normal static routine. Michael
Michael Van Canneyt wrote 11 lines
On Mon, 2 May 2005, pascalive wrote: > I noticed that TFPList class was added in cvs version as a way to get > rid of Notify performance penalty. I suggest to remove TFPList.Grow > method since its only for Delphi compability (but Delphi doesnt have > TFPlist) and AFAIK virtual methods add some degree of performance hit. I'd prefer to make it a normal static routine.

But it does nothing and does not get called.
When introducing TFPList it slipped through when I copied the interface from TList.
So I think in TFPList could be safely removed.
Otherwise it could be misleading.

Dean

Reply by email

Michael Van Canneyt
to Dean Zobec
Dean Zobec wrote 20 lines
Michael Van Canneyt wrote: > > > On Mon, 2 May 2005, pascalive wrote: > > > I noticed that TFPList class was added in cvs version as a way to get > > rid of Notify performance penalty. I suggest to remove TFPList.Grow > > method since its only for Delphi compability (but Delphi doesnt have > > TFPlist) and AFAIK virtual methods add some degree of performance > > hit. > > > I'd prefer to make it a normal static routine. But it does nothing and does not get called. When introducing TFPList it slipped through when I copied the interface from TList. So I think in TFPList could be safely removed. Otherwise it could be misleading.

Hm. I would have thought it does the same as in TList. OK, I've checked the source, and removed it.

Michael.

Reply by email