Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Paul Ishenin
30.01.2011 5:22, Sven Barth wrote: Ok, I have now implemented a set of flags in TSymtable of which one is set when the (global-,static-)symtable contains a class helper (I haven't refactored b_needs_init_final yet). Here are the new results with the search for class helpers enabled (only two

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Sven Barth
On 29.01.2011 17:31, Sven Barth wrote: On 29.01.2011 16:08, Florian Klaempfl wrote: Am 29.01.2011 13:25, schrieb Sven Barth: Do you (especially @Devs) think that this is a sufficient approach or should this be done another way? (Note: I have not yet profiled compiling the compiler once with tha

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Jonas Maebe
On 29 Jan 2011, at 21:46, Sven Barth wrote: > I have normally no problems with restraining myself, but seeing what > Borland/CodeGear/Embarcadero allows in its language dialect... I would suggest to file bug reports with Embarcadero about this. Then you can also find out whether this is intent

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Sven Barth
On 29.01.2011 21:34, Florian Klämpfl wrote: Am 29.01.2011 17:58, schrieb Sven Barth: This is one of those moments where I need to restrain myself to not swear for the whole world to hear... Ok, you'll really swear as soon as people test their code with your implementation :) I have normally

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Sven Barth
On 29.01.2011 21:30, Florian Klämpfl wrote: Am 29.01.2011 19:04, schrieb Paul Ishenin: 30.01.2011 0:50, Sven Barth wrote: Should I add the flag set to TSymtable or tstoredsymtable? (only the latter can write/save itself to ppu) I would add it to TSymTable. I really wonder that symtables hav

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Florian Klämpfl
Am 29.01.2011 17:58, schrieb Sven Barth: > > This is one of those moments where I need to restrain myself to not > swear for the whole world to hear... Ok, you'll really swear as soon as people test their code with your implementation :) ___ fpc-devel m

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Florian Klämpfl
Am 29.01.2011 19:04, schrieb Paul Ishenin: > 30.01.2011 0:50, Sven Barth wrote: >> Should I add the flag set to TSymtable or tstoredsymtable? (only the >> latter can write/save itself to ppu) > > I would add it to TSymTable. I really wonder that symtables have no flag set yet. Please move b_needs

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Paul Ishenin
30.01.2011 0:50, Sven Barth wrote: Should I add the flag set to TSymtable or tstoredsymtable? (only the latter can write/save itself to ppu) I would add it to TSymTable. Best regards, Paul Ishenin ___ fpc-devel maillist - fpc-devel@lists.freepascal

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Sven Barth
On 29.01.2011 17:41, Paul Ishenin wrote: 29.01.2011 23:31, Sven Barth wrote: Done. Differences are dramatic (at least on my Linux PC) Try to add a set of flags to the symtable class. Store/Load it from/to ppu. When you add a def into check it if it is a class helper and add a flag if so. In yo

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Sven Barth
On 29.01.2011 17:35, Paul Ishenin wrote: 29.01.2011 19:25, Sven Barth wrote: Does someone know a good place to link this page from? Maybe something where other language/compiler features are referenced from or can be linked to in the future. http://wiki.freepascal.org/delphi_language_features_w

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Sven Barth
On 29.01.2011 18:08, Paul Ishenin wrote: 29.01.2011 23:58, Sven Barth wrote: This is one of those moments where I need to restrain myself to not swear for the whole world to hear... *sigh* I had the same feeling when I found that generics are stored as TFoo<,> in the delphy symtable and theref

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Paul Ishenin
29.01.2011 23:58, Sven Barth wrote: This is one of those moments where I need to restrain myself to not swear for the whole world to hear... *sigh* I had the same feeling when I found that generics are stored as TFoo<,> in the delphy symtable and therefore it is possible to have TFoo, TFoo, T

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Sven Barth
On 29.01.2011 17:45, Paul Ishenin wrote: 29.01.2011 23:41, Paul Ishenin wrote: I also noticed that you check only unit symtables. I don't know what if you add a class helper into a record/class/object type section? This code works in delphi: program Project42; {$APPTYPE CONSOLE} type TFoo =

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Sven Barth
On 29.01.2011 17:41, Paul Ishenin wrote: 29.01.2011 23:31, Sven Barth wrote: Done. Differences are dramatic (at least on my Linux PC) Try to add a set of flags to the symtable class. Store/Load it from/to ppu. When you add a def into check it if it is a class helper and add a flag if so. In yo

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Sven Barth
On 29.01.2011 17:35, Paul Ishenin wrote: 29.01.2011 19:25, Sven Barth wrote: Does someone know a good place to link this page from? Maybe something where other language/compiler features are referenced from or can be linked to in the future. http://wiki.freepascal.org/delphi_language_features_w

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Paul Ishenin
29.01.2011 23:41, Paul Ishenin wrote: I also noticed that you check only unit symtables. I don't know what if you add a class helper into a record/class/object type section? This code works in delphi: program Project42; {$APPTYPE CONSOLE} type TFoo = class class procedure DoSomething;

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Paul Ishenin
29.01.2011 23:31, Sven Barth wrote: Done. Differences are dramatic (at least on my Linux PC) Try to add a set of flags to the symtable class. Store/Load it from/to ppu. When you add a def into check it if it is a class helper and add a flag if so. In you search algorithm test the flag first b

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Paul Ishenin
29.01.2011 19:25, Sven Barth wrote: Does someone know a good place to link this page from? Maybe something where other language/compiler features are referenced from or can be linked to in the future. http://wiki.freepascal.org/delphi_language_features_which_fpc_does_not_have http://wiki.freepa

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Sven Barth
On 29.01.2011 16:08, Florian Klaempfl wrote: Am 29.01.2011 13:25, schrieb Sven Barth: Do you (especially @Devs) think that this is a sufficient approach or should this be done another way? (Note: I have not yet profiled compiling the compiler once with that search enabled and once without) Pro

Re: [fpc-devel] Status report for "class helpers"

2011-01-29 Thread Florian Klaempfl
Am 29.01.2011 13:25, schrieb Sven Barth: Do you (especially @Devs) think that this is a sufficient approach or should this be done another way? (Note: I have not yet profiled compiling the compiler once with that search enabled and once without) Profile compiler compilation first, then we can d

[fpc-devel] Status report for "class helpers"

2011-01-29 Thread Sven Barth
Hello together! I've decided to provide a little status report on the "class helper" feature that I'm currently implementing. I found the time to put up a first version of a wiki page: http://wiki.freepascal.org/Class_helpers Does someone know a good place to link this page from? Maybe some