449566 - cairo-qpainter-surface.cpp fails to compile with gcc 4.3.x

Status: RESOLVED FIXED

Description

[Bernhard Rosenkraenzer]

User-Agent:       Mozilla/5.0 (compatible; Konqueror/3.5; en_US) KHTML/3.5.9 (like Gecko)
Build Identifier: Last hg changeset: 16538:1f6895511812

cairo-qpainter-surface.cpp uses std::auto_ptr, but doesn't #include <memory>, causing the compiler to complain about std::auto_ptr not existing.

Reproducible: Always

Steps to Reproduce:
Install gcc 4.3.x or later, compile mozilla-qt
Actual Results:  
Build barfs

Expected Results:  
It compiles

Comment 1

[Bernhard Rosenkraenzer]

Fix:

--- mozilla-qt/gfx/cairo/cairo/src/cairo-qpainter-surface.cpp.ark       2008-08-
07 12:03:00.029048960 +0200
+++ mozilla-qt/gfx/cairo/cairo/src/cairo-qpainter-surface.cpp   2008-08-07 12:03
:17.267054280 +0200
@@ -41,6 +41,8 @@

 #include "cairo-qpainter.h"

+#include <memory>
+
 #include <QtGui/QPainter>
 #include <QtGui/QPaintEngine>
 #include <QtGui/QPaintDevice>

Comment 2

[Bernhard Rosenkraenzer]

After fixing this, the build bombs out a bit later:

nsWindow.h:354: error: multiple parameters named 'aEvent'

Comment 3

[Bernhard Rosenkraenzer]

Fix for that:

--- mozilla-qt/widget/src/qt/nsWindow.h.ark     2008-08-07 12:33:38.916163623 +0200
+++ mozilla-qt/widget/src/qt/nsWindow.h 2008-08-07 12:36:18.780038219 +0200
@@ -351,7 +351,7 @@ private:
     void              *SetupPluginPort(void);
     nsresult           SetWindowIconList(const nsCStringArray &aIconList);
     void               SetDefaultIcon(void);
-    void               InitButtonEvent(nsMouseEvent &aEvent, QMouseEvent *aEvent, int aClickCount = 1);
+    void               InitButtonEvent(nsMouseEvent &event, QMouseEvent *aEvent, int aClickCount = 1);
     PRBool             DispatchCommandEvent(nsIAtom* aCommand);
     QWidget           *createQWidget(QWidget *parent, nsWidgetInitData *aInitData);

Comment 4

[Oleg Romashin (:romaxa)]

Created attachment 333016 [details] [diff] [review]
Patch a=[e-mail]

Comment 5

[Oleg Romashin (:romaxa)]

Created attachment 333091 [details] [diff] [review]
Added PATH_MAX include

Comment 6

[Oleg Romashin (:romaxa)]

Fixed:
http://hg.mozilla.org/index.cgi/mozilla-central/rev/9ba4c8bfa399