[patch] Appwizard VCS extract method

[Andreas Pakulat]

On 29.08.08 07:53:00, Kris Wong wrote:
> > >  QString AppWizardPlugin::createProject(const ApplicationInfo& info)
> > >  {
> > >      QFileInfo templateInfo(info.appTemplate);
> > >      if (!templateInfo.exists())
> > > -        return "";
> > > +        return QString();
> >
> > This isn't really needed. QString() == "" in Qt4, unless you use
> > QString::isNull() which is to be avoided by all means.
> 
> Is that true?  I would bet that
> 
> return "";
> 
> Initializes heap space, while
> 
> return QString();
> 
> Does not.

So you're saying we save a heap allocation? Well, I don't really care
which one is used, just changing from one to the other without reason is
not good.

Andreas

-- 
You have no real enemies.