Closed
Bug 552394
Opened 15 years ago
Closed 15 years ago
nsWindow.cpp doesn't compile when Q_WS_X11 is defined
Categories
(Core Graveyard :: Widget: Qt, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: MikeK, Assigned: MikeK)
Details
Attachments
(1 file, 1 obsolete file)
893 bytes,
patch
|
romaxa
:
review+
|
Details | Diff | Splinter Review |
/home/mike/MozillaCode/qt/100315/mozilla-central/widget/src/qt/nsWindow.cpp: In member function ‘virtual void* nsWindow::GetNativeData(PRUint32)’:
/home/mike/MozillaCode/qt/100315/mozilla-central/widget/src/qt/nsWindow.cpp:776: error: jump to case label
/home/mike/MozillaCode/qt/100315/mozilla-central/widget/src/qt/nsWindow.cpp:771: error: crosses initialization of ‘QWidget* widget’
/home/mike/MozillaCode/qt/100315/mozilla-central/widget/src/qt/nsWindow.cpp:782: error: jump to case label
/home/mike/MozillaCode/qt/100315/mozilla-central/widget/src/qt/nsWindow.cpp:771: error: crosses initialization of ‘QWidget* widget’
/home/mike/MozillaCode/qt/100315/mozilla-central/widget/src/qt/nsWindow.cpp:785: error: jump to case label
/home/mike/MozillaCode/qt/100315/mozilla-central/widget/src/qt/nsWindow.cpp:771: error: crosses initialization of ‘QWidget* widget’
make[7]: *** [nsWindow.o] Error 1
This is due to a missing set of {} to control the scope of the QWidget* widget local variable
Assignee | ||
Comment 1•15 years ago
|
||
Note that this might not be the only problem on trunk
Assignee: nobody → mkristoffersen
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•15 years ago
|
||
Attachment #432536 -
Attachment is obsolete: true
Attachment #432537 -
Flags: review?(romaxa)
Assignee | ||
Updated•15 years ago
|
Attachment #432536 -
Attachment is patch: false
Comment 3•15 years ago
|
||
Very good. You beat me to this one! :oD
Comment 4•15 years ago
|
||
Comment on attachment 432537 [details] [diff] [review]
Woups wrong file, this should be the right one
Oh, sorry about this bustage.
Attachment #432537 -
Flags: review?(romaxa) → review+
Assignee | ||
Updated•15 years ago
|
Keywords: checkin-needed
Comment 5•15 years ago
|
||
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•