[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1002151 - in /subversion/trunk/subversion: libsvn_auth_kwallet/kwallet.cpp svn/main.c svnsync/main.c

From: Stefan Sperling <stsp_at_elego.de>
Date: Wed, 29 Sep 2010 14:29:26 +0200

On Tue, Sep 28, 2010 at 11:21:26AM -0700, Blair Zajac wrote:
> On 9/28/2010 6:07 AM, stsp_at_apache.org wrote:
> >Author: stsp
> >Date: Tue Sep 28 13:07:23 2010
> >New Revision: 1002151
> >
> >URL: http://svn.apache.org/viewvc?rev=1002151&view=rev
> >Log:
> >Remove some goo introduced in r878078 and follow-ups, which was related to
> >the Linux-specific code which has been removed in r1002144.
>
> >- INITIALIZE_APPLICATION
> >+ QCoreApplication *app;
> >+ if (! qApp)
> >+ {
> >+ int argc = 1;
> >+ app = new QCoreApplication(argc, (char *[1]) {(char *) "svn"});
> >+ }
>
> Out of curiosity, what does this do?

No idea. I didn't understand this either.
I moved this code back out of the INITIALIZE_APPLICATION macro anyway.
The code was this way before the INITIALIZE_APPLICATION macro was
introduced.

> QCoreApplication *app isn't static, so does this do some setup logic
> that we don't need to keep track of?

That's what it looks like to me, too.
I suppose Arfrever will definitely know what this is doing.

Stefan
Received on 2010-09-29 14:30:17 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.