Last Comment Bug 706610 - remove trailing semicolon after namespace decl
: remove trailing semicolon after namespace decl
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Graphics (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla11
Assigned To: :aceman
:
: Milan Sreckovic [:milan]
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-11-30 12:07 PST by Stefan
Modified: 2012-02-01 14:00 PST (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
fix (427 bytes, patch)
2011-12-01 11:37 PST, :aceman
joe: review+
Details | Diff | Splinter Review

Description Stefan 2011-11-30 12:07:58 PST
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   };
Comment 1 Stefan 2011-11-30 12:10:42 PST
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
Comment 2 :aceman 2011-12-01 11:37:17 PST
Created attachment 578342 [details] [diff] [review]
fix
Comment 4 Ed Morley [:emorley] 2011-12-04 07:20:11 PST
https://hg.mozilla.org/mozilla-central/rev/109d10e40937

Note You need to log in before you can comment on or make changes to this bug.