remove trailing semicolon after namespace decl

RESOLVED FIXED in mozilla11

Status

()

Core
Graphics
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: Stefan, Assigned: aceman)

Tracking

Trunk
mozilla11
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

fix
427 bytes, patch
Joe Drew (not getting mail)
: review+
Details | Diff | Splinter Review
(Reporter)

Description

6 years ago
User Agent:  

Steps to reproduce:

1. Compile Source with gcc (SUSE Linux) 4.3.2 [gcc-4_3-branch revision 141291] (openSUSE Evergreen 11.1)


Actual results:

1. Compilation stops with error "gfx/gl/GLContext.h:81: error: extra ';'" while compiling gfx/gl/GLContextProviderGLX.cpp


Expected results:

The trailing semicolon at the end of line 81 shall be removed (though it has been legalized in C++11):

     77 namespace mozilla {
     78   namespace layers {
     79     class LayerManagerOGL;
     80     class ColorTextureLayerProgram;
     81   };
(Reporter)

Comment 1

6 years ago
Issue mentioned in

Bug 656824 - Use ARB_robustness and EGL_CONTEXT_LOST to detect driver resets in WebGL 

Bug 703516 - Move GLContext* code out of thebes
OS: Other → All
Hardware: Other → All
(Assignee)

Updated

6 years ago
Status: UNCONFIRMED → ASSIGNED
Component: General → Graphics
Ever confirmed: true
Product: Firefox → Core
QA Contact: general → thebes
(Assignee)

Comment 2

6 years ago
Created attachment 578342 [details] [diff] [review]
fix
Assignee: nobody → acelists
Attachment #578342 - Flags: review?(joe)
Attachment #578342 - Flags: review?(joe) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
http://hg.mozilla.org/integration/mozilla-inbound/rev/109d10e40937
Keywords: checkin-needed
Target Milestone: --- → mozilla11

Comment 4

6 years ago
https://hg.mozilla.org/mozilla-central/rev/109d10e40937
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.