[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#502559: Include from libqt3-headers Fails With -Werror.



Package: libqt3-headers
Version: 3:3.3.8b-5

Software that tries to build against Qt3's qimage.h with the -Werror
compiler flag will fail in Lenny. Apparently g++ has gotten more picky
about order of operations:

/usr/include/qt3/qimage.h: In member function 'bool
QImageTextKeyLang::operator<(const QImageTextKeyLang&) const':
/usr/include/qt3/qimage.h:61: error: suggest parentheses around &&
within ||

This can be worked around by using the -Wno-parentheses CPPFLAG, but it
would be far better if it were actually fixed in the header.

As such, I've written and tested the attached patch.

--Pat
Author: Pat Suwalski <pats@xandros.com>

This patch fixes the QImage header breaking with -Werror on Lenny:

/usr/include/qt3/qimage.h: In member function
  'bool QImageTextKeyLang::operator<(const QImageTextKeyLang&) const':
/usr/include/qt3/qimage.h:61: error: suggest parentheses around && within ||

--- a/src/kernel/qimage.h	2008-10-17 19:28:30.000000000 +0000
+++ b/src/kernel/qimage.h	2008-10-17 19:29:00.000000000 +0000
@@ -58,7 +58,7 @@
     QCString lang;
 
     bool operator< (const QImageTextKeyLang& other) const
-	{ return key < other.key || key==other.key && lang < other.lang; }
+	{ return key < other.key || (key==other.key && lang < other.lang); }
     bool operator== (const QImageTextKeyLang& other) const
 	{ return key==other.key && lang==other.lang; }
 };

Reply to: