[KPhotoAlbum] git master-branch doesn't compile
Jesper K. Pedersen
blackie at kde.org
Wed Jun 13 05:58:15 BST 2012
On Tuesday 12 June 2012 23:21:02 Johannes Zarl wrote:
> On Tuesday 12 June 2012 19:46:06 Jesper K. Pedersen wrote:
> > Thanks for the patches, see comments below.
> >
> > > 0001-Fix-compile-error.patch
> >
> > This was indeed a "bug" though harmless, did your compiler barf on that?
> > Do
> > you use any special flags? Which compiler is that?
>
> Yes, my gcc doesn't like the redefinition of "fileName". No special flags,
> just running cmake without special parameters and then calling make.
>
> # g++ --version |head -1
> g++ (Debian 4.7.0-12) 4.7.0
OK mine is 4.6.3
>
> > > 0002-Fix-compiler-warnings-wrong-cast-unused-params.patch
> >
> > I've applied the first part to this, leaving the warning in, as I really
> > do
> > not want to forget about this code. That these variables are not used
> > concerns me quite a bit.
>
> Yeah, me too. But I was not sure whether the function should do something
> with those parameters, or if the function interface should be simplified...
> > So I apply each of them, and I want to make a minor change (patch 1 had
> > some vi stuff added to the end, and some whitespace changes, patch to
> > removed a waning I wanted to keep in there).
It is a virtual method. The unused parameters are used in many other places.
> Well, I kind of intentionally smuggled this (and other) vi modelines into
> files I touched, so I don't mess up the indentation by accident. If you
> don't want them I'll try to avoid them in the future(*)...
Well that is OK, I just didn't want it in as part of a patch. BTW: You should
try Qt Creator for Qt development. Trust me, I'm a die hard emacs user (Who
even wrote a book on Emacs), but I prefer Qt Creator for coding these days.
Cheers
More information about the Kphotoalbum
mailing list