Review Request: Patch to reduce C++ parser memory consumption by removing parent AST pointer from AST nodes

Bertjan Broeksema b.broeksema at home.nl
Mon Feb 15 15:45:06 UTC 2010


On Monday 15 February 2010 16:15:36 Alexander Dymo wrote:
> > On 2010-02-15 15:00:24, Bertjan Broeksema wrote:
> > > Alexander, do you mind to create a patch which brings back this
> > > functionality after removal? I'm currently writing my thesis and don't
> > > have enough time to adapt my code to have the parent nodes stored in a
> > > different way. Obviously I'd like to show something when I have to
> > > defend my thesis in ~1 month. Otherwise, please send me an e-mail with
> > > the revision containing the removal so that I can set that as the
> > > maximum supported rev for my plugin.
> > >
> > > Preferably the first. Millian will start his internship at KDAB soon
> > > and he'll be most likely continuing my work. So maybe we can do a
> > > release of the plugin afterwards. Currently its not in a state yet that
> > > I want to release it.
> > >
> > > Cheers,
> > >
> > > Bertjan
> 
> Yeah, I can send you a patch. Btw, is your plugin somewhere in svn? I could
>  probably do the porting for you.

No not yet, I worked in a local git repository. I have to figure out first how 
to get the complete history in svn. So a patch is okay for now. Eventually I 
can send you a tar of the source of my plugin (which would be the quickest and 
easiest solution currently for me).

Cheers,

Bertjan




More information about the KDevelop-devel mailing list