Build problem?

Tomaz Canabrava tcanabrava at kde.org
Tue Apr 9 06:39:53 PDT 2013


We are in the point where different people can work on different aspects.
I`m working right now at my real-job, so I cant touch subsurface till
night. :)




2013/4/9 Dirk Hohndel <dirk at hohndel.org>

> Michael Hasselmann <michael at taschenorakel.de> writes:
>
> > On Tue, 2013-04-09 at 15:35 +0400, Alberto Mardegan wrote:
> >> On 04/09/2013 12:48 AM, Thiago Macieira wrote:
> >> > You'll need to write Q_SLOTS instead of slots.
> >> >
> >> > I'd like to drop that macro. It makes the code uglier in my opinion.
> Alberto,
> >> > did you find any conflicts?
> >>
> >> Please don't drop it :-)
> >> I found a conflict with the "signals" name. Both in subversion and in
> >> another project of mine (libaccounts-qt, which wraps a glib library).
> >>
> >> GDBus's gdbusintrospection.h has a struct with a member named "signals".
> >>
> >> After finding that, I got used to the uppercase macros, and I actually
> >> like them better than the lowercase keywords. :-)
> >
> > Fully agree with Alberto here. Actually, I'd go even further: Q_EMIT,
> > Q_FOREACH … just to make it very clear that you are now using C++
> > language extensions provided by Qt. Remember that this code will have to
> > be read by non-Qt/non-C++ experts, too. I'd think they'd welcome the
> > extra clarity.
>
> I'm ok with that. It simply needs to go in a HACKING-Qt file or
> something so people who are used to "plain" Qt know what's happening.
>
> I'm not quite sure where we are with the Qt work - is everyone waiting
> for Tomaz to create more content, are we at the point where different
> people can work on different aspects of the code?
>
>
> /D
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20130409/006f9634/attachment-0001.html>


More information about the subsurface mailing list