]> git.xonotic.org Git - xonotic/netradiant.git/blobdiff - radiant/brush.h
Merge commit '70f0925f0000a132c709b935aa5cb0942b5080d9' into master-merge
[xonotic/netradiant.git] / radiant / brush.h
index 7eeeb1df7aefd1b9af7c03f6fc9075659949f48e..f258d84afb6694632d0ad1a0ab1081b596c5810e 100644 (file)
@@ -306,8 +306,8 @@ inline void brush_check_shader( const char* name ){
 class FaceShaderObserver
 {
 public:
-       virtual void realiseShader() = 0;
-       virtual void unrealiseShader() = 0;
+virtual void realiseShader() = 0;
+virtual void unrealiseShader() = 0;
 };
 
 typedef ReferencePair<FaceShaderObserver> FaceShaderObserverPair;
@@ -316,20 +316,20 @@ typedef ReferencePair<FaceShaderObserver> FaceShaderObserverPair;
 class ContentsFlagsValue
 {
 public:
-       ContentsFlagsValue(){
-       }
+ContentsFlagsValue(){
+}
 
-       ContentsFlagsValue( int surfaceFlags, int contentFlags, int value, bool specified ) :
-               m_surfaceFlags( surfaceFlags ),
-               m_contentFlags( contentFlags ),
-               m_value( value ),
-               m_specified( specified ){
-       }
+ContentsFlagsValue( int surfaceFlags, int contentFlags, int value, bool specified ) :
+       m_surfaceFlags( surfaceFlags ),
+       m_contentFlags( contentFlags ),
+       m_value( value ),
+       m_specified( specified ){
+}
 
-       int m_surfaceFlags;
-       int m_contentFlags;
-       int m_value;
-       bool m_specified;
+int m_surfaceFlags;
+int m_contentFlags;
+int m_value;
+bool m_specified;
 };
 
 inline void ContentsFlagsValue_assignMasked( ContentsFlagsValue& flags, const ContentsFlagsValue& other ){
@@ -348,279 +348,287 @@ inline void ContentsFlagsValue_assignMasked( ContentsFlagsValue& flags, const Co
 class FaceShader : public ModuleObserver
 {
 public:
-       class SavedState
-       {
-       public:
-               CopiedString m_shader;
-               ContentsFlagsValue m_flags;
+class SavedState
+{
+public:
+CopiedString m_shader;
+ContentsFlagsValue m_flags;
 
-               SavedState( const FaceShader& faceShader ){
-                       m_shader = faceShader.getShader();
-                       m_flags = faceShader.m_flags;
-               }
+SavedState( const FaceShader& faceShader ){
+       m_shader = faceShader.getShader();
+       m_flags = faceShader.m_flags;
+}
 
-               void exportState( FaceShader& faceShader ) const {
-                       faceShader.setShader( m_shader.c_str() );
+void exportState( FaceShader& faceShader ) const {
+       faceShader.setShader( m_shader.c_str() );
                        faceShader.m_flags = m_flags;
-               }
-       };
+}
+};
 
-       CopiedString m_shader;
-       Shader* m_state;
-       ContentsFlagsValue m_flags;
-       FaceShaderObserverPair m_observers;
-       bool m_instanced;
-       bool m_realised;
+CopiedString m_shader;
+Shader* m_state;
+ContentsFlagsValue m_flags;
+FaceShaderObserverPair m_observers;
+bool m_instanced;
+bool m_realised;
 
-       FaceShader( const char* shader, const ContentsFlagsValue& flags = ContentsFlagsValue( 0, 0, 0, false ) ) :
-               m_shader( shader ),
-               m_state( 0 ),
-               m_flags( flags ),
-               m_instanced( false ),
-               m_realised( false ){
-               captureShader();
-       }
+FaceShader( const char* shader, const ContentsFlagsValue& flags = ContentsFlagsValue( 0, 0, 0, false ) ) :
+       m_shader( shader ),
+       m_state( 0 ),
+       m_flags( flags ),
+       m_instanced( false ),
+       m_realised( false ){
+       captureShader();
+}
 
-       ~FaceShader(){
-               releaseShader();
-       }
+~FaceShader(){
+       releaseShader();
+}
 
-       // copy-construction not supported
-       FaceShader( const FaceShader& other );
+// copy-construction not supported
+FaceShader( const FaceShader& other );
 
-       void instanceAttach(){
-               m_instanced = true;
-               m_state->incrementUsed();
-       }
+void instanceAttach(){
+       m_instanced = true;
+       m_state->incrementUsed();
+}
 
-       void instanceDetach(){
-               m_state->decrementUsed();
-               m_instanced = false;
-       }
+void instanceDetach(){
+       m_state->decrementUsed();
+       m_instanced = false;
+}
 
-       void captureShader(){
-               ASSERT_MESSAGE( m_state == 0, "shader cannot be captured" );
-               brush_check_shader( m_shader.c_str() );
-               m_state = GlobalShaderCache().capture( m_shader.c_str() );
-               m_state->attach( *this );
-       }
+void captureShader(){
+       ASSERT_MESSAGE( m_state == 0, "shader cannot be captured" );
+       brush_check_shader( m_shader.c_str() );
+       m_state = GlobalShaderCache().capture( m_shader.c_str() );
+       m_state->attach( *this );
+}
 
-       void releaseShader(){
-               ASSERT_MESSAGE( m_state != 0, "shader cannot be released" );
-               m_state->detach( *this );
-               GlobalShaderCache().release( m_shader.c_str() );
-               m_state = 0;
-       }
+void releaseShader(){
+       ASSERT_MESSAGE( m_state != 0, "shader cannot be released" );
+       m_state->detach( *this );
+       GlobalShaderCache().release( m_shader.c_str() );
+       m_state = 0;
+}
 
-       void realise(){
-               ASSERT_MESSAGE( !m_realised, "FaceTexdef::realise: already realised" );
-               m_realised = true;
-               m_observers.forEach([](FaceShaderObserver &observer) {
-                       observer.realiseShader();
-               });
-       }
+void realise(){
+       ASSERT_MESSAGE( !m_realised, "FaceTexdef::realise: already realised" );
+       m_realised = true;
+       m_observers.forEach([](FaceShaderObserver &observer) {
+               observer.realiseShader();
+       });
+}
 
-       void unrealise(){
-               ASSERT_MESSAGE( m_realised, "FaceTexdef::unrealise: already unrealised" );
-               m_observers.forEach([](FaceShaderObserver &observer) {
-                       observer.unrealiseShader();
-               });
-               m_realised = false;
-       }
+void unrealise(){
+       ASSERT_MESSAGE( m_realised, "FaceTexdef::unrealise: already unrealised" );
+       m_observers.forEach([](FaceShaderObserver &observer) {
+               observer.unrealiseShader();
+       });
+       m_realised = false;
+}
 
-       void attach( FaceShaderObserver& observer ){
-               m_observers.attach( observer );
-               if ( m_realised ) {
-                       observer.realiseShader();
-               }
+void attach( FaceShaderObserver& observer ){
+       m_observers.attach( observer );
+       if ( m_realised ) {
+               observer.realiseShader();
        }
+}
 
-       void detach( FaceShaderObserver& observer ){
-               if ( m_realised ) {
-                       observer.unrealiseShader();
-               }
-               m_observers.detach( observer );
+void detach( FaceShaderObserver& observer ){
+       if ( m_realised ) {
+               observer.unrealiseShader();
        }
+       m_observers.detach( observer );
+}
 
-       const char* getShader() const {
-               return m_shader.c_str();
+const char* getShader() const {
+       return m_shader.c_str();
+}
+void setShader( const char* name ){
+       if ( m_instanced ) {
+               m_state->decrementUsed();
        }
-       void setShader( const char* name ){
-               if ( m_instanced ) {
-                       m_state->decrementUsed();
-               }
-               releaseShader();
-               m_shader = name;
-               captureShader();
-               if ( m_instanced ) {
-                       m_state->incrementUsed();
-               }
+       releaseShader();
+       m_shader = name;
+       captureShader();
+       if ( m_instanced ) {
+               m_state->incrementUsed();
        }
+}
 
-       ContentsFlagsValue getFlags() const {
-               ASSERT_MESSAGE( m_realised, "FaceShader::getFlags: flags not valid when unrealised" );
-               if ( !m_flags.m_specified ) {
-                       return ContentsFlagsValue(
-                                          m_state->getTexture().surfaceFlags,
-                                          m_state->getTexture().contentFlags,
-                                          m_state->getTexture().value,
-                                          true
-                                          );
-               }
-               return m_flags;
+ContentsFlagsValue getFlags() const {
+       ASSERT_MESSAGE( m_realised, "FaceShader::getFlags: flags not valid when unrealised" );
+       if ( !m_flags.m_specified ) {
+               return ContentsFlagsValue(
+                                  m_state->getTexture().surfaceFlags,
+                                  m_state->getTexture().contentFlags,
+                                  m_state->getTexture().value,
+                                  true
+                                  );
        }
+       return m_flags;
+}
 
-       void setFlags( const ContentsFlagsValue& flags ){
-               ASSERT_MESSAGE( m_realised, "FaceShader::setFlags: flags not valid when unrealised" );
-               ContentsFlagsValue_assignMasked( m_flags, flags );
-       }
+void setFlags( const ContentsFlagsValue& flags ){
+       ASSERT_MESSAGE( m_realised, "FaceShader::setFlags: flags not valid when unrealised" );
+       ContentsFlagsValue_assignMasked( m_flags, flags );
+}
 
-       Shader* state() const {
-               return m_state;
-       }
+Shader* state() const {
+       return m_state;
+}
 
-       std::size_t width() const {
-               if ( m_realised ) {
-                       return m_state->getTexture().width;
-               }
-               return 1;
+std::size_t width() const {
+       if ( m_realised ) {
+               return m_state->getTexture().width;
        }
+       return 1;
+}
 
-       std::size_t height() const {
-               if ( m_realised ) {
-                       return m_state->getTexture().height;
-               }
-               return 1;
+std::size_t height() const {
+       if ( m_realised ) {
+               return m_state->getTexture().height;
        }
+       return 1;
+}
 
-       unsigned int shaderFlags() const {
-               if ( m_realised ) {
-                       return m_state->getFlags();
-               }
-               return 0;
+unsigned int shaderFlags() const {
+       if ( m_realised ) {
+               return m_state->getFlags();
        }
+       return 0;
+}
 };
 
 
 class FaceTexdef : public FaceShaderObserver
 {
-       // not copyable
-       FaceTexdef( const FaceTexdef& other );
+// not copyable
+FaceTexdef( const FaceTexdef& other );
 
-       // not assignable
-       FaceTexdef& operator=( const FaceTexdef& other );
+// not assignable
+FaceTexdef& operator=( const FaceTexdef& other );
 
 public:
-       class SavedState
-       {
-       public:
-               TextureProjection m_projection;
-
-               SavedState( const FaceTexdef& faceTexdef ){
-                       m_projection = faceTexdef.m_projection;
-               }
-
-               void exportState( FaceTexdef& faceTexdef ) const {
-                       Texdef_Assign( faceTexdef.m_projection, m_projection );
-               }
-       };
+class SavedState
+{
+public:
+TextureProjection m_projection;
 
-       FaceShader& m_shader;
-       TextureProjection m_projection;
-       bool m_projectionInitialised;
-       bool m_scaleApplied;
+SavedState( const FaceTexdef& faceTexdef ){
+       m_projection = faceTexdef.m_projection;
+}
 
-       FaceTexdef(
-               FaceShader& shader,
-               const TextureProjection& projection,
-               bool projectionInitialised = true
-               ) :
-               m_shader( shader ),
-               m_projection( projection ),
-               m_projectionInitialised( projectionInitialised ),
-               m_scaleApplied( false ){
-               m_shader.attach( *this );
-       }
+void exportState( FaceTexdef& faceTexdef ) const {
+       Texdef_Assign( faceTexdef.m_projection, m_projection );
+}
+};
 
-       ~FaceTexdef(){
-               m_shader.detach( *this );
-       }
+FaceShader& m_shader;
+TextureProjection m_projection;
+bool m_projectionInitialised;
+bool m_scaleApplied;
 
-       void addScale(){
-               ASSERT_MESSAGE( !m_scaleApplied, "texture scale aready added" );
-               m_scaleApplied = true;
-               m_projection.m_brushprimit_texdef.addScale( m_shader.width(), m_shader.height() );
-       }
+FaceTexdef(
+       FaceShader& shader,
+       const TextureProjection& projection,
+       bool projectionInitialised = true
+       ) :
+       m_shader( shader ),
+       m_projection( projection ),
+       m_projectionInitialised( projectionInitialised ),
+       m_scaleApplied( false ){
+       m_shader.attach( *this );
+}
 
-       void removeScale(){
-               ASSERT_MESSAGE( m_scaleApplied, "texture scale aready removed" );
-               m_scaleApplied = false;
-               m_projection.m_brushprimit_texdef.removeScale( m_shader.width(), m_shader.height() );
-       }
+~FaceTexdef(){
+       m_shader.detach( *this );
+}
 
-       void realiseShader(){
-               if ( m_projectionInitialised && !m_scaleApplied ) {
-                       addScale();
-               }
-       }
+void addScale(){
+       ASSERT_MESSAGE( !m_scaleApplied, "texture scale aready added" );
+       m_scaleApplied = true;
+       m_projection.m_brushprimit_texdef.addScale( m_shader.width(), m_shader.height() );
+}
 
-       void unrealiseShader(){
-               if ( m_projectionInitialised && m_scaleApplied ) {
-                       removeScale();
-               }
-       }
+void removeScale(){
+       ASSERT_MESSAGE( m_scaleApplied, "texture scale aready removed" );
+       m_scaleApplied = false;
+       m_projection.m_brushprimit_texdef.removeScale( m_shader.width(), m_shader.height() );
+}
 
-       void setTexdef( const TextureProjection& projection ){
-               removeScale();
-               Texdef_Assign( m_projection, projection );
+void realiseShader(){
+       if ( m_projectionInitialised && !m_scaleApplied ) {
                addScale();
        }
+}
 
-       void shift( float s, float t ){
-               ASSERT_MESSAGE( texdef_sane( m_projection.m_texdef ), "FaceTexdef::shift: bad texdef" );
+void unrealiseShader(){
+       if ( m_projectionInitialised && m_scaleApplied ) {
                removeScale();
-               Texdef_Shift( m_projection, s, t );
-               addScale();
        }
+}
 
-       void scale( float s, float t ){
-               removeScale();
-               Texdef_Scale( m_projection, s, t );
-               addScale();
-       }
+void setTexdef( const TextureProjection& projection ){
+       removeScale();
+       Texdef_Assign( m_projection, projection );
+       addScale();
+}
 
-       void rotate( float angle ){
-               removeScale();
-               Texdef_Rotate( m_projection, angle );
-               addScale();
-       }
+void shift( float s, float t ){
+       ASSERT_MESSAGE( texdef_sane( m_projection.m_texdef ), "FaceTexdef::shift: bad texdef" );
+       removeScale();
+       Texdef_Shift( m_projection, s, t );
+       addScale();
+}
 
-       void fit( const Vector3& normal, const Winding& winding, float s_repeat, float t_repeat ){
-               Texdef_FitTexture( m_projection, m_shader.width(), m_shader.height(), normal, winding, s_repeat, t_repeat );
-       }
+void scale( float s, float t ){
+       removeScale();
+       Texdef_Scale( m_projection, s, t );
+       addScale();
+}
 
-       void emitTextureCoordinates( Winding& winding, const Vector3& normal, const Matrix4& localToWorld ){
-               Texdef_EmitTextureCoordinates( m_projection, m_shader.width(), m_shader.height(), winding, normal, localToWorld );
-       }
+void rotate( float angle ){
+       removeScale();
+       Texdef_Rotate( m_projection, angle );
+       addScale();
+}
 
-       void transform( const Plane3& plane, const Matrix4& matrix ){
-               removeScale();
-               Texdef_transformLocked( m_projection, m_shader.width(), m_shader.height(), plane, matrix );
-               addScale();
-       }
+void fit( const Vector3& normal, const Winding& winding, float s_repeat, float t_repeat ){
+       Texdef_FitTexture( m_projection, m_shader.width(), m_shader.height(), normal, winding, s_repeat, t_repeat );
+}
 
-       TextureProjection normalised() const {
-               brushprimit_texdef_t tmp( m_projection.m_brushprimit_texdef );
-               tmp.removeScale( m_shader.width(), m_shader.height() );
-               return TextureProjection( m_projection.m_texdef, tmp, m_projection.m_basis_s, m_projection.m_basis_t );
-       }
+void fitW( const Vector3& normal, const Winding& winding, float s_repeat, float t_repeat ){
+       Texdef_FitTextureW( m_projection, m_shader.width(), m_shader.height(), normal, winding, s_repeat, t_repeat );
+}
 
-       void setBasis( const Vector3& normal ){
-               Matrix4 basis;
-               Normal_GetTransform( normal, basis );
-               m_projection.m_basis_s = Vector3( basis.xx(), basis.yx(), basis.zx() );
-               m_projection.m_basis_t = Vector3( -basis.xy(), -basis.yy(), -basis.zy() );
-       }
+void fitH( const Vector3& normal, const Winding& winding, float s_repeat, float t_repeat ){
+       Texdef_FitTextureH( m_projection, m_shader.width(), m_shader.height(), normal, winding, s_repeat, t_repeat );
+}
+
+void emitTextureCoordinates( Winding& winding, const Vector3& normal, const Matrix4& localToWorld ){
+       Texdef_EmitTextureCoordinates( m_projection, m_shader.width(), m_shader.height(), winding, normal, localToWorld );
+}
+
+void transform( const Plane3& plane, const Matrix4& matrix ){
+       removeScale();
+       Texdef_transformLocked( m_projection, m_shader.width(), m_shader.height(), plane, matrix );
+       addScale();
+}
+
+TextureProjection normalised() const {
+       brushprimit_texdef_t tmp( m_projection.m_brushprimit_texdef );
+       tmp.removeScale( m_shader.width(), m_shader.height() );
+       return TextureProjection( m_projection.m_texdef, tmp, m_projection.m_basis_s, m_projection.m_basis_t );
+}
+
+void setBasis( const Vector3& normal ){
+       Matrix4 basis;
+       Normal_GetTransform( normal, basis );
+       m_projection.m_basis_s = Vector3( basis.xx(), basis.yx(), basis.zx() );
+       m_projection.m_basis_t = Vector3( -basis.xy(), -basis.yy(), -basis.zy() );
+}
 };
 
 inline void planepts_print( const PlanePoints& planePoints, TextOutputStream& ostream ){
@@ -642,192 +650,192 @@ inline Plane3 Plane3_applyTransform( const Plane3& plane, const Matrix4& matrix
 
 class FacePlane
 {
-       PlanePoints m_planepts;
-       Plane3 m_planeCached;
-       Plane3 m_plane;
+PlanePoints m_planepts;
+Plane3 m_planeCached;
+Plane3 m_plane;
 public:
-       Vector3 m_funcStaticOrigin;
+Vector3 m_funcStaticOrigin;
 
-       static EBrushType m_type;
+static EBrushType m_type;
 
-       static bool isDoom3Plane(){
-               return FacePlane::m_type == eBrushTypeDoom3 || FacePlane::m_type == eBrushTypeQuake4;
-       }
+static bool isDoom3Plane(){
+       return FacePlane::m_type == eBrushTypeDoom3 || FacePlane::m_type == eBrushTypeQuake4;
+}
 
        FacePlane& operator=(const FacePlane&) = default;
 
-       class SavedState
+class SavedState
+{
+public:
+PlanePoints m_planepts;
+Plane3 m_plane;
+
+SavedState( const FacePlane& facePlane ){
+       if ( facePlane.isDoom3Plane() ) {
+               m_plane = facePlane.m_plane;
+       }
+       else
        {
-       public:
-               PlanePoints m_planepts;
-               Plane3 m_plane;
+               planepts_assign( m_planepts, facePlane.planePoints() );
+       }
+}
 
-               SavedState( const FacePlane& facePlane ){
-                       if ( facePlane.isDoom3Plane() ) {
-                               m_plane = facePlane.m_plane;
-                       }
-                       else
-                       {
-                               planepts_assign( m_planepts, facePlane.planePoints() );
-                       }
-               }
+void exportState( FacePlane& facePlane ) const {
+       if ( facePlane.isDoom3Plane() ) {
+               facePlane.m_plane = m_plane;
+               facePlane.updateTranslated();
+       }
+       else
+       {
+               planepts_assign( facePlane.planePoints(), m_planepts );
+               facePlane.MakePlane();
+       }
+}
+};
 
-               void exportState( FacePlane& facePlane ) const {
-                       if ( facePlane.isDoom3Plane() ) {
-                               facePlane.m_plane = m_plane;
-                               facePlane.updateTranslated();
-                       }
-                       else
-                       {
-                               planepts_assign( facePlane.planePoints(), m_planepts );
-                               facePlane.MakePlane();
-                       }
-               }
-       };
+FacePlane() : m_funcStaticOrigin( 0, 0, 0 ){
+}
 
-       FacePlane() : m_funcStaticOrigin( 0, 0, 0 ){
+FacePlane( const FacePlane& other ) : m_funcStaticOrigin( 0, 0, 0 ){
+       if ( !isDoom3Plane() ) {
+               planepts_assign( m_planepts, other.m_planepts );
+               MakePlane();
        }
-
-       FacePlane( const FacePlane& other ) : m_funcStaticOrigin( 0, 0, 0 ){
-               if ( !isDoom3Plane() ) {
-                       planepts_assign( m_planepts, other.m_planepts );
-                       MakePlane();
-               }
-               else
-               {
-                       m_plane = other.m_plane;
-                       updateTranslated();
-               }
+       else
+       {
+               m_plane = other.m_plane;
+               updateTranslated();
        }
+}
 
-       void MakePlane(){
-               if ( !isDoom3Plane() ) {
+void MakePlane(){
+       if ( !isDoom3Plane() ) {
 #if 0
-                       if ( check_plane_is_integer( m_planepts ) ) {
-                               globalErrorStream() << "non-integer planepts: ";
-                               planepts_print( m_planepts, globalErrorStream() );
-                               globalErrorStream() << "\n";
-                       }
-#endif
-                       m_planeCached = plane3_for_points( m_planepts );
+               if ( check_plane_is_integer( m_planepts ) ) {
+                       globalErrorStream() << "non-integer planepts: ";
+                       planepts_print( m_planepts, globalErrorStream() );
+                       globalErrorStream() << "\n";
                }
+#endif
+               m_planeCached = plane3_for_points( m_planepts );
        }
+}
 
-       void reverse(){
-               if ( !isDoom3Plane() ) {
-                       vector3_swap( m_planepts[0], m_planepts[2] );
-                       MakePlane();
-               }
-               else
-               {
-                       m_planeCached = plane3_flipped( m_plane );
-                       updateSource();
-               }
+void reverse(){
+       if ( !isDoom3Plane() ) {
+               vector3_swap( m_planepts[0], m_planepts[2] );
+               MakePlane();
        }
+       else
+       {
+               m_planeCached = plane3_flipped( m_plane );
+               updateSource();
+       }
+}
 
-       void transform( const Matrix4& matrix, bool mirror ){
-               if ( !isDoom3Plane() ) {
+void transform( const Matrix4& matrix, bool mirror ){
+       if ( !isDoom3Plane() ) {
 
 #if 0
-                       bool off = check_plane_is_integer( planePoints() );
+               bool off = check_plane_is_integer( planePoints() );
 #endif
 
-                       matrix4_transform_point( matrix, m_planepts[0] );
-                       matrix4_transform_point( matrix, m_planepts[1] );
-                       matrix4_transform_point( matrix, m_planepts[2] );
+               matrix4_transform_point( matrix, m_planepts[0] );
+               matrix4_transform_point( matrix, m_planepts[1] );
+               matrix4_transform_point( matrix, m_planepts[2] );
 
-                       if ( mirror ) {
-                               reverse();
-                       }
+               if ( mirror ) {
+                       reverse();
+               }
 
 #if 0
-                       if ( check_plane_is_integer( planePoints() ) ) {
-                               if ( !off ) {
-                                       globalErrorStream() << "caused by transform\n";
-                               }
+               if ( check_plane_is_integer( planePoints() ) ) {
+                       if ( !off ) {
+                               globalErrorStream() << "caused by transform\n";
                        }
-#endif
-                       MakePlane();
-               }
-               else
-               {
-                       m_planeCached = Plane3_applyTransform( m_planeCached, matrix );
-                       updateSource();
                }
+#endif
+               MakePlane();
+       }
+       else
+       {
+               m_planeCached = Plane3_applyTransform( m_planeCached, matrix );
+               updateSource();
        }
+}
 
-       void offset( float offset ){
-               if ( !isDoom3Plane() ) {
-                       Vector3 move( vector3_scaled( m_planeCached.normal(), -offset ) );
+void offset( float offset ){
+       if ( !isDoom3Plane() ) {
+               Vector3 move( vector3_scaled( m_planeCached.normal(), -offset ) );
 
-                       vector3_subtract( m_planepts[0], move );
-                       vector3_subtract( m_planepts[1], move );
-                       vector3_subtract( m_planepts[2], move );
+               vector3_subtract( m_planepts[0], move );
+               vector3_subtract( m_planepts[1], move );
+               vector3_subtract( m_planepts[2], move );
 
-                       MakePlane();
-               }
-               else
-               {
-                       m_planeCached.d += offset;
-                       updateSource();
-               }
+               MakePlane();
        }
-
-       void updateTranslated(){
-               m_planeCached = Plane3_applyTranslation( m_plane, m_funcStaticOrigin );
+       else
+       {
+               m_planeCached.d += offset;
+               updateSource();
        }
+}
 
-       void updateSource(){
-               m_plane = Plane3_applyTranslation( m_planeCached, vector3_negated( m_funcStaticOrigin ) );
-       }
+void updateTranslated(){
+       m_planeCached = Plane3_applyTranslation( m_plane, m_funcStaticOrigin );
+}
 
+void updateSource(){
+       m_plane = Plane3_applyTranslation( m_planeCached, vector3_negated( m_funcStaticOrigin ) );
+}
 
-       PlanePoints& planePoints(){
-               return m_planepts;
-       }
 
-       const PlanePoints& planePoints() const {
-               return m_planepts;
-       }
+PlanePoints& planePoints(){
+       return m_planepts;
+}
 
-       const Plane3& plane3() const {
-               return m_planeCached;
-       }
+const PlanePoints& planePoints() const {
+       return m_planepts;
+}
 
-       void setDoom3Plane( const Plane3& plane ){
-               m_plane = plane;
-               updateTranslated();
-       }
+const Plane3& plane3() const {
+       return m_planeCached;
+}
 
-       const Plane3& getDoom3Plane() const {
-               return m_plane;
-       }
+void setDoom3Plane( const Plane3& plane ){
+       m_plane = plane;
+       updateTranslated();
+}
 
-       void copy( const FacePlane& other ){
-               if ( !isDoom3Plane() ) {
-                       planepts_assign( m_planepts, other.m_planepts );
-                       MakePlane();
-               }
-               else
-               {
-                       m_planeCached = other.m_plane;
-                       updateSource();
-               }
+const Plane3& getDoom3Plane() const {
+       return m_plane;
+}
+
+void copy( const FacePlane& other ){
+       if ( !isDoom3Plane() ) {
+               planepts_assign( m_planepts, other.m_planepts );
+               MakePlane();
+       }
+       else
+       {
+               m_planeCached = other.m_plane;
+               updateSource();
        }
+}
 
-       void copy( const Vector3& p0, const Vector3& p1, const Vector3& p2 ){
-               if ( !isDoom3Plane() ) {
-                       m_planepts[0] = p0;
-                       m_planepts[1] = p1;
-                       m_planepts[2] = p2;
-                       MakePlane();
-               }
-               else
-               {
-                       m_planeCached = plane3_for_points( p2, p1, p0 );
-                       updateSource();
-               }
+void copy( const Vector3& p0, const Vector3& p1, const Vector3& p2 ){
+       if ( !isDoom3Plane() ) {
+               m_planepts[0] = p0;
+               m_planepts[1] = p1;
+               m_planepts[2] = p2;
+               MakePlane();
+       }
+       else
+       {
+               m_planeCached = plane3_for_points( p2, p1, p0 );
+               updateSource();
        }
+}
 };
 
 inline void Winding_testSelect( Winding& winding, SelectionTest& test, SelectionIntersection& best ){
@@ -851,7 +859,7 @@ class Face;
 class FaceFilter
 {
 public:
-       virtual bool filter( const Face& face ) const = 0;
+virtual bool filter( const Face& face ) const = 0;
 };
 
 bool face_filtered( Face& face );
@@ -868,10 +876,10 @@ extern bool g_brush_texturelock_enabled;
 class FaceObserver
 {
 public:
-       virtual void planeChanged() = 0;
-       virtual void connectivityChanged() = 0;
-       virtual void shaderChanged() = 0;
-       virtual void evaluateTransform() = 0;
+virtual void planeChanged() = 0;
+virtual void connectivityChanged() = 0;
+virtual void shaderChanged() = 0;
+virtual void evaluateTransform() = 0;
 };
 
 class Face :
@@ -1227,6 +1235,18 @@ void FitTexture( float s_repeat, float t_repeat ){
        texdefChanged();
 }
 
+void FitTextureW( float s_repeat, float t_repeat ){
+       undoSave();
+       m_texdef.fitW( m_plane.plane3().normal(), m_winding, s_repeat, t_repeat );
+       texdefChanged();
+}
+
+void FitTextureH( float s_repeat, float t_repeat ){
+       undoSave();
+       m_texdef.fitH( m_plane.plane3().normal(), m_winding, s_repeat, t_repeat );
+       texdefChanged();
+}
+
 void EmitTextureCoordinates(){
        Texdef_EmitTextureCoordinates( m_texdefTransformed, m_shader.width(), m_shader.height(), m_winding, plane3().normal(), g_matrix4_identity );
 }
@@ -1310,21 +1330,21 @@ bool is_bounded() const {
 
 class FaceVertexId
 {
-       std::size_t m_face;
-       std::size_t m_vertex;
+std::size_t m_face;
+std::size_t m_vertex;
 
 public:
-       FaceVertexId( std::size_t face, std::size_t vertex )
-               : m_face( face ), m_vertex( vertex ){
-       }
+FaceVertexId( std::size_t face, std::size_t vertex )
+       : m_face( face ), m_vertex( vertex ){
+}
 
-       std::size_t getFace() const {
-               return m_face;
-       }
+std::size_t getFace() const {
+       return m_face;
+}
 
-       std::size_t getVertex() const {
-               return m_vertex;
-       }
+std::size_t getVertex() const {
+       return m_vertex;
+}
 };
 
 typedef std::size_t faceIndex_t;
@@ -1388,7 +1408,7 @@ typedef std::vector<Brush*> brush_vector_t;
 class BrushFilter
 {
 public:
-       virtual bool filter( const Brush& brush ) const = 0;
+virtual bool filter( const Brush& brush ) const = 0;
 };
 
 bool brush_filtered( Brush& brush );
@@ -1431,81 +1451,81 @@ inline FaceVertexId next_vertex( const Faces& faces, FaceVertexId faceVertex ){
 
 class SelectableEdge
 {
-       Vector3 getEdge() const {
-               const Winding& winding = getFace().getWinding();
-               return vector3_mid( winding[m_faceVertex.getVertex()].vertex, winding[Winding_next( winding, m_faceVertex.getVertex() )].vertex );
-       }
+Vector3 getEdge() const {
+       const Winding& winding = getFace().getWinding();
+       return vector3_mid( winding[m_faceVertex.getVertex()].vertex, winding[Winding_next( winding, m_faceVertex.getVertex() )].vertex );
+}
 
 public:
-       Faces& m_faces;
-       FaceVertexId m_faceVertex;
+Faces& m_faces;
+FaceVertexId m_faceVertex;
 
-       SelectableEdge( Faces& faces, FaceVertexId faceVertex )
-               : m_faces( faces ), m_faceVertex( faceVertex ){
-       }
+SelectableEdge( Faces& faces, FaceVertexId faceVertex )
+       : m_faces( faces ), m_faceVertex( faceVertex ){
+}
 
-       SelectableEdge& operator=( const SelectableEdge& other ){
-               m_faceVertex = other.m_faceVertex;
-               return *this;
-       }
+SelectableEdge& operator=( const SelectableEdge& other ){
+       m_faceVertex = other.m_faceVertex;
+       return *this;
+}
 
-       Face& getFace() const {
-               return *m_faces[m_faceVertex.getFace()];
-       }
+Face& getFace() const {
+       return *m_faces[m_faceVertex.getFace()];
+}
 
-       void testSelect( SelectionTest& test, SelectionIntersection& best ){
-               test.TestPoint( getEdge(), best );
-       }
+void testSelect( SelectionTest& test, SelectionIntersection& best ){
+       test.TestPoint( getEdge(), best );
+}
 };
 
 class SelectableVertex
 {
-       Vector3 getVertex() const {
-               return getFace().getWinding()[m_faceVertex.getVertex()].vertex;
-       }
+Vector3 getVertex() const {
+       return getFace().getWinding()[m_faceVertex.getVertex()].vertex;
+}
 
 public:
-       Faces& m_faces;
-       FaceVertexId m_faceVertex;
+Faces& m_faces;
+FaceVertexId m_faceVertex;
 
-       SelectableVertex( Faces& faces, FaceVertexId faceVertex )
-               : m_faces( faces ), m_faceVertex( faceVertex ){
-       }
+SelectableVertex( Faces& faces, FaceVertexId faceVertex )
+       : m_faces( faces ), m_faceVertex( faceVertex ){
+}
 
-       SelectableVertex& operator=( const SelectableVertex& other ){
-               m_faceVertex = other.m_faceVertex;
-               return *this;
-       }
+SelectableVertex& operator=( const SelectableVertex& other ){
+       m_faceVertex = other.m_faceVertex;
+       return *this;
+}
 
-       Face& getFace() const {
-               return *m_faces[m_faceVertex.getFace()];
-       }
+Face& getFace() const {
+       return *m_faces[m_faceVertex.getFace()];
+}
 
-       void testSelect( SelectionTest& test, SelectionIntersection& best ){
-               test.TestPoint( getVertex(), best );
-       }
+void testSelect( SelectionTest& test, SelectionIntersection& best ){
+       test.TestPoint( getVertex(), best );
+}
 };
 
 class BrushObserver
 {
 public:
-       virtual void reserve( std::size_t size ) = 0;
-       virtual void clear() = 0;
-       virtual void push_back( Face& face ) = 0;
-       virtual void pop_back() = 0;
-       virtual void erase( std::size_t index ) = 0;
-       virtual void connectivityChanged() = 0;
-       virtual void edge_clear() = 0;
-       virtual void edge_push_back( SelectableEdge& edge ) = 0;
-       virtual void vertex_clear() = 0;
-       virtual void vertex_push_back( SelectableVertex& vertex ) = 0;
-       virtual void DEBUG_verify() const = 0;
+virtual void reserve( std::size_t size ) = 0;
+virtual void clear() = 0;
+virtual void push_back( Face& face ) = 0;
+virtual void pop_back() = 0;
+virtual void erase( std::size_t index ) = 0;
+virtual void connectivityChanged() = 0;
+virtual void edge_clear() = 0;
+virtual void edge_push_back( SelectableEdge& edge ) = 0;
+virtual void vertex_clear() = 0;
+virtual void vertex_push_back( SelectableVertex& vertex ) = 0;
+virtual void DEBUG_verify() const = 0;
 };
 
 class BrushVisitor
 {
 public:
-       virtual void visit( Face& face ) const = 0;
+virtual void visit( Face& face ) const = 0;
 };
 
 class Brush :
@@ -1520,878 +1540,878 @@ class Brush :
        public BrushDoom3
 {
 private:
-       scene::Node* m_node;
-       typedef UniqueSet<BrushObserver*> Observers;
-       Observers m_observers;
-       UndoObserver* m_undoable_observer;
-       MapFile* m_map;
+scene::Node* m_node;
+typedef UniqueSet<BrushObserver*> Observers;
+Observers m_observers;
+UndoObserver* m_undoable_observer;
+MapFile* m_map;
 
-       // state
-       Faces m_faces;
-       // ----
+// state
+Faces m_faces;
+// ----
 
-       // cached data compiled from state
-       Array<PointVertex> m_faceCentroidPoints;
-       RenderablePointArray m_render_faces;
+// cached data compiled from state
+Array<PointVertex> m_faceCentroidPoints;
+RenderablePointArray m_render_faces;
 
-       Array<PointVertex> m_uniqueVertexPoints;
-       typedef std::vector<SelectableVertex> SelectableVertices;
-       SelectableVertices m_select_vertices;
-       RenderablePointArray m_render_vertices;
+Array<PointVertex> m_uniqueVertexPoints;
+typedef std::vector<SelectableVertex> SelectableVertices;
+SelectableVertices m_select_vertices;
+RenderablePointArray m_render_vertices;
 
-       Array<PointVertex> m_uniqueEdgePoints;
-       typedef std::vector<SelectableEdge> SelectableEdges;
-       SelectableEdges m_select_edges;
-       RenderablePointArray m_render_edges;
+Array<PointVertex> m_uniqueEdgePoints;
+typedef std::vector<SelectableEdge> SelectableEdges;
+SelectableEdges m_select_edges;
+RenderablePointArray m_render_edges;
 
-       Array<EdgeRenderIndices> m_edge_indices;
-       Array<EdgeFaces> m_edge_faces;
+Array<EdgeRenderIndices> m_edge_indices;
+Array<EdgeFaces> m_edge_faces;
 
-       AABB m_aabb_local;
-       // ----
+AABB m_aabb_local;
+// ----
 
-       Callback<void()> m_evaluateTransform;
-       Callback<void()> m_boundsChanged;
+Callback<void()> m_evaluateTransform;
+Callback<void()> m_boundsChanged;
 
-       mutable bool m_planeChanged;   // b-rep evaluation required
-       mutable bool m_transformChanged;   // transform evaluation required
-       // ----
+mutable bool m_planeChanged;   // b-rep evaluation required
+mutable bool m_transformChanged;   // transform evaluation required
+// ----
 
 public:
-       STRING_CONSTANT( Name, "Brush" );
-
-       Callback<void()> m_lightsChanged;
-
-       // static data
-       static Shader* m_state_point;
-       // ----
-
-       static EBrushType m_type;
-       static double m_maxWorldCoord;
-
-       Brush( scene::Node& node, const Callback<void()>& evaluateTransform, const Callback<void()>& boundsChanged ) :
-               m_node( &node ),
-               m_undoable_observer( 0 ),
-               m_map( 0 ),
-               m_render_faces( m_faceCentroidPoints, GL_POINTS ),
-               m_render_vertices( m_uniqueVertexPoints, GL_POINTS ),
-               m_render_edges( m_uniqueEdgePoints, GL_POINTS ),
-               m_evaluateTransform( evaluateTransform ),
-               m_boundsChanged( boundsChanged ),
-               m_planeChanged( false ),
-               m_transformChanged( false ){
-               planeChanged();
-       }
-       Brush( const Brush& other, scene::Node& node, const Callback<void()>& evaluateTransform, const Callback<void()>& boundsChanged ) :
-               m_node( &node ),
-               m_undoable_observer( 0 ),
-               m_map( 0 ),
-               m_render_faces( m_faceCentroidPoints, GL_POINTS ),
-               m_render_vertices( m_uniqueVertexPoints, GL_POINTS ),
-               m_render_edges( m_uniqueEdgePoints, GL_POINTS ),
-               m_evaluateTransform( evaluateTransform ),
-               m_boundsChanged( boundsChanged ),
-               m_planeChanged( false ),
-               m_transformChanged( false ){
-               copy( other );
-       }
-
-       Brush( const Brush& other ) :
-               TransformNode( other ),
-               Bounded( other ),
-               Cullable( other ),
-               Snappable(),
-               Undoable( other ),
-               FaceObserver( other ),
-               Filterable( other ),
-               Nameable( other ),
-               BrushDoom3( other ),
-               m_node( 0 ),
-               m_undoable_observer( 0 ),
-               m_map( 0 ),
-               m_render_faces( m_faceCentroidPoints, GL_POINTS ),
-               m_render_vertices( m_uniqueVertexPoints, GL_POINTS ),
-               m_render_edges( m_uniqueEdgePoints, GL_POINTS ),
-               m_planeChanged( false ),
-               m_transformChanged( false ){
-               copy( other );
-       }
-
-       ~Brush(){
-               ASSERT_MESSAGE( m_observers.empty(), "Brush::~Brush: observers still attached" );
-       }
-
-       // assignment not supported
-       Brush& operator=( const Brush& other );
-
-       void setDoom3GroupOrigin( const Vector3& origin ){
-               //globalOutputStream() << "func_static origin before: " << m_funcStaticOrigin << " after: " << origin << "\n";
-               for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
-               {
-                       ( *i )->getPlane().m_funcStaticOrigin = origin;
-                       ( *i )->getPlane().updateTranslated();
-                       ( *i )->planeChanged();
-               }
-               planeChanged();
-       }
+STRING_CONSTANT( Name, "Brush" );
 
-       void attach( BrushObserver& observer ){
-               for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
-               {
-                       observer.push_back( *( *i ) );
-               }
+Callback<void()> m_lightsChanged;
 
-               for ( SelectableEdges::iterator i = m_select_edges.begin(); i != m_select_edges.end(); ++i )
-               {
-                       observer.edge_push_back( *i );
-               }
+// static data
+static Shader* m_state_point;
+// ----
 
-               for ( SelectableVertices::iterator i = m_select_vertices.begin(); i != m_select_vertices.end(); ++i )
-               {
-                       observer.vertex_push_back( *i );
-               }
+static EBrushType m_type;
+static double m_maxWorldCoord;
 
-               m_observers.insert( &observer );
-       }
+Brush( scene::Node& node, const Callback<void()>& evaluateTransform, const Callback<void()>& boundsChanged ) :
+       m_node( &node ),
+       m_undoable_observer( 0 ),
+       m_map( 0 ),
+       m_render_faces( m_faceCentroidPoints, GL_POINTS ),
+       m_render_vertices( m_uniqueVertexPoints, GL_POINTS ),
+       m_render_edges( m_uniqueEdgePoints, GL_POINTS ),
+       m_evaluateTransform( evaluateTransform ),
+       m_boundsChanged( boundsChanged ),
+       m_planeChanged( false ),
+       m_transformChanged( false ){
+       planeChanged();
+}
+Brush( const Brush& other, scene::Node& node, const Callback<void()>& evaluateTransform, const Callback<void()>& boundsChanged ) :
+       m_node( &node ),
+       m_undoable_observer( 0 ),
+       m_map( 0 ),
+       m_render_faces( m_faceCentroidPoints, GL_POINTS ),
+       m_render_vertices( m_uniqueVertexPoints, GL_POINTS ),
+       m_render_edges( m_uniqueEdgePoints, GL_POINTS ),
+       m_evaluateTransform( evaluateTransform ),
+       m_boundsChanged( boundsChanged ),
+       m_planeChanged( false ),
+       m_transformChanged( false ){
+       copy( other );
+}
+
+Brush( const Brush& other ) :
+       TransformNode( other ),
+       Bounded( other ),
+       Cullable( other ),
+       Snappable(),
+       Undoable( other ),
+       FaceObserver( other ),
+       Filterable( other ),
+       Nameable( other ),
+       BrushDoom3( other ),
+       m_node( 0 ),
+       m_undoable_observer( 0 ),
+       m_map( 0 ),
+       m_render_faces( m_faceCentroidPoints, GL_POINTS ),
+       m_render_vertices( m_uniqueVertexPoints, GL_POINTS ),
+       m_render_edges( m_uniqueEdgePoints, GL_POINTS ),
+       m_planeChanged( false ),
+       m_transformChanged( false ){
+       copy( other );
+}
+
+~Brush(){
+       ASSERT_MESSAGE( m_observers.empty(), "Brush::~Brush: observers still attached" );
+}
+
+// assignment not supported
+Brush& operator=( const Brush& other );
 
-       void detach( BrushObserver& observer ){
-               m_observers.erase( &observer );
+void setDoom3GroupOrigin( const Vector3& origin ){
+       //globalOutputStream() << "func_static origin before: " << m_funcStaticOrigin << " after: " << origin << "\n";
+       for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
+       {
+               ( *i )->getPlane().m_funcStaticOrigin = origin;
+               ( *i )->getPlane().updateTranslated();
+               ( *i )->planeChanged();
        }
+       planeChanged();
+}
 
-       void forEachFace( const BrushVisitor& visitor ) const {
-               for ( Faces::const_iterator i = m_faces.begin(); i != m_faces.end(); ++i )
-               {
-                       visitor.visit( *( *i ) );
-               }
+void attach( BrushObserver& observer ){
+       for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
+       {
+               observer.push_back( *( *i ) );
        }
 
-       void forEachFace_instanceAttach( MapFile* map ) const {
-               for ( Faces::const_iterator i = m_faces.begin(); i != m_faces.end(); ++i )
-               {
-                       ( *i )->instanceAttach( map );
-               }
+       for ( SelectableEdges::iterator i = m_select_edges.begin(); i != m_select_edges.end(); ++i )
+       {
+               observer.edge_push_back( *i );
        }
 
-       void forEachFace_instanceDetach( MapFile* map ) const {
-               for ( Faces::const_iterator i = m_faces.begin(); i != m_faces.end(); ++i )
-               {
-                       ( *i )->instanceDetach( map );
-               }
+       for ( SelectableVertices::iterator i = m_select_vertices.begin(); i != m_select_vertices.end(); ++i )
+       {
+               observer.vertex_push_back( *i );
        }
 
-       InstanceCounter m_instanceCounter;
+       m_observers.insert( &observer );
+}
 
-       void instanceAttach( const scene::Path& path ){
-               if ( ++m_instanceCounter.m_count == 1 ) {
-                       m_map = path_find_mapfile( path.begin(), path.end() );
-                       m_undoable_observer = GlobalUndoSystem().observer( this );
-                       GlobalFilterSystem().registerFilterable( *this );
-                       forEachFace_instanceAttach( m_map );
-               }
-               else
-               {
-                       ASSERT_MESSAGE( path_find_mapfile( path.begin(), path.end() ) == m_map, "node is instanced across more than one file" );
-               }
-       }
+void detach( BrushObserver& observer ){
+       m_observers.erase( &observer );
+}
 
-       void instanceDetach( const scene::Path& path ){
-               if ( --m_instanceCounter.m_count == 0 ) {
-                       forEachFace_instanceDetach( m_map );
-                       GlobalFilterSystem().unregisterFilterable( *this );
-                       m_map = 0;
-                       m_undoable_observer = 0;
-                       GlobalUndoSystem().release( this );
-               }
+void forEachFace( const BrushVisitor& visitor ) const {
+       for ( Faces::const_iterator i = m_faces.begin(); i != m_faces.end(); ++i )
+       {
+               visitor.visit( *( *i ) );
        }
+}
 
-       // nameable
-       const char* name() const {
-               return "brush";
+void forEachFace_instanceAttach( MapFile* map ) const {
+       for ( Faces::const_iterator i = m_faces.begin(); i != m_faces.end(); ++i )
+       {
+               ( *i )->instanceAttach( map );
        }
+}
 
-       void attach( const NameCallback& callback ){
+void forEachFace_instanceDetach( MapFile* map ) const {
+       for ( Faces::const_iterator i = m_faces.begin(); i != m_faces.end(); ++i )
+       {
+               ( *i )->instanceDetach( map );
        }
+}
 
-       void detach( const NameCallback& callback ){
-       }
+InstanceCounter m_instanceCounter;
 
-       // filterable
-       void updateFiltered(){
-               if ( m_node != 0 ) {
-                       if ( brush_filtered( *this ) ) {
-                               m_node->enable( scene::Node::eFiltered );
-                       }
-                       else
-                       {
-                               m_node->disable( scene::Node::eFiltered );
-                       }
-               }
+void instanceAttach( const scene::Path& path ){
+       if ( ++m_instanceCounter.m_count == 1 ) {
+               m_map = path_find_mapfile( path.begin(), path.end() );
+               m_undoable_observer = GlobalUndoSystem().observer( this );
+               GlobalFilterSystem().registerFilterable( *this );
+               forEachFace_instanceAttach( m_map );
        }
-
-       // observer
-       void planeChanged(){
-               m_planeChanged = true;
-               aabbChanged();
-               m_lightsChanged();
+       else
+       {
+               ASSERT_MESSAGE( path_find_mapfile( path.begin(), path.end() ) == m_map, "node is instanced across more than one file" );
        }
+}
 
-       void shaderChanged(){
-               updateFiltered();
-               planeChanged();
+void instanceDetach( const scene::Path& path ){
+       if ( --m_instanceCounter.m_count == 0 ) {
+               forEachFace_instanceDetach( m_map );
+               GlobalFilterSystem().unregisterFilterable( *this );
+               m_map = 0;
+               m_undoable_observer = 0;
+               GlobalUndoSystem().release( this );
        }
+}
 
-       void evaluateBRep() const {
-               if ( m_planeChanged ) {
-                       m_planeChanged = false;
-                       const_cast<Brush*>( this )->buildBRep();
-               }
-       }
+// nameable
+const char* name() const {
+       return "brush";
+}
 
-       void transformChanged(){
-               m_transformChanged = true;
-               planeChanged();
-       }
+void attach( const NameCallback& callback ){
+}
 
-       typedef MemberCaller<Brush, void(), &Brush::transformChanged> TransformChangedCaller;
+void detach( const NameCallback& callback ){
+}
 
-       void evaluateTransform(){
-               if ( m_transformChanged ) {
-                       m_transformChanged = false;
-                       revertTransform();
-                       m_evaluateTransform();
+// filterable
+void updateFiltered(){
+       if ( m_node != 0 ) {
+               if ( brush_filtered( *this ) ) {
+                       m_node->enable( scene::Node::eFiltered );
+               }
+               else
+               {
+                       m_node->disable( scene::Node::eFiltered );
                }
        }
+}
 
-       const Matrix4& localToParent() const {
-               return g_matrix4_identity;
-       }
+// observer
+void planeChanged(){
+       m_planeChanged = true;
+       aabbChanged();
+       m_lightsChanged();
+}
 
-       void aabbChanged(){
-               m_boundsChanged();
-       }
+void shaderChanged(){
+       updateFiltered();
+       planeChanged();
+}
 
-       const AABB& localAABB() const {
-               evaluateBRep();
-               return m_aabb_local;
+void evaluateBRep() const {
+       if ( m_planeChanged ) {
+               m_planeChanged = false;
+               const_cast<Brush*>( this )->buildBRep();
        }
+}
 
-       VolumeIntersectionValue intersectVolume( const VolumeTest& test, const Matrix4& localToWorld ) const {
-               return test.TestAABB( m_aabb_local, localToWorld );
-       }
+void transformChanged(){
+       m_transformChanged = true;
+       planeChanged();
+}
 
-       void renderComponents( SelectionSystem::EComponentMode mode, Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
-               switch ( mode )
-               {
-               case SelectionSystem::eVertex:
-                       renderer.addRenderable( m_render_vertices, localToWorld );
-                       break;
-               case SelectionSystem::eEdge:
-                       renderer.addRenderable( m_render_edges, localToWorld );
-                       break;
-               case SelectionSystem::eFace:
-                       renderer.addRenderable( m_render_faces, localToWorld );
-                       break;
-               default:
-                       break;
-               }
+typedef MemberCaller<Brush, void(), &Brush::transformChanged> TransformChangedCaller;
+
+void evaluateTransform(){
+       if ( m_transformChanged ) {
+               m_transformChanged = false;
+               revertTransform();
+               m_evaluateTransform();
        }
+}
 
-       void transform( const Matrix4& matrix ){
-               bool mirror = matrix4_handedness( matrix ) == MATRIX4_LEFTHANDED;
+const Matrix4& localToParent() const {
+       return g_matrix4_identity;
+}
 
-               for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
-               {
-                       ( *i )->transform( matrix, mirror );
-               }
-       }
+void aabbChanged(){
+       m_boundsChanged();
+}
 
-       void snapto( float snap ){
-               for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
-               {
-                       ( *i )->snapto( snap );
-               }
+const AABB& localAABB() const {
+       evaluateBRep();
+       return m_aabb_local;
+}
+
+VolumeIntersectionValue intersectVolume( const VolumeTest& test, const Matrix4& localToWorld ) const {
+       return test.TestAABB( m_aabb_local, localToWorld );
+}
+
+void renderComponents( SelectionSystem::EComponentMode mode, Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
+       switch ( mode )
+       {
+       case SelectionSystem::eVertex:
+               renderer.addRenderable( m_render_vertices, localToWorld );
+               break;
+       case SelectionSystem::eEdge:
+               renderer.addRenderable( m_render_edges, localToWorld );
+               break;
+       case SelectionSystem::eFace:
+               renderer.addRenderable( m_render_faces, localToWorld );
+               break;
+       default:
+               break;
        }
+}
 
-       void revertTransform(){
-               for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
-               {
-                       ( *i )->revertTransform();
-               }
+void transform( const Matrix4& matrix ){
+       bool mirror = matrix4_handedness( matrix ) == MATRIX4_LEFTHANDED;
+
+       for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
+       {
+               ( *i )->transform( matrix, mirror );
        }
+}
 
-       void freezeTransform(){
-               for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
-               {
-                       ( *i )->freezeTransform();
-               }
+void snapto( float snap ){
+       for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
+       {
+               ( *i )->snapto( snap );
        }
+}
 
-       /// \brief Returns the absolute index of the \p faceVertex.
-       std::size_t absoluteIndex( FaceVertexId faceVertex ){
-               std::size_t index = 0;
-               for ( std::size_t i = 0; i < faceVertex.getFace(); ++i )
-               {
-                       index += m_faces[i]->getWinding().numpoints;
-               }
-               return index + faceVertex.getVertex();
+void revertTransform(){
+       for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
+       {
+               ( *i )->revertTransform();
        }
+}
 
-       void appendFaces( const Faces& other ){
-               clear();
-               for ( Faces::const_iterator i = other.begin(); i != other.end(); ++i )
-               {
-                       push_back( *i );
-               }
+void freezeTransform(){
+       for ( Faces::iterator i = m_faces.begin(); i != m_faces.end(); ++i )
+       {
+               ( *i )->freezeTransform();
        }
+}
 
-       /// \brief The undo memento for a brush stores only the list of face references - the faces are not copied.
-       class BrushUndoMemento : public UndoMemento
+/// \brief Returns the absolute index of the \p faceVertex.
+std::size_t absoluteIndex( FaceVertexId faceVertex ){
+       std::size_t index = 0;
+       for ( std::size_t i = 0; i < faceVertex.getFace(); ++i )
        {
-       public:
-               BrushUndoMemento( const Faces& faces ) : m_faces( faces ){
+               index += m_faces[i]->getWinding().numpoints;
        }
+       return index + faceVertex.getVertex();
+}
 
-       void release(){
-               delete this;
+void appendFaces( const Faces& other ){
+       clear();
+       for ( Faces::const_iterator i = other.begin(); i != other.end(); ++i )
+       {
+               push_back( *i );
        }
+}
 
-       Faces m_faces;
-       };
+/// \brief The undo memento for a brush stores only the list of face references - the faces are not copied.
+class BrushUndoMemento : public UndoMemento
+{
+public:
+BrushUndoMemento( const Faces& faces ) : m_faces( faces ){
+}
 
-       void undoSave(){
-               if ( m_map != 0 ) {
-                       m_map->changed();
-               }
-               if ( m_undoable_observer != 0 ) {
-                       m_undoable_observer->save( this );
-               }
-       }
+void release(){
+       delete this;
+}
 
-       UndoMemento* exportState() const {
-               return new BrushUndoMemento( m_faces );
+Faces m_faces;
+};
+
+void undoSave(){
+       if ( m_map != 0 ) {
+               m_map->changed();
        }
+       if ( m_undoable_observer != 0 ) {
+               m_undoable_observer->save( this );
+       }
+}
 
-       void importState( const UndoMemento* state ){
-               undoSave();
-               appendFaces( static_cast<const BrushUndoMemento*>( state )->m_faces );
-               planeChanged();
+UndoMemento* exportState() const {
+       return new BrushUndoMemento( m_faces );
+}
 
-               for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
-               {
-                       ( *i )->DEBUG_verify();
-               }
-       }
+void importState( const UndoMemento* state ){
+       undoSave();
+       appendFaces( static_cast<const BrushUndoMemento*>( state )->m_faces );
+       planeChanged();
 
-       bool isDetail(){
-               return !m_faces.empty() && m_faces.front()->isDetail();
+       for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
+       {
+               ( *i )->DEBUG_verify();
        }
+}
+
+bool isDetail(){
+       return !m_faces.empty() && m_faces.front()->isDetail();
+}
 
-       /// \brief Appends a copy of \p face to the end of the face list.
+/// \brief Appends a copy of \p face to the end of the face list.
        std::shared_ptr<Face> addFace( const Face& face ){
-               if ( m_faces.size() == c_brush_maxFaces ) {
-                       return 0;
-               }
-               undoSave();
-               push_back( std::make_shared<Face>( face, this ) );
-               m_faces.back()->setDetail( isDetail() );
-               planeChanged();
-               return m_faces.back();
+       if ( m_faces.size() == c_brush_maxFaces ) {
+               return 0;
        }
+       undoSave();
+               push_back( std::make_shared<Face>( face, this ) );
+       m_faces.back()->setDetail( isDetail() );
+       planeChanged();
+       return m_faces.back();
+}
 
-       /// \brief Appends a new face constructed from the parameters to the end of the face list.
+/// \brief Appends a new face constructed from the parameters to the end of the face list.
        std::shared_ptr<Face> addPlane( const Vector3& p0, const Vector3& p1, const Vector3& p2, const char* shader, const TextureProjection& projection ){
-               if ( m_faces.size() == c_brush_maxFaces ) {
-                       return 0;
-               }
-               undoSave();
-               push_back( std::make_shared<Face>( p0, p1, p2, shader, projection, this ) );
-               m_faces.back()->setDetail( isDetail() );
-               planeChanged();
-               return m_faces.back();
+       if ( m_faces.size() == c_brush_maxFaces ) {
+               return 0;
        }
+       undoSave();
+               push_back( std::make_shared<Face>( p0, p1, p2, shader, projection, this ) );
+       m_faces.back()->setDetail( isDetail() );
+       planeChanged();
+       return m_faces.back();
+}
 
-       static void constructStatic( EBrushType type ){
-               m_type = type;
-               Face::m_type = type;
-               FacePlane::m_type = type;
+static void constructStatic( EBrushType type ){
+       m_type = type;
+       Face::m_type = type;
+       FacePlane::m_type = type;
 
-               g_bp_globals.m_texdefTypeId = TEXDEFTYPEID_QUAKE;
-               if ( m_type == eBrushTypeQuake3BP || m_type == eBrushTypeDoom3 || m_type == eBrushTypeQuake4 ) {
-                       g_bp_globals.m_texdefTypeId = TEXDEFTYPEID_BRUSHPRIMITIVES;
-                       // g_brush_texturelock_enabled = true; // bad idea, this overrides user setting
-               }
-               else if ( m_type == eBrushTypeHalfLife ) {
-                       g_bp_globals.m_texdefTypeId = TEXDEFTYPEID_HALFLIFE;
-                       // g_brush_texturelock_enabled = true; // bad idea, this overrides user setting
-               }
+       g_bp_globals.m_texdefTypeId = TEXDEFTYPEID_QUAKE;
+       if ( m_type == eBrushTypeQuake3BP || m_type == eBrushTypeDoom3 || m_type == eBrushTypeQuake4 ) {
+               g_bp_globals.m_texdefTypeId = TEXDEFTYPEID_BRUSHPRIMITIVES;
+               // g_brush_texturelock_enabled = true; // bad idea, this overrides user setting
+       }
+       else if ( m_type == eBrushTypeHalfLife ) {
+               g_bp_globals.m_texdefTypeId = TEXDEFTYPEID_HALFLIFE;
+               // g_brush_texturelock_enabled = true; // bad idea, this overrides user setting
+       }
 
-               Face::m_quantise = ( m_type == eBrushTypeQuake ) ? quantiseInteger : quantiseFloating;
+       Face::m_quantise = ( m_type == eBrushTypeQuake ) ? quantiseInteger : quantiseFloating;
 
-               m_state_point = GlobalShaderCache().capture( "$POINT" );
-       }
+       m_state_point = GlobalShaderCache().capture( "$POINT" );
+}
 
-       static void destroyStatic(){
-               GlobalShaderCache().release( "$POINT" );
-       }
+static void destroyStatic(){
+       GlobalShaderCache().release( "$POINT" );
+}
 
-       std::size_t DEBUG_size(){
-               return m_faces.size();
-       }
+std::size_t DEBUG_size(){
+       return m_faces.size();
+}
 
-       typedef Faces::const_iterator const_iterator;
+typedef Faces::const_iterator const_iterator;
 
-       const_iterator begin() const {
-               return m_faces.begin();
-       }
+const_iterator begin() const {
+       return m_faces.begin();
+}
 
-       const_iterator end() const {
-               return m_faces.end();
-       }
+const_iterator end() const {
+       return m_faces.end();
+}
 
        std::shared_ptr<Face> back(){
-               return m_faces.back();
-       }
+       return m_faces.back();
+}
 
        const std::shared_ptr<Face> back() const {
-               return m_faces.back();
-       }
+       return m_faces.back();
+}
 
-       void reserve( std::size_t count ){
-               m_faces.reserve( count );
-               for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
-               {
-                       ( *i )->reserve( count );
-               }
+void reserve( std::size_t count ){
+       m_faces.reserve( count );
+       for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
+       {
+               ( *i )->reserve( count );
        }
+}
 
-       void push_back( Faces::value_type face ){
-               m_faces.push_back( face );
-               if ( m_instanceCounter.m_count != 0 ) {
-                       m_faces.back()->instanceAttach( m_map );
-               }
-               for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
-               {
-                       ( *i )->push_back( *face );
-                       ( *i )->DEBUG_verify();
-               }
+void push_back( Faces::value_type face ){
+       m_faces.push_back( face );
+       if ( m_instanceCounter.m_count != 0 ) {
+               m_faces.back()->instanceAttach( m_map );
+       }
+       for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
+       {
+               ( *i )->push_back( *face );
+               ( *i )->DEBUG_verify();
        }
+}
 
-       void pop_back(){
-               if ( m_instanceCounter.m_count != 0 ) {
-                       m_faces.back()->instanceDetach( m_map );
-               }
-               m_faces.pop_back();
-               for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
-               {
-                       ( *i )->pop_back();
-                       ( *i )->DEBUG_verify();
-               }
+void pop_back(){
+       if ( m_instanceCounter.m_count != 0 ) {
+               m_faces.back()->instanceDetach( m_map );
+       }
+       m_faces.pop_back();
+       for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
+       {
+               ( *i )->pop_back();
+               ( *i )->DEBUG_verify();
        }
+}
 
-       void erase( std::size_t index ){
-               if ( m_instanceCounter.m_count != 0 ) {
-                       m_faces[index]->instanceDetach( m_map );
-               }
-               m_faces.erase( m_faces.begin() + index );
-               for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
-               {
-                       ( *i )->erase( index );
-                       ( *i )->DEBUG_verify();
-               }
+void erase( std::size_t index ){
+       if ( m_instanceCounter.m_count != 0 ) {
+               m_faces[index]->instanceDetach( m_map );
+       }
+       m_faces.erase( m_faces.begin() + index );
+       for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
+       {
+               ( *i )->erase( index );
+               ( *i )->DEBUG_verify();
        }
+}
 
-       void connectivityChanged(){
-               for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
-               {
-                       ( *i )->connectivityChanged();
-               }
+void connectivityChanged(){
+       for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
+       {
+               ( *i )->connectivityChanged();
        }
+}
 
 
-       void clear(){
-               undoSave();
-               if ( m_instanceCounter.m_count != 0 ) {
-                       forEachFace_instanceDetach( m_map );
-               }
-               m_faces.clear();
-               for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
-               {
-                       ( *i )->clear();
-                       ( *i )->DEBUG_verify();
-               }
+void clear(){
+       undoSave();
+       if ( m_instanceCounter.m_count != 0 ) {
+               forEachFace_instanceDetach( m_map );
        }
-
-       std::size_t size() const {
-               return m_faces.size();
+       m_faces.clear();
+       for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
+       {
+               ( *i )->clear();
+               ( *i )->DEBUG_verify();
        }
+}
 
-       bool empty() const {
-               return m_faces.empty();
-       }
+std::size_t size() const {
+       return m_faces.size();
+}
 
-       /// \brief Returns true if any face of the brush contributes to the final B-Rep.
-       bool hasContributingFaces() const {
-               for ( const_iterator i = begin(); i != end(); ++i )
-               {
-                       if ( ( *i )->contributes() ) {
-                               return true;
-                       }
+bool empty() const {
+       return m_faces.empty();
+}
+
+/// \brief Returns true if any face of the brush contributes to the final B-Rep.
+bool hasContributingFaces() const {
+       for ( const_iterator i = begin(); i != end(); ++i )
+       {
+               if ( ( *i )->contributes() ) {
+                       return true;
                }
-               return false;
        }
+       return false;
+}
 
-       /// \brief Removes faces that do not contribute to the brush. This is useful for cleaning up after CSG operations on the brush.
-       /// Note: removal of empty faces is not performed during direct brush manipulations, because it would make a manipulation irreversible if it created an empty face.
-       void removeEmptyFaces(){
-               evaluateBRep();
+/// \brief Removes faces that do not contribute to the brush. This is useful for cleaning up after CSG operations on the brush.
+/// Note: removal of empty faces is not performed during direct brush manipulations, because it would make a manipulation irreversible if it created an empty face.
+void removeEmptyFaces(){
+       evaluateBRep();
 
+       {
+               std::size_t i = 0;
+               while ( i < m_faces.size() )
                {
-                       std::size_t i = 0;
-                       while ( i < m_faces.size() )
+                       if ( !m_faces[i]->contributes() ) {
+                               erase( i );
+                               planeChanged();
+                       }
+                       else
                        {
-                               if ( !m_faces[i]->contributes() ) {
-                                       erase( i );
-                                       planeChanged();
-                               }
-                               else
-                               {
-                                       ++i;
-                               }
+                               ++i;
                        }
                }
        }
+}
 
-       /// \brief Constructs \p winding from the intersection of \p plane with the other planes of the brush.
-       void windingForClipPlane( Winding& winding, const Plane3& plane ) const {
-               FixedWinding buffer[2];
-               bool swap = false;
+/// \brief Constructs \p winding from the intersection of \p plane with the other planes of the brush.
+void windingForClipPlane( Winding& winding, const Plane3& plane ) const {
+       FixedWinding buffer[2];
+       bool swap = false;
 
-               // get a poly that covers an effectively infinite area
-               Winding_createInfinite( buffer[swap], plane, m_maxWorldCoord + 1 );
+       // get a poly that covers an effectively infinite area
+       Winding_createInfinite( buffer[swap], plane, m_maxWorldCoord + 1 );
 
-               // chop the poly by all of the other faces
+       // chop the poly by all of the other faces
+       {
+               for ( std::size_t i = 0; i < m_faces.size(); ++i )
                {
-                       for ( std::size_t i = 0; i < m_faces.size(); ++i )
-                       {
-                               const Face& clip = *m_faces[i];
+                       const Face& clip = *m_faces[i];
 
-                               if ( plane3_equal( clip.plane3(), plane )
-                                        || !plane3_valid( clip.plane3() ) || !plane_unique( i )
-                                        || plane3_opposing( plane, clip.plane3() ) ) {
-                                       continue;
-                               }
+                       if ( plane3_equal( clip.plane3(), plane )
+                                || !plane3_valid( clip.plane3() ) || !plane_unique( i )
+                                || plane3_opposing( plane, clip.plane3() ) ) {
+                               continue;
+                       }
 
-                               buffer[!swap].clear();
+                       buffer[!swap].clear();
 
 #if BRUSH_CONNECTIVITY_DEBUG
-                               globalOutputStream() << "clip vs face: " << i << "\n";
+                       globalOutputStream() << "clip vs face: " << i << "\n";
 #endif
 
-                               {
-                                       // flip the plane, because we want to keep the back side
-                                       Plane3 clipPlane( vector3_negated( clip.plane3().normal() ), -clip.plane3().dist() );
-                                       Winding_Clip( buffer[swap], plane, clipPlane, i, buffer[!swap] );
-                               }
+                       {
+                               // flip the plane, because we want to keep the back side
+                               Plane3 clipPlane( vector3_negated( clip.plane3().normal() ), -clip.plane3().dist() );
+                               Winding_Clip( buffer[swap], plane, clipPlane, i, buffer[!swap] );
+                       }
 
 #if BRUSH_CONNECTIVITY_DEBUG
-                               for ( FixedWinding::Points::iterator k = buffer[!swap].points.begin(), j = buffer[!swap].points.end() - 1; k != buffer[!swap].points.end(); j = k, ++k )
-                               {
-                                       if ( vector3_length_squared( vector3_subtracted( ( *k ).vertex, ( *j ).vertex ) ) < 1 ) {
-                                               globalOutputStream() << "v: " << std::distance( buffer[!swap].points.begin(), j ) << " tiny edge adjacent to face " << ( *j ).adjacent << "\n";
-                                       }
+                       for ( FixedWinding::Points::iterator k = buffer[!swap].points.begin(), j = buffer[!swap].points.end() - 1; k != buffer[!swap].points.end(); j = k, ++k )
+                       {
+                               if ( vector3_length_squared( vector3_subtracted( ( *k ).vertex, ( *j ).vertex ) ) < 1 ) {
+                                       globalOutputStream() << "v: " << std::distance( buffer[!swap].points.begin(), j ) << " tiny edge adjacent to face " << ( *j ).adjacent << "\n";
                                }
+                       }
 #endif
 
-                               //ASSERT_MESSAGE(buffer[!swap].numpoints != 1, "created single-point winding");
+                       //ASSERT_MESSAGE(buffer[!swap].numpoints != 1, "created single-point winding");
 
-                               swap = !swap;
-                       }
+                       swap = !swap;
                }
+       }
 
-               Winding_forFixedWinding( winding, buffer[swap] );
+       Winding_forFixedWinding( winding, buffer[swap] );
 
 #if BRUSH_CONNECTIVITY_DEBUG
-               Winding_printConnectivity( winding );
+       Winding_printConnectivity( winding );
 
-               for ( Winding::iterator i = winding.begin(), j = winding.end() - 1; i != winding.end(); j = i, ++i )
-               {
-                       if ( vector3_length_squared( vector3_subtracted( ( *i ).vertex, ( *j ).vertex ) ) < 1 ) {
-                               globalOutputStream() << "v: " << std::distance( winding.begin(), j ) << " tiny edge adjacent to face " << ( *j ).adjacent << "\n";
-                       }
+       for ( Winding::iterator i = winding.begin(), j = winding.end() - 1; i != winding.end(); j = i, ++i )
+       {
+               if ( vector3_length_squared( vector3_subtracted( ( *i ).vertex, ( *j ).vertex ) ) < 1 ) {
+                       globalOutputStream() << "v: " << std::distance( winding.begin(), j ) << " tiny edge adjacent to face " << ( *j ).adjacent << "\n";
                }
-#endif
        }
+#endif
+}
 
-       void update_wireframe( RenderableWireframe& wire, const bool* faces_visible ) const {
-               wire.m_faceVertex.resize( m_edge_indices.size() );
-               wire.m_vertices = m_uniqueVertexPoints.data();
-               wire.m_size = 0;
-               for ( std::size_t i = 0; i < m_edge_faces.size(); ++i )
-               {
-                       if ( faces_visible[m_edge_faces[i].first]
-                                || faces_visible[m_edge_faces[i].second] ) {
-                               wire.m_faceVertex[wire.m_size++] = m_edge_indices[i];
-                       }
+void update_wireframe( RenderableWireframe& wire, const bool* faces_visible ) const {
+       wire.m_faceVertex.resize( m_edge_indices.size() );
+       wire.m_vertices = m_uniqueVertexPoints.data();
+       wire.m_size = 0;
+       for ( std::size_t i = 0; i < m_edge_faces.size(); ++i )
+       {
+               if ( faces_visible[m_edge_faces[i].first]
+                        || faces_visible[m_edge_faces[i].second] ) {
+                       wire.m_faceVertex[wire.m_size++] = m_edge_indices[i];
                }
        }
+}
 
 
-       void update_faces_wireframe( Array<PointVertex>& wire, const bool* faces_visible ) const {
-               std::size_t count = 0;
-               for ( std::size_t i = 0; i < m_faceCentroidPoints.size(); ++i )
-               {
-                       if ( faces_visible[i] ) {
-                               ++count;
-                       }
+void update_faces_wireframe( Array<PointVertex>& wire, const bool* faces_visible ) const {
+       std::size_t count = 0;
+       for ( std::size_t i = 0; i < m_faceCentroidPoints.size(); ++i )
+       {
+               if ( faces_visible[i] ) {
+                       ++count;
                }
+       }
 
-               wire.resize( count );
-               Array<PointVertex>::iterator p = wire.begin();
-               for ( std::size_t i = 0; i < m_faceCentroidPoints.size(); ++i )
-               {
-                       if ( faces_visible[i] ) {
-                               *p++ = m_faceCentroidPoints[i];
-                       }
+       wire.resize( count );
+       Array<PointVertex>::iterator p = wire.begin();
+       for ( std::size_t i = 0; i < m_faceCentroidPoints.size(); ++i )
+       {
+               if ( faces_visible[i] ) {
+                       *p++ = m_faceCentroidPoints[i];
                }
        }
+}
 
-       /// \brief Makes this brush a deep-copy of the \p other.
-       void copy( const Brush& other ){
-               for ( Faces::const_iterator i = other.m_faces.begin(); i != other.m_faces.end(); ++i )
-               {
-                       addFace( *( *i ) );
-               }
-               planeChanged();
+/// \brief Makes this brush a deep-copy of the \p other.
+void copy( const Brush& other ){
+       for ( Faces::const_iterator i = other.m_faces.begin(); i != other.m_faces.end(); ++i )
+       {
+               addFace( *( *i ) );
        }
+       planeChanged();
+}
 
-       private:
-       void edge_push_back( FaceVertexId faceVertex ){
-               m_select_edges.push_back( SelectableEdge( m_faces, faceVertex ) );
-               for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
-               {
-                       ( *i )->edge_push_back( m_select_edges.back() );
-               }
+private:
+void edge_push_back( FaceVertexId faceVertex ){
+       m_select_edges.push_back( SelectableEdge( m_faces, faceVertex ) );
+       for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
+       {
+               ( *i )->edge_push_back( m_select_edges.back() );
        }
+}
 
-       void edge_clear(){
-               m_select_edges.clear();
-               for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
-               {
-                       ( *i )->edge_clear();
-               }
+void edge_clear(){
+       m_select_edges.clear();
+       for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
+       {
+               ( *i )->edge_clear();
        }
+}
 
-       void vertex_push_back( FaceVertexId faceVertex ){
-               m_select_vertices.push_back( SelectableVertex( m_faces, faceVertex ) );
-               for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
-               {
-                       ( *i )->vertex_push_back( m_select_vertices.back() );
-               }
+void vertex_push_back( FaceVertexId faceVertex ){
+       m_select_vertices.push_back( SelectableVertex( m_faces, faceVertex ) );
+       for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
+       {
+               ( *i )->vertex_push_back( m_select_vertices.back() );
        }
+}
 
-       void vertex_clear(){
-               m_select_vertices.clear();
-               for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
-               {
-                       ( *i )->vertex_clear();
-               }
+void vertex_clear(){
+       m_select_vertices.clear();
+       for ( Observers::iterator i = m_observers.begin(); i != m_observers.end(); ++i )
+       {
+               ( *i )->vertex_clear();
        }
+}
 
-       /// \brief Returns true if the face identified by \p index is preceded by another plane that takes priority over it.
-       bool plane_unique( std::size_t index ) const {
-               // duplicate plane
-               for ( std::size_t i = 0; i < m_faces.size(); ++i )
-               {
-                       if ( index != i && !plane3_inside( m_faces[index]->plane3(), m_faces[i]->plane3(), index < i ) ) {
-                               return false;
-                       }
+/// \brief Returns true if the face identified by \p index is preceded by another plane that takes priority over it.
+bool plane_unique( std::size_t index ) const {
+       // duplicate plane
+       for ( std::size_t i = 0; i < m_faces.size(); ++i )
+       {
+               if ( index != i && !plane3_inside( m_faces[index]->plane3(), m_faces[i]->plane3(), index < i ) ) {
+                       return false;
                }
-               return true;
        }
+       return true;
+}
 
-       /// \brief Removes edges that are smaller than the tolerance used when generating brush windings.
-       void removeDegenerateEdges(){
-               for ( std::size_t i = 0; i < m_faces.size(); ++i )
+/// \brief Removes edges that are smaller than the tolerance used when generating brush windings.
+void removeDegenerateEdges(){
+       for ( std::size_t i = 0; i < m_faces.size(); ++i )
+       {
+               Winding& winding = m_faces[i]->getWinding();
+               for ( Winding::iterator j = winding.begin(); j != winding.end(); )
                {
-                       Winding& winding = m_faces[i]->getWinding();
-                       for ( Winding::iterator j = winding.begin(); j != winding.end(); )
-                       {
-                               std::size_t index = std::distance( winding.begin(), j );
-                               std::size_t next = Winding_next( winding, index );
-                               if ( Edge_isDegenerate( winding[index].vertex, winding[next].vertex ) ) {
+                       std::size_t index = std::distance( winding.begin(), j );
+                       std::size_t next = Winding_next( winding, index );
+                       if ( Edge_isDegenerate( winding[index].vertex, winding[next].vertex ) ) {
 #if BRUSH_DEGENERATE_DEBUG
-                                       globalOutputStream() << "Brush::buildWindings: face " << i << ": degenerate edge adjacent to " << winding[index].adjacent << "\n";
+                               globalOutputStream() << "Brush::buildWindings: face " << i << ": degenerate edge adjacent to " << winding[index].adjacent << "\n";
 #endif
-                                       Winding& other = m_faces[winding[index].adjacent]->getWinding();
-                                       std::size_t adjacent = Winding_FindAdjacent( other, i );
-                                       if ( adjacent != c_brush_maxFaces ) {
-                                               other.erase( other.begin() + adjacent );
-                                       }
-                                       winding.erase( j );
-                               }
-                               else
-                               {
-                                       ++j;
+                               Winding& other = m_faces[winding[index].adjacent]->getWinding();
+                               std::size_t adjacent = Winding_FindAdjacent( other, i );
+                               if ( adjacent != c_brush_maxFaces ) {
+                                       other.erase( other.begin() + adjacent );
                                }
+                               winding.erase( j );
+                       }
+                       else
+                       {
+                               ++j;
                        }
                }
        }
+}
 
-       /// \brief Invalidates faces that have only two vertices in their winding, while preserving edge-connectivity information.
-       void removeDegenerateFaces(){
-               // save adjacency info for degenerate faces
-               for ( std::size_t i = 0; i < m_faces.size(); ++i )
-               {
-                       Winding& degen = m_faces[i]->getWinding();
+/// \brief Invalidates faces that have only two vertices in their winding, while preserving edge-connectivity information.
+void removeDegenerateFaces(){
+       // save adjacency info for degenerate faces
+       for ( std::size_t i = 0; i < m_faces.size(); ++i )
+       {
+               Winding& degen = m_faces[i]->getWinding();
 
-                       if ( degen.numpoints == 2 ) {
+               if ( degen.numpoints == 2 ) {
 #if BRUSH_DEGENERATE_DEBUG
-                               globalOutputStream() << "Brush::buildWindings: face " << i << ": degenerate winding adjacent to " << degen[0].adjacent << ", " << degen[1].adjacent << "\n";
+                       globalOutputStream() << "Brush::buildWindings: face " << i << ": degenerate winding adjacent to " << degen[0].adjacent << ", " << degen[1].adjacent << "\n";
 #endif
-                               // this is an "edge" face, where the plane touches the edge of the brush
-                               {
-                                       Winding& winding = m_faces[degen[0].adjacent]->getWinding();
-                                       std::size_t index = Winding_FindAdjacent( winding, i );
-                                       if ( index != c_brush_maxFaces ) {
+                       // this is an "edge" face, where the plane touches the edge of the brush
+                       {
+                               Winding& winding = m_faces[degen[0].adjacent]->getWinding();
+                               std::size_t index = Winding_FindAdjacent( winding, i );
+                               if ( index != c_brush_maxFaces ) {
 #if BRUSH_DEGENERATE_DEBUG
-                                               globalOutputStream() << "Brush::buildWindings: face " << degen[0].adjacent << ": remapping adjacent " << winding[index].adjacent << " to " << degen[1].adjacent << "\n";
+                                       globalOutputStream() << "Brush::buildWindings: face " << degen[0].adjacent << ": remapping adjacent " << winding[index].adjacent << " to " << degen[1].adjacent << "\n";
 #endif
-                                               winding[index].adjacent = degen[1].adjacent;
-                                       }
+                                       winding[index].adjacent = degen[1].adjacent;
                                }
+                       }
 
-                               {
-                                       Winding& winding = m_faces[degen[1].adjacent]->getWinding();
-                                       std::size_t index = Winding_FindAdjacent( winding, i );
-                                       if ( index != c_brush_maxFaces ) {
+                       {
+                               Winding& winding = m_faces[degen[1].adjacent]->getWinding();
+                               std::size_t index = Winding_FindAdjacent( winding, i );
+                               if ( index != c_brush_maxFaces ) {
 #if BRUSH_DEGENERATE_DEBUG
-                                               globalOutputStream() << "Brush::buildWindings: face " << degen[1].adjacent << ": remapping adjacent " << winding[index].adjacent << " to " << degen[0].adjacent << "\n";
+                                       globalOutputStream() << "Brush::buildWindings: face " << degen[1].adjacent << ": remapping adjacent " << winding[index].adjacent << " to " << degen[0].adjacent << "\n";
 #endif
-                                               winding[index].adjacent = degen[0].adjacent;
-                                       }
+                                       winding[index].adjacent = degen[0].adjacent;
                                }
-
-                               degen.resize( 0 );
                        }
+
+                       degen.resize( 0 );
                }
        }
+}
 
-       /// \brief Removes edges that have the same adjacent-face as their immediate neighbour.
-       void removeDuplicateEdges(){
-               // verify face connectivity graph
-               for ( std::size_t i = 0; i < m_faces.size(); ++i )
+/// \brief Removes edges that have the same adjacent-face as their immediate neighbour.
+void removeDuplicateEdges(){
+       // verify face connectivity graph
+       for ( std::size_t i = 0; i < m_faces.size(); ++i )
+       {
+               //if(m_faces[i]->contributes())
                {
-                       //if(m_faces[i]->contributes())
+                       Winding& winding = m_faces[i]->getWinding();
+                       for ( std::size_t j = 0; j != winding.numpoints; )
                        {
-                               Winding& winding = m_faces[i]->getWinding();
-                               for ( std::size_t j = 0; j != winding.numpoints; )
-                               {
-                                       std::size_t next = Winding_next( winding, j );
-                                       if ( winding[j].adjacent == winding[next].adjacent ) {
+                               std::size_t next = Winding_next( winding, j );
+                               if ( winding[j].adjacent == winding[next].adjacent ) {
 #if BRUSH_DEGENERATE_DEBUG
-                                               globalOutputStream() << "Brush::buildWindings: face " << i << ": removed duplicate edge adjacent to face " << winding[j].adjacent << "\n";
+                                       globalOutputStream() << "Brush::buildWindings: face " << i << ": removed duplicate edge adjacent to face " << winding[j].adjacent << "\n";
 #endif
-                                               winding.erase( winding.begin() + next );
-                                       }
-                                       else
-                                       {
-                                               ++j;
-                                       }
+                                       winding.erase( winding.begin() + next );
+                               }
+                               else
+                               {
+                                       ++j;
                                }
                        }
                }
        }
+}
 
-       /// \brief Removes edges that do not have a matching pair in their adjacent-face.
-       void verifyConnectivityGraph(){
-               // verify face connectivity graph
-               for ( std::size_t i = 0; i < m_faces.size(); ++i )
+/// \brief Removes edges that do not have a matching pair in their adjacent-face.
+void verifyConnectivityGraph(){
+       // verify face connectivity graph
+       for ( std::size_t i = 0; i < m_faces.size(); ++i )
+       {
+               //if(m_faces[i]->contributes())
                {
-                       //if(m_faces[i]->contributes())
+                       Winding& winding = m_faces[i]->getWinding();
+                       for ( Winding::iterator j = winding.begin(); j != winding.end(); )
                        {
-                               Winding& winding = m_faces[i]->getWinding();
-                               for ( Winding::iterator j = winding.begin(); j != winding.end(); )
-                               {
 #if BRUSH_CONNECTIVITY_DEBUG
-                                       globalOutputStream() << "Brush::buildWindings: face " << i << ": adjacent to face " << ( *j ).adjacent << "\n";
+                               globalOutputStream() << "Brush::buildWindings: face " << i << ": adjacent to face " << ( *j ).adjacent << "\n";
 #endif
-                                       // remove unidirectional graph edges
-                                       if ( ( *j ).adjacent == c_brush_maxFaces
-                                                || Winding_FindAdjacent( m_faces[( *j ).adjacent]->getWinding(), i ) == c_brush_maxFaces ) {
+                               // remove unidirectional graph edges
+                               if ( ( *j ).adjacent == c_brush_maxFaces
+                                        || Winding_FindAdjacent( m_faces[( *j ).adjacent]->getWinding(), i ) == c_brush_maxFaces ) {
 #if BRUSH_CONNECTIVITY_DEBUG
-                                               globalOutputStream() << "Brush::buildWindings: face " << i << ": removing unidirectional connectivity graph edge adjacent to face " << ( *j ).adjacent << "\n";
+                                       globalOutputStream() << "Brush::buildWindings: face " << i << ": removing unidirectional connectivity graph edge adjacent to face " << ( *j ).adjacent << "\n";
 #endif
-                                               winding.erase( j );
-                                       }
-                                       else
-                                       {
-                                               ++j;
-                                       }
+                                       winding.erase( j );
+                               }
+                               else
+                               {
+                                       ++j;
                                }
                        }
                }
        }
+}
 
-       /// \brief Returns true if the brush is a finite volume. A brush without a finite volume extends past the maximum world bounds and is not valid.
-       bool isBounded(){
-               for ( const_iterator i = begin(); i != end(); ++i )
-               {
-                       if ( !( *i )->is_bounded() ) {
-                               return false;
-                       }
+/// \brief Returns true if the brush is a finite volume. A brush without a finite volume extends past the maximum world bounds and is not valid.
+bool isBounded(){
+       for ( const_iterator i = begin(); i != end(); ++i )
+       {
+               if ( !( *i )->is_bounded() ) {
+                       return false;
                }
-               return true;
        }
+       return true;
+}
 
-       /// \brief Constructs the polygon windings for each face of the brush. Also updates the brush bounding-box and face texture-coordinates.
-       bool buildWindings(){
+/// \brief Constructs the polygon windings for each face of the brush. Also updates the brush bounding-box and face texture-coordinates.
+bool buildWindings(){
 
+       {
+               m_aabb_local = AABB();
+
+               for ( std::size_t i = 0; i < m_faces.size(); ++i )
                {
-                       m_aabb_local = AABB();
+                       Face& f = *m_faces[i];
 
-                       for ( std::size_t i = 0; i < m_faces.size(); ++i )
+                       if ( !plane3_valid( f.plane3() ) || !plane_unique( i ) ) {
+                               f.getWinding().resize( 0 );
+                       }
+                       else
                        {
-                               Face& f = *m_faces[i];
-
-                               if ( !plane3_valid( f.plane3() ) || !plane_unique( i ) ) {
-                                       f.getWinding().resize( 0 );
-                               }
-                               else
-                               {
 #if BRUSH_CONNECTIVITY_DEBUG
-                                       globalOutputStream() << "face: " << i << "\n";
+                               globalOutputStream() << "face: " << i << "\n";
 #endif
-                                       windingForClipPlane( f.getWinding(), f.plane3() );
+                               windingForClipPlane( f.getWinding(), f.plane3() );
 
-                                       // update brush bounds
-                                       const Winding& winding = f.getWinding();
-                                       for ( Winding::const_iterator i = winding.begin(); i != winding.end(); ++i )
-                                       {
-                                               aabb_extend_by_point_safe( m_aabb_local, ( *i ).vertex );
-                                       }
-
-                                       // update texture coordinates
-                                       f.EmitTextureCoordinates();
+                               // update brush bounds
+                               const Winding& winding = f.getWinding();
+                               for ( Winding::const_iterator i = winding.begin(); i != winding.end(); ++i )
+                               {
+                                       aabb_extend_by_point_safe( m_aabb_local, ( *i ).vertex );
                                }
+
+                               // update texture coordinates
+                               f.EmitTextureCoordinates();
                        }
                }
+       }
 
-               bool degenerate = !isBounded();
-
-               if ( !degenerate ) {
-                       // clean up connectivity information.
-                       // these cleanups must be applied in a specific order.
-                       removeDegenerateEdges();
-                       removeDegenerateFaces();
-                       removeDuplicateEdges();
-                       verifyConnectivityGraph();
-               }
+       bool degenerate = !isBounded();
 
-               return degenerate;
+       if ( !degenerate ) {
+               // clean up connectivity information.
+               // these cleanups must be applied in a specific order.
+               removeDegenerateEdges();
+               removeDegenerateFaces();
+               removeDuplicateEdges();
+               verifyConnectivityGraph();
        }
 
-       /// \brief Constructs the face windings and updates anything that depends on them.
-       void buildBRep();
+       return degenerate;
+}
+
+/// \brief Constructs the face windings and updates anything that depends on them.
+void buildBRep();
 };
 
 
@@ -2399,32 +2419,32 @@ class FaceInstance;
 
 class FaceInstanceSet
 {
-       typedef SelectionList<FaceInstance> FaceInstances;
-       FaceInstances m_faceInstances;
+typedef SelectionList<FaceInstance> FaceInstances;
+FaceInstances m_faceInstances;
 public:
-       void insert( FaceInstance& faceInstance ){
-               m_faceInstances.append( faceInstance );
-       }
+void insert( FaceInstance& faceInstance ){
+       m_faceInstances.append( faceInstance );
+}
 
-       void erase( FaceInstance& faceInstance ){
-               m_faceInstances.erase( faceInstance );
-       }
+void erase( FaceInstance& faceInstance ){
+       m_faceInstances.erase( faceInstance );
+}
 
-       template<typename Functor>
-       void foreach( Functor functor ){
-               for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       functor( *( *i ) );
-               }
+template<typename Functor>
+void foreach( Functor functor ){
+       for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               functor( *( *i ) );
        }
+}
 
-       bool empty() const {
-               return m_faceInstances.empty();
-       }
+bool empty() const {
+       return m_faceInstances.empty();
+}
 
-       FaceInstance& last() const {
-               return m_faceInstances.back();
-       }
+FaceInstance& last() const {
+       return m_faceInstances.back();
+}
 };
 
 extern FaceInstanceSet g_SelectedFaceInstances;
@@ -2475,498 +2495,498 @@ typedef PlanePointer* PlanesIterator;
 
 class VectorLightList : public LightList
 {
-       typedef std::vector<const RendererLight*> Lights;
-       Lights m_lights;
+typedef std::vector<const RendererLight*> Lights;
+Lights m_lights;
 public:
-       void addLight( const RendererLight& light ){
-               m_lights.push_back( &light );
-       }
+void addLight( const RendererLight& light ){
+       m_lights.push_back( &light );
+}
 
-       void clear(){
-               m_lights.clear();
-       }
+void clear(){
+       m_lights.clear();
+}
 
-       void evaluateLights() const {
-       }
+void evaluateLights() const {
+}
 
-       void lightsChanged() const {
-       }
+void lightsChanged() const {
+}
 
-       void forEachLight( const RendererLightCallback& callback ) const {
-               for ( Lights::const_iterator i = m_lights.begin(); i != m_lights.end(); ++i )
-               {
-                       callback( *( *i ) );
-               }
+void forEachLight( const RendererLightCallback& callback ) const {
+       for ( Lights::const_iterator i = m_lights.begin(); i != m_lights.end(); ++i )
+       {
+               callback( *( *i ) );
        }
+}
 };
 
 class FaceInstance
 {
-       Face* m_face;
-       ObservedSelectable m_selectable;
-       ObservedSelectable m_selectableVertices;
-       ObservedSelectable m_selectableEdges;
-       SelectionChangeCallback m_selectionChanged;
+Face* m_face;
+ObservedSelectable m_selectable;
+ObservedSelectable m_selectableVertices;
+ObservedSelectable m_selectableEdges;
+SelectionChangeCallback m_selectionChanged;
 
-       VertexSelection m_vertexSelection;
-       VertexSelection m_edgeSelection;
+VertexSelection m_vertexSelection;
+VertexSelection m_edgeSelection;
 
 public:
-       mutable VectorLightList m_lights;
+mutable VectorLightList m_lights;
 
-       FaceInstance( Face& face, const SelectionChangeCallback& observer ) :
-               m_face( &face ),
-               m_selectable( SelectedChangedCaller( *this ) ),
-               m_selectableVertices( observer ),
-               m_selectableEdges( observer ),
-               m_selectionChanged( observer ){
-       }
+FaceInstance( Face& face, const SelectionChangeCallback& observer ) :
+       m_face( &face ),
+       m_selectable( SelectedChangedCaller( *this ) ),
+       m_selectableVertices( observer ),
+       m_selectableEdges( observer ),
+       m_selectionChanged( observer ){
+}
 
-       FaceInstance( const FaceInstance& other ) :
-               m_face( other.m_face ),
-               m_selectable( SelectedChangedCaller( *this ) ),
-               m_selectableVertices( other.m_selectableVertices ),
-               m_selectableEdges( other.m_selectableEdges ),
-               m_selectionChanged( other.m_selectionChanged ){
-       }
+FaceInstance( const FaceInstance& other ) :
+       m_face( other.m_face ),
+       m_selectable( SelectedChangedCaller( *this ) ),
+       m_selectableVertices( other.m_selectableVertices ),
+       m_selectableEdges( other.m_selectableEdges ),
+       m_selectionChanged( other.m_selectionChanged ){
+}
 
-       FaceInstance& operator=( const FaceInstance& other ){
-               m_face = other.m_face;
-               return *this;
-       }
+FaceInstance& operator=( const FaceInstance& other ){
+       m_face = other.m_face;
+       return *this;
+}
 
-       Face& getFace(){
-               return *m_face;
-       }
+Face& getFace(){
+       return *m_face;
+}
 
-       const Face& getFace() const {
-               return *m_face;
-       }
+const Face& getFace() const {
+       return *m_face;
+}
 
-       void selectedChanged( const Selectable& selectable ){
-               if ( selectable.isSelected() ) {
-                       g_SelectedFaceInstances.insert( *this );
-               }
-               else
-               {
-                       g_SelectedFaceInstances.erase( *this );
-               }
-               m_selectionChanged( selectable );
+void selectedChanged( const Selectable& selectable ){
+       if ( selectable.isSelected() ) {
+               g_SelectedFaceInstances.insert( *this );
        }
+       else
+       {
+               g_SelectedFaceInstances.erase( *this );
+       }
+       m_selectionChanged( selectable );
+}
 
-       typedef MemberCaller<FaceInstance, void(const Selectable&), &FaceInstance::selectedChanged> SelectedChangedCaller;
+typedef MemberCaller<FaceInstance, void(const Selectable&), &FaceInstance::selectedChanged> SelectedChangedCaller;
 
-       bool selectedVertices() const {
-               return !m_vertexSelection.empty();
-       }
+bool selectedVertices() const {
+       return !m_vertexSelection.empty();
+}
 
-       bool selectedEdges() const {
-               return !m_edgeSelection.empty();
-       }
+bool selectedEdges() const {
+       return !m_edgeSelection.empty();
+}
 
-       bool isSelected() const {
-               return m_selectable.isSelected();
-       }
+bool isSelected() const {
+       return m_selectable.isSelected();
+}
 
-       bool selectedComponents() const {
-               return selectedVertices() || selectedEdges() || isSelected();
-       }
+bool selectedComponents() const {
+       return selectedVertices() || selectedEdges() || isSelected();
+}
 
-       bool selectedComponents( SelectionSystem::EComponentMode mode ) const {
-               switch ( mode )
-               {
-               case SelectionSystem::eVertex:
-                       return selectedVertices();
-               case SelectionSystem::eEdge:
-                       return selectedEdges();
-               case SelectionSystem::eFace:
-                       return isSelected();
-               default:
-                       return false;
-               }
+bool selectedComponents( SelectionSystem::EComponentMode mode ) const {
+       switch ( mode )
+       {
+       case SelectionSystem::eVertex:
+               return selectedVertices();
+       case SelectionSystem::eEdge:
+               return selectedEdges();
+       case SelectionSystem::eFace:
+               return isSelected();
+       default:
+               return false;
        }
+}
 
-       void setSelected( SelectionSystem::EComponentMode mode, bool select ){
-               switch ( mode )
-               {
-               case SelectionSystem::eFace:
-                       m_selectable.setSelected( select );
-                       break;
-               case SelectionSystem::eVertex:
-                       ASSERT_MESSAGE( !select, "select-all not supported" );
-
-                       m_vertexSelection.clear();
-                       m_selectableVertices.setSelected( false );
-                       break;
-               case SelectionSystem::eEdge:
-                       ASSERT_MESSAGE( !select, "select-all not supported" );
-
-                       m_edgeSelection.clear();
-                       m_selectableEdges.setSelected( false );
-                       break;
-               default:
-                       break;
-               }
-       }
+void setSelected( SelectionSystem::EComponentMode mode, bool select ){
+       switch ( mode )
+       {
+       case SelectionSystem::eFace:
+               m_selectable.setSelected( select );
+               break;
+       case SelectionSystem::eVertex:
+               ASSERT_MESSAGE( !select, "select-all not supported" );
 
-       template<typename Functor>
-       void SelectedVertices_foreach( Functor functor ) const {
-               for ( VertexSelection::const_iterator i = m_vertexSelection.begin(); i != m_vertexSelection.end(); ++i )
-               {
-                       std::size_t index = Winding_FindAdjacent( getFace().getWinding(), *i );
-                       if ( index != c_brush_maxFaces ) {
-                               functor( getFace().getWinding()[index].vertex );
-                       }
-               }
+               m_vertexSelection.clear();
+               m_selectableVertices.setSelected( false );
+               break;
+       case SelectionSystem::eEdge:
+               ASSERT_MESSAGE( !select, "select-all not supported" );
+
+               m_edgeSelection.clear();
+               m_selectableEdges.setSelected( false );
+               break;
+       default:
+               break;
        }
+}
 
-       template<typename Functor>
-       void SelectedEdges_foreach( Functor functor ) const {
-               for ( VertexSelection::const_iterator i = m_edgeSelection.begin(); i != m_edgeSelection.end(); ++i )
-               {
-                       std::size_t index = Winding_FindAdjacent( getFace().getWinding(), *i );
-                       if ( index != c_brush_maxFaces ) {
-                               const Winding& winding = getFace().getWinding();
-                               std::size_t adjacent = Winding_next( winding, index );
-                               functor( vector3_mid( winding[index].vertex, winding[adjacent].vertex ) );
-                       }
+template<typename Functor>
+void SelectedVertices_foreach( Functor functor ) const {
+       for ( VertexSelection::const_iterator i = m_vertexSelection.begin(); i != m_vertexSelection.end(); ++i )
+       {
+               std::size_t index = Winding_FindAdjacent( getFace().getWinding(), *i );
+               if ( index != c_brush_maxFaces ) {
+                       functor( getFace().getWinding()[index].vertex );
                }
        }
+}
 
-       template<typename Functor>
-       void SelectedFaces_foreach( Functor functor ) const {
-               if ( isSelected() ) {
-                       functor( centroid() );
+template<typename Functor>
+void SelectedEdges_foreach( Functor functor ) const {
+       for ( VertexSelection::const_iterator i = m_edgeSelection.begin(); i != m_edgeSelection.end(); ++i )
+       {
+               std::size_t index = Winding_FindAdjacent( getFace().getWinding(), *i );
+               if ( index != c_brush_maxFaces ) {
+                       const Winding& winding = getFace().getWinding();
+                       std::size_t adjacent = Winding_next( winding, index );
+                       functor( vector3_mid( winding[index].vertex, winding[adjacent].vertex ) );
                }
        }
+}
 
-       template<typename Functor>
-       void SelectedComponents_foreach( Functor functor ) const {
-               SelectedVertices_foreach( functor );
-               SelectedEdges_foreach( functor );
-               SelectedFaces_foreach( functor );
+template<typename Functor>
+void SelectedFaces_foreach( Functor functor ) const {
+       if ( isSelected() ) {
+               functor( centroid() );
        }
+}
 
-       void iterate_selected( AABB& aabb ) const {
-               SelectedComponents_foreach([&](const Vector3 &point) {
-                       aabb_extend_by_point_safe(aabb, point);
-               });
-       }
+template<typename Functor>
+void SelectedComponents_foreach( Functor functor ) const {
+       SelectedVertices_foreach( functor );
+       SelectedEdges_foreach( functor );
+       SelectedFaces_foreach( functor );
+}
 
-       void iterate_selected( RenderablePointVector& points ) const {
-               SelectedComponents_foreach([&](const Vector3 &point) {
-                       const Colour4b colour_selected(0, 0, 255, 255);
-                       points.push_back(pointvertex_for_windingpoint(point, colour_selected));
-               });
-       }
+void iterate_selected( AABB& aabb ) const {
+       SelectedComponents_foreach([&](const Vector3 &point) {
+               aabb_extend_by_point_safe(aabb, point);
+       });
+}
 
-       bool intersectVolume( const VolumeTest& volume, const Matrix4& localToWorld ) const {
-               return m_face->intersectVolume( volume, localToWorld );
-       }
+void iterate_selected( RenderablePointVector& points ) const {
+       SelectedComponents_foreach([&](const Vector3 &point) {
+               const Colour4b colour_selected(0, 0, 255, 255);
+               points.push_back(pointvertex_for_windingpoint(point, colour_selected));
+       });
+}
 
-       void render( Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
-               if ( !m_face->isFiltered() && m_face->contributes() && intersectVolume( volume, localToWorld ) ) {
-                       renderer.PushState();
-                       if ( selectedComponents() ) {
-                               renderer.Highlight( Renderer::eFace );
-                       }
-                       m_face->render( renderer, localToWorld );
-                       renderer.PopState();
+bool intersectVolume( const VolumeTest& volume, const Matrix4& localToWorld ) const {
+       return m_face->intersectVolume( volume, localToWorld );
+}
+
+void render( Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
+       if ( !m_face->isFiltered() && m_face->contributes() && intersectVolume( volume, localToWorld ) ) {
+               renderer.PushState();
+               if ( selectedComponents() ) {
+                       renderer.Highlight( Renderer::eFace );
                }
+               m_face->render( renderer, localToWorld );
+               renderer.PopState();
+       }
+}
+
+void testSelect( SelectionTest& test, SelectionIntersection& best ){
+       if ( !m_face->isFiltered() ) {
+               m_face->testSelect( test, best );
        }
+}
 
-       void testSelect( SelectionTest& test, SelectionIntersection& best ){
-               if ( !m_face->isFiltered() ) {
-                       m_face->testSelect( test, best );
-               }
+void testSelect( Selector& selector, SelectionTest& test ){
+       SelectionIntersection best;
+       testSelect( test, best );
+       if ( best.valid() ) {
+               Selector_add( selector, m_selectable, best );
        }
+}
 
-       void testSelect( Selector& selector, SelectionTest& test ){
+void testSelect_centroid( Selector& selector, SelectionTest& test ){
+       if ( m_face->contributes() && !m_face->isFiltered() ) {
                SelectionIntersection best;
-               testSelect( test, best );
+               m_face->testSelect_centroid( test, best );
                if ( best.valid() ) {
                        Selector_add( selector, m_selectable, best );
                }
        }
+}
 
-       void testSelect_centroid( Selector& selector, SelectionTest& test ){
-               if ( m_face->contributes() && !m_face->isFiltered() ) {
-                       SelectionIntersection best;
-                       m_face->testSelect_centroid( test, best );
-                       if ( best.valid() ) {
-                               Selector_add( selector, m_selectable, best );
-                       }
+void selectPlane( Selector& selector, const Line& line, PlanesIterator first, PlanesIterator last, const PlaneCallback& selectedPlaneCallback ){
+       for ( Winding::const_iterator i = getFace().getWinding().begin(); i != getFace().getWinding().end(); ++i )
+       {
+               Vector3 v( vector3_subtracted( line_closest_point( line, ( *i ).vertex ), ( *i ).vertex ) );
+               double dot = vector3_dot( getFace().plane3().normal(), v );
+               if ( dot <= 0 ) {
+                       return;
                }
        }
 
-       void selectPlane( Selector& selector, const Line& line, PlanesIterator first, PlanesIterator last, const PlaneCallback& selectedPlaneCallback ){
-               for ( Winding::const_iterator i = getFace().getWinding().begin(); i != getFace().getWinding().end(); ++i )
-               {
-                       Vector3 v( vector3_subtracted( line_closest_point( line, ( *i ).vertex ), ( *i ).vertex ) );
-                       double dot = vector3_dot( getFace().plane3().normal(), v );
-                       if ( dot <= 0 ) {
-                               return;
-                       }
-               }
+       Selector_add( selector, m_selectable );
 
-               Selector_add( selector, m_selectable );
+       selectedPlaneCallback( getFace().plane3() );
+}
 
-               selectedPlaneCallback( getFace().plane3() );
+void selectReversedPlane( Selector& selector, const SelectedPlanes& selectedPlanes ){
+       if ( selectedPlanes.contains( plane3_flipped( getFace().plane3() ) ) ) {
+               Selector_add( selector, m_selectable );
        }
+}
 
-       void selectReversedPlane( Selector& selector, const SelectedPlanes& selectedPlanes ){
-               if ( selectedPlanes.contains( plane3_flipped( getFace().plane3() ) ) ) {
-                       Selector_add( selector, m_selectable );
-               }
+void transformComponents( const Matrix4& matrix ){
+       if ( isSelected() ) {
+               m_face->transform( matrix, false );
        }
-
-       void transformComponents( const Matrix4& matrix ){
-               if ( isSelected() ) {
-                       m_face->transform( matrix, false );
+       if ( selectedVertices() ) {
+               if ( m_vertexSelection.size() == 1 ) {
+                       matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[1] );
+                       m_face->assign_planepts( m_face->m_move_planeptsTransformed );
                }
-               if ( selectedVertices() ) {
-                       if ( m_vertexSelection.size() == 1 ) {
-                               matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[1] );
-                               m_face->assign_planepts( m_face->m_move_planeptsTransformed );
-                       }
-                       else if ( m_vertexSelection.size() == 2 ) {
-                               matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[1] );
-                               matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[2] );
-                               m_face->assign_planepts( m_face->m_move_planeptsTransformed );
-                       }
-                       else if ( m_vertexSelection.size() >= 3 ) {
-                               matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[0] );
-                               matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[1] );
-                               matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[2] );
-                               m_face->assign_planepts( m_face->m_move_planeptsTransformed );
-                       }
+               else if ( m_vertexSelection.size() == 2 ) {
+                       matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[1] );
+                       matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[2] );
+                       m_face->assign_planepts( m_face->m_move_planeptsTransformed );
                }
-               if ( selectedEdges() ) {
-                       if ( m_edgeSelection.size() == 1 ) {
-                               matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[0] );
-                               matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[1] );
-                               m_face->assign_planepts( m_face->m_move_planeptsTransformed );
-                       }
-                       else if ( m_edgeSelection.size() >= 2 ) {
-                               matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[0] );
-                               matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[1] );
-                               matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[2] );
-                               m_face->assign_planepts( m_face->m_move_planeptsTransformed );
-                       }
+               else if ( m_vertexSelection.size() >= 3 ) {
+                       matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[0] );
+                       matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[1] );
+                       matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[2] );
+                       m_face->assign_planepts( m_face->m_move_planeptsTransformed );
                }
        }
-
-       void snapto( float snap ){
-               m_face->snapto( snap );
-       }
-
-       void snapComponents( float snap ){
-               if ( isSelected() ) {
-                       snapto( snap );
+       if ( selectedEdges() ) {
+               if ( m_edgeSelection.size() == 1 ) {
+                       matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[0] );
+                       matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[1] );
+                       m_face->assign_planepts( m_face->m_move_planeptsTransformed );
                }
-               if ( selectedVertices() ) {
-                       vector3_snap( m_face->m_move_planepts[0], snap );
-                       vector3_snap( m_face->m_move_planepts[1], snap );
-                       vector3_snap( m_face->m_move_planepts[2], snap );
-                       m_face->assign_planepts( m_face->m_move_planepts );
-                       planepts_assign( m_face->m_move_planeptsTransformed, m_face->m_move_planepts );
-                       m_face->freezeTransform();
-               }
-               if ( selectedEdges() ) {
-                       vector3_snap( m_face->m_move_planepts[0], snap );
-                       vector3_snap( m_face->m_move_planepts[1], snap );
-                       vector3_snap( m_face->m_move_planepts[2], snap );
-                       m_face->assign_planepts( m_face->m_move_planepts );
-                       planepts_assign( m_face->m_move_planeptsTransformed, m_face->m_move_planepts );
-                       m_face->freezeTransform();
+               else if ( m_edgeSelection.size() >= 2 ) {
+                       matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[0] );
+                       matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[1] );
+                       matrix4_transform_point( matrix, m_face->m_move_planeptsTransformed[2] );
+                       m_face->assign_planepts( m_face->m_move_planeptsTransformed );
                }
        }
+}
 
-       void update_move_planepts_vertex( std::size_t index ){
-               m_face->update_move_planepts_vertex( index, m_face->m_move_planepts );
+void snapto( float snap ){
+       m_face->snapto( snap );
+}
+
+void snapComponents( float snap ){
+       if ( isSelected() ) {
+               snapto( snap );
+       }
+       if ( selectedVertices() ) {
+               vector3_snap( m_face->m_move_planepts[0], snap );
+               vector3_snap( m_face->m_move_planepts[1], snap );
+               vector3_snap( m_face->m_move_planepts[2], snap );
+               m_face->assign_planepts( m_face->m_move_planepts );
+               planepts_assign( m_face->m_move_planeptsTransformed, m_face->m_move_planepts );
+               m_face->freezeTransform();
        }
+       if ( selectedEdges() ) {
+               vector3_snap( m_face->m_move_planepts[0], snap );
+               vector3_snap( m_face->m_move_planepts[1], snap );
+               vector3_snap( m_face->m_move_planepts[2], snap );
+               m_face->assign_planepts( m_face->m_move_planepts );
+               planepts_assign( m_face->m_move_planeptsTransformed, m_face->m_move_planepts );
+               m_face->freezeTransform();
+       }
+}
 
-       void update_move_planepts_vertex2( std::size_t index, std::size_t other ){
-               const std::size_t numpoints = m_face->getWinding().numpoints;
-               ASSERT_MESSAGE( index < numpoints, "select_vertex: invalid index" );
+void update_move_planepts_vertex( std::size_t index ){
+       m_face->update_move_planepts_vertex( index, m_face->m_move_planepts );
+}
 
-               const std::size_t opposite = Winding_Opposite( m_face->getWinding(), index, other );
+void update_move_planepts_vertex2( std::size_t index, std::size_t other ){
+       const std::size_t numpoints = m_face->getWinding().numpoints;
+       ASSERT_MESSAGE( index < numpoints, "select_vertex: invalid index" );
 
-               if ( triangle_reversed( index, other, opposite ) ) {
-                       std::swap( index, other );
-               }
+       const std::size_t opposite = Winding_Opposite( m_face->getWinding(), index, other );
 
-               ASSERT_MESSAGE(
-                       triangles_same_winding(
-                               m_face->getWinding()[opposite].vertex,
-                               m_face->getWinding()[index].vertex,
-                               m_face->getWinding()[other].vertex,
-                               m_face->getWinding()[0].vertex,
-                               m_face->getWinding()[1].vertex,
-                               m_face->getWinding()[2].vertex
-                               ),
-                       "update_move_planepts_vertex2: error"
-                       );
-
-               m_face->m_move_planepts[0] = m_face->getWinding()[opposite].vertex;
-               m_face->m_move_planepts[1] = m_face->getWinding()[index].vertex;
-               m_face->m_move_planepts[2] = m_face->getWinding()[other].vertex;
-               planepts_quantise( m_face->m_move_planepts, GRID_MIN ); // winding points are very inaccurate
-       }
-
-       void update_selection_vertex(){
-               if ( m_vertexSelection.size() == 0 ) {
-                       m_selectableVertices.setSelected( false );
-               }
-               else
-               {
-                       m_selectableVertices.setSelected( true );
+       if ( triangle_reversed( index, other, opposite ) ) {
+               std::swap( index, other );
+       }
 
-                       if ( m_vertexSelection.size() == 1 ) {
-                               std::size_t index = Winding_FindAdjacent( getFace().getWinding(), *m_vertexSelection.begin() );
+       ASSERT_MESSAGE(
+               triangles_same_winding(
+                       m_face->getWinding()[opposite].vertex,
+                       m_face->getWinding()[index].vertex,
+                       m_face->getWinding()[other].vertex,
+                       m_face->getWinding()[0].vertex,
+                       m_face->getWinding()[1].vertex,
+                       m_face->getWinding()[2].vertex
+                       ),
+               "update_move_planepts_vertex2: error"
+               );
 
-                               if ( index != c_brush_maxFaces ) {
-                                       update_move_planepts_vertex( index );
-                               }
-                       }
-                       else if ( m_vertexSelection.size() == 2 ) {
-                               std::size_t index = Winding_FindAdjacent( getFace().getWinding(), *m_vertexSelection.begin() );
-                               std::size_t other = Winding_FindAdjacent( getFace().getWinding(), *( ++m_vertexSelection.begin() ) );
+       m_face->m_move_planepts[0] = m_face->getWinding()[opposite].vertex;
+       m_face->m_move_planepts[1] = m_face->getWinding()[index].vertex;
+       m_face->m_move_planepts[2] = m_face->getWinding()[other].vertex;
+       planepts_quantise( m_face->m_move_planepts, GRID_MIN ); // winding points are very inaccurate
+}
 
-                               if ( index != c_brush_maxFaces
-                                        && other != c_brush_maxFaces ) {
-                                       update_move_planepts_vertex2( index, other );
-                               }
-                       }
-               }
+void update_selection_vertex(){
+       if ( m_vertexSelection.size() == 0 ) {
+               m_selectableVertices.setSelected( false );
        }
+       else
+       {
+               m_selectableVertices.setSelected( true );
 
-       void select_vertex( std::size_t index, bool select ){
-               if ( select ) {
-                       VertexSelection_insert( m_vertexSelection, getFace().getWinding()[index].adjacent );
-               }
-               else
-               {
-                       VertexSelection_erase( m_vertexSelection, getFace().getWinding()[index].adjacent );
+               if ( m_vertexSelection.size() == 1 ) {
+                       std::size_t index = Winding_FindAdjacent( getFace().getWinding(), *m_vertexSelection.begin() );
+
+                       if ( index != c_brush_maxFaces ) {
+                               update_move_planepts_vertex( index );
+                       }
                }
+               else if ( m_vertexSelection.size() == 2 ) {
+                       std::size_t index = Winding_FindAdjacent( getFace().getWinding(), *m_vertexSelection.begin() );
+                       std::size_t other = Winding_FindAdjacent( getFace().getWinding(), *( ++m_vertexSelection.begin() ) );
 
-               SceneChangeNotify();
-               update_selection_vertex();
+                       if ( index != c_brush_maxFaces
+                                && other != c_brush_maxFaces ) {
+                               update_move_planepts_vertex2( index, other );
+                       }
+               }
        }
+}
 
-       bool selected_vertex( std::size_t index ) const {
-               return VertexSelection_find( m_vertexSelection, getFace().getWinding()[index].adjacent ) != m_vertexSelection.end();
+void select_vertex( std::size_t index, bool select ){
+       if ( select ) {
+               VertexSelection_insert( m_vertexSelection, getFace().getWinding()[index].adjacent );
+       }
+       else
+       {
+               VertexSelection_erase( m_vertexSelection, getFace().getWinding()[index].adjacent );
        }
 
-       void update_move_planepts_edge( std::size_t index ){
-               std::size_t numpoints = m_face->getWinding().numpoints;
-               ASSERT_MESSAGE( index < numpoints, "select_edge: invalid index" );
+       SceneChangeNotify();
+       update_selection_vertex();
+}
 
-               std::size_t adjacent = Winding_next( m_face->getWinding(), index );
-               std::size_t opposite = Winding_Opposite( m_face->getWinding(), index );
-               m_face->m_move_planepts[0] = m_face->getWinding()[index].vertex;
-               m_face->m_move_planepts[1] = m_face->getWinding()[adjacent].vertex;
-               m_face->m_move_planepts[2] = m_face->getWinding()[opposite].vertex;
-               planepts_quantise( m_face->m_move_planepts, GRID_MIN ); // winding points are very inaccurate
-       }
+bool selected_vertex( std::size_t index ) const {
+       return VertexSelection_find( m_vertexSelection, getFace().getWinding()[index].adjacent ) != m_vertexSelection.end();
+}
 
-       void update_selection_edge(){
-               if ( m_edgeSelection.size() == 0 ) {
-                       m_selectableEdges.setSelected( false );
-               }
-               else
-               {
-                       m_selectableEdges.setSelected( true );
+void update_move_planepts_edge( std::size_t index ){
+       std::size_t numpoints = m_face->getWinding().numpoints;
+       ASSERT_MESSAGE( index < numpoints, "select_edge: invalid index" );
 
-                       if ( m_edgeSelection.size() == 1 ) {
-                               std::size_t index = Winding_FindAdjacent( getFace().getWinding(), *m_edgeSelection.begin() );
+       std::size_t adjacent = Winding_next( m_face->getWinding(), index );
+       std::size_t opposite = Winding_Opposite( m_face->getWinding(), index );
+       m_face->m_move_planepts[0] = m_face->getWinding()[index].vertex;
+       m_face->m_move_planepts[1] = m_face->getWinding()[adjacent].vertex;
+       m_face->m_move_planepts[2] = m_face->getWinding()[opposite].vertex;
+       planepts_quantise( m_face->m_move_planepts, GRID_MIN ); // winding points are very inaccurate
+}
 
-                               if ( index != c_brush_maxFaces ) {
-                                       update_move_planepts_edge( index );
-                               }
-                       }
-               }
+void update_selection_edge(){
+       if ( m_edgeSelection.size() == 0 ) {
+               m_selectableEdges.setSelected( false );
        }
+       else
+       {
+               m_selectableEdges.setSelected( true );
 
-       void select_edge( std::size_t index, bool select ){
-               if ( select ) {
-                       VertexSelection_insert( m_edgeSelection, getFace().getWinding()[index].adjacent );
-               }
-               else
-               {
-                       VertexSelection_erase( m_edgeSelection, getFace().getWinding()[index].adjacent );
-               }
+               if ( m_edgeSelection.size() == 1 ) {
+                       std::size_t index = Winding_FindAdjacent( getFace().getWinding(), *m_edgeSelection.begin() );
 
-               SceneChangeNotify();
-               update_selection_edge();
+                       if ( index != c_brush_maxFaces ) {
+                               update_move_planepts_edge( index );
+                       }
+               }
        }
+}
 
-       bool selected_edge( std::size_t index ) const {
-               return VertexSelection_find( m_edgeSelection, getFace().getWinding()[index].adjacent ) != m_edgeSelection.end();
+void select_edge( std::size_t index, bool select ){
+       if ( select ) {
+               VertexSelection_insert( m_edgeSelection, getFace().getWinding()[index].adjacent );
        }
-
-       const Vector3& centroid() const {
-               return m_face->centroid();
+       else
+       {
+               VertexSelection_erase( m_edgeSelection, getFace().getWinding()[index].adjacent );
        }
 
-       void connectivityChanged(){
-               // This occurs when a face is added or removed.
-               // The current vertex and edge selections no longer valid and must be cleared.
-               m_vertexSelection.clear();
-               m_selectableVertices.setSelected( false );
-               m_edgeSelection.clear();
-               m_selectableEdges.setSelected( false );
-       }
+       SceneChangeNotify();
+       update_selection_edge();
+}
+
+bool selected_edge( std::size_t index ) const {
+       return VertexSelection_find( m_edgeSelection, getFace().getWinding()[index].adjacent ) != m_edgeSelection.end();
+}
+
+const Vector3& centroid() const {
+       return m_face->centroid();
+}
+
+void connectivityChanged(){
+       // This occurs when a face is added or removed.
+       // The current vertex and edge selections no longer valid and must be cleared.
+       m_vertexSelection.clear();
+       m_selectableVertices.setSelected( false );
+       m_edgeSelection.clear();
+       m_selectableEdges.setSelected( false );
+}
 };
 
 class BrushClipPlane : public OpenGLRenderable
 {
-       Plane3 m_plane;
-       Winding m_winding;
-       static Shader* m_state;
+Plane3 m_plane;
+Winding m_winding;
+static Shader* m_state;
 public:
-       static void constructStatic(){
-               m_state = GlobalShaderCache().capture( "$CLIPPER_OVERLAY" );
-       }
+static void constructStatic(){
+       m_state = GlobalShaderCache().capture( "$CLIPPER_OVERLAY" );
+}
 
-       static void destroyStatic(){
-               GlobalShaderCache().release( "$CLIPPER_OVERLAY" );
-       }
+static void destroyStatic(){
+       GlobalShaderCache().release( "$CLIPPER_OVERLAY" );
+}
 
-       void setPlane( const Brush& brush, const Plane3& plane ){
-               m_plane = plane;
-               if ( plane3_valid( m_plane ) ) {
-                       brush.windingForClipPlane( m_winding, m_plane );
-               }
-               else
-               {
-                       m_winding.resize( 0 );
-               }
+void setPlane( const Brush& brush, const Plane3& plane ){
+       m_plane = plane;
+       if ( plane3_valid( m_plane ) ) {
+               brush.windingForClipPlane( m_winding, m_plane );
+       }
+       else
+       {
+               m_winding.resize( 0 );
        }
+}
 
-       void render( RenderStateFlags state ) const {
-               if ( ( state & RENDER_FILL ) != 0 ) {
-                       Winding_Draw( m_winding, m_plane.normal(), state );
-               }
-               else
-               {
-                       Winding_DrawWireframe( m_winding );
+void render( RenderStateFlags state ) const {
+       if ( ( state & RENDER_FILL ) != 0 ) {
+               Winding_Draw( m_winding, m_plane.normal(), state );
+       }
+       else
+       {
+               Winding_DrawWireframe( m_winding );
 
-                       // also draw a line indicating the direction of the cut
-                       Vector3 lineverts[2];
-                       Winding_Centroid( m_winding, m_plane, lineverts[0] );
-                       lineverts[1] = vector3_added( lineverts[0], vector3_scaled( m_plane.normal(), Brush::m_maxWorldCoord * 4 ) );
+               // also draw a line indicating the direction of the cut
+               Vector3 lineverts[2];
+               Winding_Centroid( m_winding, m_plane, lineverts[0] );
+               lineverts[1] = vector3_added( lineverts[0], vector3_scaled( m_plane.normal(), Brush::m_maxWorldCoord * 4 ) );
 
-                       glVertexPointer( 3, GL_FLOAT, sizeof( Vector3 ), &lineverts[0] );
-                       glDrawArrays( GL_LINES, 0, GLsizei( 2 ) );
-               }
+               glVertexPointer( 3, GL_FLOAT, sizeof( Vector3 ), &lineverts[0] );
+               glDrawArrays( GL_LINES, 0, GLsizei( 2 ) );
        }
+}
 
-       void render( Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
-               renderer.SetState( m_state, Renderer::eWireframeOnly );
-               renderer.SetState( m_state, Renderer::eFullMaterials );
-               renderer.addRenderable( *this, localToWorld );
-       }
+void render( Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
+       renderer.SetState( m_state, Renderer::eWireframeOnly );
+       renderer.SetState( m_state, Renderer::eFullMaterials );
+       renderer.addRenderable( *this, localToWorld );
+}
 };
 
 inline void Face_addLight( const FaceInstance& face, const Matrix4& localToWorld, const RendererLight& light ){
@@ -2984,114 +3004,114 @@ typedef std::vector<FaceInstance> FaceInstances;
 
 class EdgeInstance : public Selectable
 {
-       FaceInstances& m_faceInstances;
-       SelectableEdge* m_edge;
-
-       void select_edge( bool select ){
-               FaceVertexId faceVertex = m_edge->m_faceVertex;
-               m_faceInstances[faceVertex.getFace()].select_edge( faceVertex.getVertex(), select );
-               faceVertex = next_edge( m_edge->m_faces, faceVertex );
-               m_faceInstances[faceVertex.getFace()].select_edge( faceVertex.getVertex(), select );
-       }
+FaceInstances& m_faceInstances;
+SelectableEdge* m_edge;
 
-       bool selected_edge() const {
-               FaceVertexId faceVertex = m_edge->m_faceVertex;
-               if ( !m_faceInstances[faceVertex.getFace()].selected_edge( faceVertex.getVertex() ) ) {
-                       return false;
-               }
-               faceVertex = next_edge( m_edge->m_faces, faceVertex );
-               if ( !m_faceInstances[faceVertex.getFace()].selected_edge( faceVertex.getVertex() ) ) {
-                       return false;
-               }
+void select_edge( bool select ){
+       FaceVertexId faceVertex = m_edge->m_faceVertex;
+       m_faceInstances[faceVertex.getFace()].select_edge( faceVertex.getVertex(), select );
+       faceVertex = next_edge( m_edge->m_faces, faceVertex );
+       m_faceInstances[faceVertex.getFace()].select_edge( faceVertex.getVertex(), select );
+}
 
-               return true;
+bool selected_edge() const {
+       FaceVertexId faceVertex = m_edge->m_faceVertex;
+       if ( !m_faceInstances[faceVertex.getFace()].selected_edge( faceVertex.getVertex() ) ) {
+               return false;
+       }
+       faceVertex = next_edge( m_edge->m_faces, faceVertex );
+       if ( !m_faceInstances[faceVertex.getFace()].selected_edge( faceVertex.getVertex() ) ) {
+               return false;
        }
 
+       return true;
+}
+
 public:
-       EdgeInstance( FaceInstances& faceInstances, SelectableEdge& edge )
-               : m_faceInstances( faceInstances ), m_edge( &edge ){
-       }
-       EdgeInstance& operator=( const EdgeInstance& other ){
-               m_edge = other.m_edge;
-               return *this;
-       }
+EdgeInstance( FaceInstances& faceInstances, SelectableEdge& edge )
+       : m_faceInstances( faceInstances ), m_edge( &edge ){
+}
+EdgeInstance& operator=( const EdgeInstance& other ){
+       m_edge = other.m_edge;
+       return *this;
+}
 
-       void setSelected( bool select ){
-               select_edge( select );
-       }
+void setSelected( bool select ){
+       select_edge( select );
+}
 
-       bool isSelected() const {
-               return selected_edge();
-       }
+bool isSelected() const {
+       return selected_edge();
+}
 
 
-       void testSelect( Selector& selector, SelectionTest& test ){
-               SelectionIntersection best;
-               m_edge->testSelect( test, best );
-               if ( best.valid() ) {
-                       Selector_add( selector, *this, best );
-               }
+void testSelect( Selector& selector, SelectionTest& test ){
+       SelectionIntersection best;
+       m_edge->testSelect( test, best );
+       if ( best.valid() ) {
+               Selector_add( selector, *this, best );
        }
+}
 };
 
 class VertexInstance : public Selectable
 {
-       FaceInstances& m_faceInstances;
-       SelectableVertex* m_vertex;
+FaceInstances& m_faceInstances;
+SelectableVertex* m_vertex;
 
-       void select_vertex( bool select ){
-               FaceVertexId faceVertex = m_vertex->m_faceVertex;
-               do
-               {
-                       m_faceInstances[faceVertex.getFace()].select_vertex( faceVertex.getVertex(), select );
-                       faceVertex = next_vertex( m_vertex->m_faces, faceVertex );
-               }
-               while ( faceVertex.getFace() != m_vertex->m_faceVertex.getFace() );
+void select_vertex( bool select ){
+       FaceVertexId faceVertex = m_vertex->m_faceVertex;
+       do
+       {
+               m_faceInstances[faceVertex.getFace()].select_vertex( faceVertex.getVertex(), select );
+               faceVertex = next_vertex( m_vertex->m_faces, faceVertex );
        }
+       while ( faceVertex.getFace() != m_vertex->m_faceVertex.getFace() );
+}
 
-       bool selected_vertex() const {
-               FaceVertexId faceVertex = m_vertex->m_faceVertex;
-               do
-               {
-                       if ( !m_faceInstances[faceVertex.getFace()].selected_vertex( faceVertex.getVertex() ) ) {
-                               return false;
-                       }
-                       faceVertex = next_vertex( m_vertex->m_faces, faceVertex );
+bool selected_vertex() const {
+       FaceVertexId faceVertex = m_vertex->m_faceVertex;
+       do
+       {
+               if ( !m_faceInstances[faceVertex.getFace()].selected_vertex( faceVertex.getVertex() ) ) {
+                       return false;
                }
-               while ( faceVertex.getFace() != m_vertex->m_faceVertex.getFace() );
-               return true;
+               faceVertex = next_vertex( m_vertex->m_faces, faceVertex );
        }
+       while ( faceVertex.getFace() != m_vertex->m_faceVertex.getFace() );
+       return true;
+}
 
-       public:
-       VertexInstance( FaceInstances& faceInstances, SelectableVertex& vertex )
-               : m_faceInstances( faceInstances ), m_vertex( &vertex ){
-       }
-       VertexInstance& operator=( const VertexInstance& other ){
-               m_vertex = other.m_vertex;
-               return *this;
-       }
+public:
+VertexInstance( FaceInstances& faceInstances, SelectableVertex& vertex )
+       : m_faceInstances( faceInstances ), m_vertex( &vertex ){
+}
+VertexInstance& operator=( const VertexInstance& other ){
+       m_vertex = other.m_vertex;
+       return *this;
+}
 
-       void setSelected( bool select ){
-               select_vertex( select );
-       }
+void setSelected( bool select ){
+       select_vertex( select );
+}
 
-       bool isSelected() const {
-               return selected_vertex();
-       }
+bool isSelected() const {
+       return selected_vertex();
+}
 
-       void testSelect( Selector& selector, SelectionTest& test ){
-               SelectionIntersection best;
-               m_vertex->testSelect( test, best );
-               if ( best.valid() ) {
-                       Selector_add( selector, *this, best );
-               }
+void testSelect( Selector& selector, SelectionTest& test ){
+       SelectionIntersection best;
+       m_vertex->testSelect( test, best );
+       if ( best.valid() ) {
+               Selector_add( selector, *this, best );
        }
+}
 };
 
 class BrushInstanceVisitor
 {
 public:
-       virtual void visit( FaceInstance& face ) const = 0;
+virtual void visit( FaceInstance& face ) const = 0;
 };
 
 class BrushInstance :
@@ -3106,498 +3126,498 @@ class BrushInstance :
        public PlaneSelectable,
        public LightCullable
 {
-       class TypeCasts
-       {
-               InstanceTypeCastTable m_casts;
-       public:
-               TypeCasts(){
-                       InstanceStaticCast<BrushInstance, Selectable>::install( m_casts );
-                       InstanceContainedCast<BrushInstance, Bounded>::install( m_casts );
-                       InstanceContainedCast<BrushInstance, Cullable>::install( m_casts );
-                       InstanceStaticCast<BrushInstance, Renderable>::install( m_casts );
-                       InstanceStaticCast<BrushInstance, SelectionTestable>::install( m_casts );
-                       InstanceStaticCast<BrushInstance, ComponentSelectionTestable>::install( m_casts );
-                       InstanceStaticCast<BrushInstance, ComponentEditable>::install( m_casts );
-                       InstanceStaticCast<BrushInstance, ComponentSnappable>::install( m_casts );
-                       InstanceStaticCast<BrushInstance, PlaneSelectable>::install( m_casts );
-                       InstanceIdentityCast<BrushInstance>::install( m_casts );
-                       InstanceContainedCast<BrushInstance, Transformable>::install( m_casts );
-               }
-
-               InstanceTypeCastTable& get(){
-                       return m_casts;
-               }
-       };
+class TypeCasts
+{
+InstanceTypeCastTable m_casts;
+public:
+TypeCasts(){
+       InstanceStaticCast<BrushInstance, Selectable>::install( m_casts );
+       InstanceContainedCast<BrushInstance, Bounded>::install( m_casts );
+       InstanceContainedCast<BrushInstance, Cullable>::install( m_casts );
+       InstanceStaticCast<BrushInstance, Renderable>::install( m_casts );
+       InstanceStaticCast<BrushInstance, SelectionTestable>::install( m_casts );
+       InstanceStaticCast<BrushInstance, ComponentSelectionTestable>::install( m_casts );
+       InstanceStaticCast<BrushInstance, ComponentEditable>::install( m_casts );
+       InstanceStaticCast<BrushInstance, ComponentSnappable>::install( m_casts );
+       InstanceStaticCast<BrushInstance, PlaneSelectable>::install( m_casts );
+       InstanceIdentityCast<BrushInstance>::install( m_casts );
+       InstanceContainedCast<BrushInstance, Transformable>::install( m_casts );
+}
+
+InstanceTypeCastTable& get(){
+       return m_casts;
+}
+};
 
 
-       Brush& m_brush;
+Brush& m_brush;
 
-       FaceInstances m_faceInstances;
+FaceInstances m_faceInstances;
 
-       typedef std::vector<EdgeInstance> EdgeInstances;
-       EdgeInstances m_edgeInstances;
-       typedef std::vector<VertexInstance> VertexInstances;
-       VertexInstances m_vertexInstances;
+typedef std::vector<EdgeInstance> EdgeInstances;
+EdgeInstances m_edgeInstances;
+typedef std::vector<VertexInstance> VertexInstances;
+VertexInstances m_vertexInstances;
 
-       ObservedSelectable m_selectable;
+ObservedSelectable m_selectable;
 
-       mutable RenderableWireframe m_render_wireframe;
-       mutable RenderablePointVector m_render_selected;
-       mutable AABB m_aabb_component;
-       mutable Array<PointVertex> m_faceCentroidPointsCulled;
-       RenderablePointArray m_render_faces_wireframe;
-       mutable bool m_viewChanged;   // requires re-evaluation of view-dependent cached data
+mutable RenderableWireframe m_render_wireframe;
+mutable RenderablePointVector m_render_selected;
+mutable AABB m_aabb_component;
+mutable Array<PointVertex> m_faceCentroidPointsCulled;
+RenderablePointArray m_render_faces_wireframe;
+mutable bool m_viewChanged;   // requires re-evaluation of view-dependent cached data
 
-       BrushClipPlane m_clipPlane;
+BrushClipPlane m_clipPlane;
 
-       static Shader* m_state_selpoint;
+static Shader* m_state_selpoint;
 
-       const LightList* m_lightList;
+const LightList* m_lightList;
 
-       TransformModifier m_transform;
+TransformModifier m_transform;
 
-       BrushInstance( const BrushInstance& other ); // NOT COPYABLE
-       BrushInstance& operator=( const BrushInstance& other ); // NOT ASSIGNABLE
+BrushInstance( const BrushInstance& other ); // NOT COPYABLE
+BrushInstance& operator=( const BrushInstance& other ); // NOT ASSIGNABLE
 
 public:
-       static Counter* m_counter;
+static Counter* m_counter;
 
-       typedef LazyStatic<TypeCasts> StaticTypeCasts;
+typedef LazyStatic<TypeCasts> StaticTypeCasts;
 
-       void lightsChanged(){
-               m_lightList->lightsChanged();
-       }
+void lightsChanged(){
+       m_lightList->lightsChanged();
+}
 
-       typedef MemberCaller<BrushInstance, void(), &BrushInstance::lightsChanged> LightsChangedCaller;
+typedef MemberCaller<BrushInstance, void(), &BrushInstance::lightsChanged> LightsChangedCaller;
 
-       STRING_CONSTANT( Name, "BrushInstance" );
+STRING_CONSTANT( Name, "BrushInstance" );
 
-       BrushInstance( const scene::Path& path, scene::Instance* parent, Brush& brush ) :
-               Instance( path, parent, this, StaticTypeCasts::instance().get() ),
-               m_brush( brush ),
-               m_selectable( SelectedChangedCaller( *this ) ),
-               m_render_selected( GL_POINTS ),
-               m_render_faces_wireframe( m_faceCentroidPointsCulled, GL_POINTS ),
-               m_viewChanged( false ),
-               m_transform( Brush::TransformChangedCaller( m_brush ), ApplyTransformCaller( *this ) ){
-               m_brush.instanceAttach( Instance::path() );
-               m_brush.attach( *this );
-               m_counter->increment();
+BrushInstance( const scene::Path& path, scene::Instance* parent, Brush& brush ) :
+       Instance( path, parent, this, StaticTypeCasts::instance().get() ),
+       m_brush( brush ),
+       m_selectable( SelectedChangedCaller( *this ) ),
+       m_render_selected( GL_POINTS ),
+       m_render_faces_wireframe( m_faceCentroidPointsCulled, GL_POINTS ),
+       m_viewChanged( false ),
+       m_transform( Brush::TransformChangedCaller( m_brush ), ApplyTransformCaller( *this ) ){
+       m_brush.instanceAttach( Instance::path() );
+       m_brush.attach( *this );
+       m_counter->increment();
 
-               m_lightList = &GlobalShaderCache().attach( *this );
-               m_brush.m_lightsChanged = LightsChangedCaller( *this ); ///\todo Make this work with instancing.
+       m_lightList = &GlobalShaderCache().attach( *this );
+       m_brush.m_lightsChanged = LightsChangedCaller( *this ); ///\todo Make this work with instancing.
 
-               Instance::setTransformChangedCallback( LightsChangedCaller( *this ) );
-       }
+       Instance::setTransformChangedCallback( LightsChangedCaller( *this ) );
+}
 
-       ~BrushInstance(){
-               Instance::setTransformChangedCallback( Callback<void()>() );
+~BrushInstance(){
+       Instance::setTransformChangedCallback( Callback<void()>() );
 
-               m_brush.m_lightsChanged = Callback<void()>();
-               GlobalShaderCache().detach( *this );
+       m_brush.m_lightsChanged = Callback<void()>();
+       GlobalShaderCache().detach( *this );
 
-               m_counter->decrement();
-               m_brush.detach( *this );
-               m_brush.instanceDetach( Instance::path() );
-       }
+       m_counter->decrement();
+       m_brush.detach( *this );
+       m_brush.instanceDetach( Instance::path() );
+}
 
-       Brush& getBrush(){
-               return m_brush;
-       }
-       const Brush& getBrush() const {
-               return m_brush;
-       }
+Brush& getBrush(){
+       return m_brush;
+}
+const Brush& getBrush() const {
+       return m_brush;
+}
 
-       Bounded& get( NullType<Bounded>){
-               return m_brush;
-       }
+Bounded& get( NullType<Bounded>){
+       return m_brush;
+}
 
-       Cullable& get( NullType<Cullable>){
-               return m_brush;
-       }
+Cullable& get( NullType<Cullable>){
+       return m_brush;
+}
 
-       Transformable& get( NullType<Transformable>){
-               return m_transform;
-       }
+Transformable& get( NullType<Transformable>){
+       return m_transform;
+}
 
-       void selectedChanged( const Selectable& selectable ){
-               GlobalSelectionSystem().getObserver ( SelectionSystem::ePrimitive )( selectable );
-               GlobalSelectionSystem().onSelectedChanged( *this, selectable );
+void selectedChanged( const Selectable& selectable ){
+       GlobalSelectionSystem().getObserver ( SelectionSystem::ePrimitive )( selectable );
+       GlobalSelectionSystem().onSelectedChanged( *this, selectable );
 
-               Instance::selectedChanged();
-       }
-       typedef MemberCaller<BrushInstance, void(const Selectable&), &BrushInstance::selectedChanged> SelectedChangedCaller;
+       Instance::selectedChanged();
+}
+typedef MemberCaller<BrushInstance, void(const Selectable&), &BrushInstance::selectedChanged> SelectedChangedCaller;
 
-       void selectedChangedComponent( const Selectable& selectable ){
-               GlobalSelectionSystem().getObserver ( SelectionSystem::eComponent )( selectable );
-               GlobalSelectionSystem().onComponentSelection( *this, selectable );
-       }
-       typedef MemberCaller<BrushInstance, void(const Selectable&), &BrushInstance::selectedChangedComponent> SelectedChangedComponentCaller;
+void selectedChangedComponent( const Selectable& selectable ){
+       GlobalSelectionSystem().getObserver ( SelectionSystem::eComponent )( selectable );
+       GlobalSelectionSystem().onComponentSelection( *this, selectable );
+}
+typedef MemberCaller<BrushInstance, void(const Selectable&), &BrushInstance::selectedChangedComponent> SelectedChangedComponentCaller;
 
-       const BrushInstanceVisitor& forEachFaceInstance( const BrushInstanceVisitor& visitor ){
-               for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       visitor.visit( *i );
-               }
-               return visitor;
+const BrushInstanceVisitor& forEachFaceInstance( const BrushInstanceVisitor& visitor ){
+       for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               visitor.visit( *i );
        }
+       return visitor;
+}
 
-       static void constructStatic(){
-               m_state_selpoint = GlobalShaderCache().capture( "$SELPOINT" );
-       }
+static void constructStatic(){
+       m_state_selpoint = GlobalShaderCache().capture( "$SELPOINT" );
+}
 
-       static void destroyStatic(){
-               GlobalShaderCache().release( "$SELPOINT" );
-       }
+static void destroyStatic(){
+       GlobalShaderCache().release( "$SELPOINT" );
+}
 
-       void clear(){
-               m_faceInstances.clear();
-       }
+void clear(){
+       m_faceInstances.clear();
+}
 
-       void reserve( std::size_t size ){
-               m_faceInstances.reserve( size );
-       }
+void reserve( std::size_t size ){
+       m_faceInstances.reserve( size );
+}
 
-       void push_back( Face& face ){
-               m_faceInstances.push_back( FaceInstance( face, SelectedChangedComponentCaller( *this ) ) );
-       }
+void push_back( Face& face ){
+       m_faceInstances.push_back( FaceInstance( face, SelectedChangedComponentCaller( *this ) ) );
+}
 
-       void pop_back(){
-               ASSERT_MESSAGE( !m_faceInstances.empty(), "erasing invalid element" );
-               m_faceInstances.pop_back();
-       }
+void pop_back(){
+       ASSERT_MESSAGE( !m_faceInstances.empty(), "erasing invalid element" );
+       m_faceInstances.pop_back();
+}
 
-       void erase( std::size_t index ){
-               ASSERT_MESSAGE( index < m_faceInstances.size(), "erasing invalid element" );
-               m_faceInstances.erase( m_faceInstances.begin() + index );
-       }
+void erase( std::size_t index ){
+       ASSERT_MESSAGE( index < m_faceInstances.size(), "erasing invalid element" );
+       m_faceInstances.erase( m_faceInstances.begin() + index );
+}
 
-       void connectivityChanged(){
-               for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       ( *i ).connectivityChanged();
-               }
+void connectivityChanged(){
+       for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               ( *i ).connectivityChanged();
        }
+}
 
-       void edge_clear(){
-               m_edgeInstances.clear();
-       }
+void edge_clear(){
+       m_edgeInstances.clear();
+}
 
-       void edge_push_back( SelectableEdge& edge ){
-               m_edgeInstances.push_back( EdgeInstance( m_faceInstances, edge ) );
-       }
+void edge_push_back( SelectableEdge& edge ){
+       m_edgeInstances.push_back( EdgeInstance( m_faceInstances, edge ) );
+}
 
-       void vertex_clear(){
-               m_vertexInstances.clear();
-       }
+void vertex_clear(){
+       m_vertexInstances.clear();
+}
 
-       void vertex_push_back( SelectableVertex& vertex ){
-               m_vertexInstances.push_back( VertexInstance( m_faceInstances, vertex ) );
-       }
+void vertex_push_back( SelectableVertex& vertex ){
+       m_vertexInstances.push_back( VertexInstance( m_faceInstances, vertex ) );
+}
 
-       void DEBUG_verify() const {
-               ASSERT_MESSAGE( m_faceInstances.size() == m_brush.DEBUG_size(), "FATAL: mismatch" );
-       }
+void DEBUG_verify() const {
+       ASSERT_MESSAGE( m_faceInstances.size() == m_brush.DEBUG_size(), "FATAL: mismatch" );
+}
 
-       bool isSelected() const {
-               return m_selectable.isSelected();
-       }
+bool isSelected() const {
+       return m_selectable.isSelected();
+}
 
-       void setSelected( bool select ){
-               m_selectable.setSelected( select );
-       }
+void setSelected( bool select ){
+       m_selectable.setSelected( select );
+}
 
-       void update_selected() const {
-               m_render_selected.clear();
-               for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       if ( ( *i ).getFace().contributes() ) {
-                               ( *i ).iterate_selected( m_render_selected );
-                       }
+void update_selected() const {
+       m_render_selected.clear();
+       for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               if ( ( *i ).getFace().contributes() ) {
+                       ( *i ).iterate_selected( m_render_selected );
                }
        }
+}
 
-       void evaluateViewDependent( const VolumeTest& volume, const Matrix4& localToWorld ) const {
-               if ( m_viewChanged ) {
-                       m_viewChanged = false;
+void evaluateViewDependent( const VolumeTest& volume, const Matrix4& localToWorld ) const {
+       if ( m_viewChanged ) {
+               m_viewChanged = false;
 
-                       bool faces_visible[c_brush_maxFaces];
+               bool faces_visible[c_brush_maxFaces];
+               {
+                       bool* j = faces_visible;
+                       for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i, ++j )
                        {
-                               bool* j = faces_visible;
-                               for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i, ++j )
-                               {
-                                       *j = ( *i ).intersectVolume( volume, localToWorld );
-                               }
+                               *j = ( *i ).intersectVolume( volume, localToWorld );
                        }
-
-                       m_brush.update_wireframe( m_render_wireframe, faces_visible );
-                       m_brush.update_faces_wireframe( m_faceCentroidPointsCulled, faces_visible );
                }
+
+               m_brush.update_wireframe( m_render_wireframe, faces_visible );
+               m_brush.update_faces_wireframe( m_faceCentroidPointsCulled, faces_visible );
        }
+}
 
-       void renderComponentsSelected( Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
-               m_brush.evaluateBRep();
+void renderComponentsSelected( Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
+       m_brush.evaluateBRep();
 
-               update_selected();
-               if ( !m_render_selected.empty() ) {
-                       renderer.Highlight( Renderer::ePrimitive, false );
-                       renderer.SetState( m_state_selpoint, Renderer::eWireframeOnly );
-                       renderer.SetState( m_state_selpoint, Renderer::eFullMaterials );
-                       renderer.addRenderable( m_render_selected, localToWorld );
-               }
+       update_selected();
+       if ( !m_render_selected.empty() ) {
+               renderer.Highlight( Renderer::ePrimitive, false );
+               renderer.SetState( m_state_selpoint, Renderer::eWireframeOnly );
+               renderer.SetState( m_state_selpoint, Renderer::eFullMaterials );
+               renderer.addRenderable( m_render_selected, localToWorld );
        }
+}
 
-       void renderComponents( Renderer& renderer, const VolumeTest& volume ) const {
-               m_brush.evaluateBRep();
+void renderComponents( Renderer& renderer, const VolumeTest& volume ) const {
+       m_brush.evaluateBRep();
 
-               const Matrix4& localToWorld = Instance::localToWorld();
+       const Matrix4& localToWorld = Instance::localToWorld();
 
-               renderer.SetState( m_brush.m_state_point, Renderer::eWireframeOnly );
-               renderer.SetState( m_brush.m_state_point, Renderer::eFullMaterials );
+       renderer.SetState( m_brush.m_state_point, Renderer::eWireframeOnly );
+       renderer.SetState( m_brush.m_state_point, Renderer::eFullMaterials );
 
-               if ( volume.fill() && GlobalSelectionSystem().ComponentMode() == SelectionSystem::eFace ) {
-                       evaluateViewDependent( volume, localToWorld );
-                       renderer.addRenderable( m_render_faces_wireframe, localToWorld );
-               }
-               else
-               {
-                       m_brush.renderComponents( GlobalSelectionSystem().ComponentMode(), renderer, volume, localToWorld );
-               }
+       if ( volume.fill() && GlobalSelectionSystem().ComponentMode() == SelectionSystem::eFace ) {
+               evaluateViewDependent( volume, localToWorld );
+               renderer.addRenderable( m_render_faces_wireframe, localToWorld );
+       }
+       else
+       {
+               m_brush.renderComponents( GlobalSelectionSystem().ComponentMode(), renderer, volume, localToWorld );
        }
+}
 
-       void renderClipPlane( Renderer& renderer, const VolumeTest& volume ) const {
-               if ( GlobalSelectionSystem().ManipulatorMode() == SelectionSystem::eClip && isSelected() ) {
-                       m_clipPlane.render( renderer, volume, localToWorld() );
-               }
+void renderClipPlane( Renderer& renderer, const VolumeTest& volume ) const {
+       if ( GlobalSelectionSystem().ManipulatorMode() == SelectionSystem::eClip && isSelected() ) {
+               m_clipPlane.render( renderer, volume, localToWorld() );
        }
+}
 
-       void renderCommon( Renderer& renderer, const VolumeTest& volume ) const {
-               bool componentMode = GlobalSelectionSystem().Mode() == SelectionSystem::eComponent;
+void renderCommon( Renderer& renderer, const VolumeTest& volume ) const {
+       bool componentMode = GlobalSelectionSystem().Mode() == SelectionSystem::eComponent;
 
-               if ( componentMode && isSelected() ) {
-                       renderComponents( renderer, volume );
-               }
+       if ( componentMode && isSelected() ) {
+               renderComponents( renderer, volume );
+       }
 
-               if ( parentSelected() ) {
-                       if ( !componentMode ) {
-                               renderer.Highlight( Renderer::eFace );
-                       }
-                       renderer.Highlight( Renderer::ePrimitive );
+       if ( parentSelected() ) {
+               if ( !componentMode ) {
+                       renderer.Highlight( Renderer::eFace );
                }
+               renderer.Highlight( Renderer::ePrimitive );
        }
+}
 
-       void renderSolid( Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
-               //renderCommon(renderer, volume);
-
-               m_lightList->evaluateLights();
+void renderSolid( Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
+       //renderCommon(renderer, volume);
 
-               for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       renderer.setLights( ( *i ).m_lights );
-                       ( *i ).render( renderer, volume, localToWorld );
-               }
+       m_lightList->evaluateLights();
 
-               renderComponentsSelected( renderer, volume, localToWorld );
+       for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               renderer.setLights( ( *i ).m_lights );
+               ( *i ).render( renderer, volume, localToWorld );
        }
 
-       void renderWireframe( Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
-               //renderCommon(renderer, volume);
+       renderComponentsSelected( renderer, volume, localToWorld );
+}
 
-               evaluateViewDependent( volume, localToWorld );
+void renderWireframe( Renderer& renderer, const VolumeTest& volume, const Matrix4& localToWorld ) const {
+       //renderCommon(renderer, volume);
 
-               if ( m_render_wireframe.m_size != 0 ) {
-                       renderer.addRenderable( m_render_wireframe, localToWorld );
-               }
+       evaluateViewDependent( volume, localToWorld );
 
-               renderComponentsSelected( renderer, volume, localToWorld );
+       if ( m_render_wireframe.m_size != 0 ) {
+               renderer.addRenderable( m_render_wireframe, localToWorld );
        }
 
-       void renderSolid( Renderer& renderer, const VolumeTest& volume ) const {
-               m_brush.evaluateBRep();
+       renderComponentsSelected( renderer, volume, localToWorld );
+}
 
-               renderClipPlane( renderer, volume );
+void renderSolid( Renderer& renderer, const VolumeTest& volume ) const {
+       m_brush.evaluateBRep();
 
-               renderSolid( renderer, volume, localToWorld() );
-       }
+       renderClipPlane( renderer, volume );
 
-       void renderWireframe( Renderer& renderer, const VolumeTest& volume ) const {
-               m_brush.evaluateBRep();
+       renderSolid( renderer, volume, localToWorld() );
+}
 
-               renderClipPlane( renderer, volume );
+void renderWireframe( Renderer& renderer, const VolumeTest& volume ) const {
+       m_brush.evaluateBRep();
 
-               renderWireframe( renderer, volume, localToWorld() );
-       }
+       renderClipPlane( renderer, volume );
 
-       void viewChanged() const {
-               m_viewChanged = true;
-       }
+       renderWireframe( renderer, volume, localToWorld() );
+}
 
-       void testSelect( Selector& selector, SelectionTest& test ){
-               test.BeginMesh( localToWorld() );
+void viewChanged() const {
+       m_viewChanged = true;
+}
 
-               SelectionIntersection best;
-               for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       ( *i ).testSelect( test, best );
-               }
-               if ( best.valid() ) {
-                       selector.addIntersection( best );
-               }
+void testSelect( Selector& selector, SelectionTest& test ){
+       test.BeginMesh( localToWorld() );
+
+       SelectionIntersection best;
+       for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               ( *i ).testSelect( test, best );
+       }
+       if ( best.valid() ) {
+               selector.addIntersection( best );
        }
+}
 
-       bool isSelectedComponents() const {
-               for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       if ( ( *i ).selectedComponents() ) {
-                               return true;
-                       }
+bool isSelectedComponents() const {
+       for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               if ( ( *i ).selectedComponents() ) {
+                       return true;
                }
-               return false;
        }
+       return false;
+}
 
-       void setSelectedComponents( bool select, SelectionSystem::EComponentMode mode ){
-               for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       ( *i ).setSelected( mode, select );
-               }
+void setSelectedComponents( bool select, SelectionSystem::EComponentMode mode ){
+       for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               ( *i ).setSelected( mode, select );
        }
+}
 
-       void testSelectComponents( Selector& selector, SelectionTest& test, SelectionSystem::EComponentMode mode ){
-               test.BeginMesh( localToWorld() );
+void testSelectComponents( Selector& selector, SelectionTest& test, SelectionSystem::EComponentMode mode ){
+       test.BeginMesh( localToWorld() );
 
-               switch ( mode )
-               {
-               case SelectionSystem::eVertex:
+       switch ( mode )
+       {
+       case SelectionSystem::eVertex:
+       {
+               for ( VertexInstances::iterator i = m_vertexInstances.begin(); i != m_vertexInstances.end(); ++i )
                {
-                       for ( VertexInstances::iterator i = m_vertexInstances.begin(); i != m_vertexInstances.end(); ++i )
-                       {
-                               ( *i ).testSelect( selector, test );
-                       }
+                       ( *i ).testSelect( selector, test );
                }
-               break;
-               case SelectionSystem::eEdge:
+       }
+       break;
+       case SelectionSystem::eEdge:
+       {
+               for ( EdgeInstances::iterator i = m_edgeInstances.begin(); i != m_edgeInstances.end(); ++i )
                {
-                       for ( EdgeInstances::iterator i = m_edgeInstances.begin(); i != m_edgeInstances.end(); ++i )
+                       ( *i ).testSelect( selector, test );
+               }
+       }
+       break;
+       case SelectionSystem::eFace:
+       {
+               if ( test.getVolume().fill() ) {
+                       for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
                        {
                                ( *i ).testSelect( selector, test );
                        }
                }
-               break;
-               case SelectionSystem::eFace:
+               else
                {
-                       if ( test.getVolume().fill() ) {
-                               for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-                               {
-                                       ( *i ).testSelect( selector, test );
-                               }
-                       }
-                       else
+                       for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
                        {
-                               for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-                               {
-                                       ( *i ).testSelect_centroid( selector, test );
-                               }
+                               ( *i ).testSelect_centroid( selector, test );
                        }
                }
+       }
+       break;
+       default:
                break;
-               default:
-                       break;
-               }
        }
+}
 
-       void selectPlanes( Selector& selector, SelectionTest& test, const PlaneCallback& selectedPlaneCallback ){
-               test.BeginMesh( localToWorld() );
+void selectPlanes( Selector& selector, SelectionTest& test, const PlaneCallback& selectedPlaneCallback ){
+       test.BeginMesh( localToWorld() );
 
-               PlanePointer brushPlanes[c_brush_maxFaces];
-               PlanesIterator j = brushPlanes;
+       PlanePointer brushPlanes[c_brush_maxFaces];
+       PlanesIterator j = brushPlanes;
 
-               for ( Brush::const_iterator i = m_brush.begin(); i != m_brush.end(); ++i )
-               {
-                       *j++ = &( *i )->plane3();
-               }
+       for ( Brush::const_iterator i = m_brush.begin(); i != m_brush.end(); ++i )
+       {
+               *j++ = &( *i )->plane3();
+       }
 
-               for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       ( *i ).selectPlane( selector, Line( test.getNear(), test.getFar() ), brushPlanes, j, selectedPlaneCallback );
-               }
+       for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               ( *i ).selectPlane( selector, Line( test.getNear(), test.getFar() ), brushPlanes, j, selectedPlaneCallback );
        }
+}
 
-       void selectReversedPlanes( Selector& selector, const SelectedPlanes& selectedPlanes ){
-               for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       ( *i ).selectReversedPlane( selector, selectedPlanes );
-               }
+void selectReversedPlanes( Selector& selector, const SelectedPlanes& selectedPlanes ){
+       for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               ( *i ).selectReversedPlane( selector, selectedPlanes );
        }
+}
 
 
-       void transformComponents( const Matrix4& matrix ){
-               for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       ( *i ).transformComponents( matrix );
-               }
+void transformComponents( const Matrix4& matrix ){
+       for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               ( *i ).transformComponents( matrix );
        }
+}
 
-       const AABB& getSelectedComponentsBounds() const {
-               m_aabb_component = AABB();
-
-               for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       ( *i ).iterate_selected( m_aabb_component );
-               }
+const AABB& getSelectedComponentsBounds() const {
+       m_aabb_component = AABB();
 
-               return m_aabb_component;
+       for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               ( *i ).iterate_selected( m_aabb_component );
        }
 
-       void snapComponents( float snap ){
-               for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       ( *i ).snapComponents( snap );
-               }
+       return m_aabb_component;
+}
+
+void snapComponents( float snap ){
+       for ( FaceInstances::iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               ( *i ).snapComponents( snap );
        }
+}
 
-       void evaluateTransform(){
-               Matrix4 matrix( m_transform.calculateTransform() );
-               //globalOutputStream() << "matrix: " << matrix << "\n";
+void evaluateTransform(){
+       Matrix4 matrix( m_transform.calculateTransform() );
+       //globalOutputStream() << "matrix: " << matrix << "\n";
 
-               if ( m_transform.getType() == TRANSFORM_PRIMITIVE ) {
-                       m_brush.transform( matrix );
-               }
-               else
-               {
-                       transformComponents( matrix );
-               }
+       if ( m_transform.getType() == TRANSFORM_PRIMITIVE ) {
+               m_brush.transform( matrix );
        }
-
-       void applyTransform(){
-               m_brush.revertTransform();
-               evaluateTransform();
-               m_brush.freezeTransform();
+       else
+       {
+               transformComponents( matrix );
        }
+}
 
-       typedef MemberCaller<BrushInstance, void(), &BrushInstance::applyTransform> ApplyTransformCaller;
+void applyTransform(){
+       m_brush.revertTransform();
+       evaluateTransform();
+       m_brush.freezeTransform();
+}
 
-       void setClipPlane( const Plane3& plane ){
-               m_clipPlane.setPlane( m_brush, plane );
-       }
+typedef MemberCaller<BrushInstance, void(), &BrushInstance::applyTransform> ApplyTransformCaller;
 
-       bool testLight( const RendererLight& light ) const {
-               return light.testAABB( worldAABB() );
-       }
+void setClipPlane( const Plane3& plane ){
+       m_clipPlane.setPlane( m_brush, plane );
+}
 
-       void insertLight( const RendererLight& light ){
-               const Matrix4& localToWorld = Instance::localToWorld();
-               for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       Face_addLight( *i, localToWorld, light );
-               }
+bool testLight( const RendererLight& light ) const {
+       return light.testAABB( worldAABB() );
+}
+
+void insertLight( const RendererLight& light ){
+       const Matrix4& localToWorld = Instance::localToWorld();
+       for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               Face_addLight( *i, localToWorld, light );
        }
+}
 
-       void clearLights(){
-               for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
-               {
-                       ( *i ).m_lights.clear();
-               }
+void clearLights(){
+       for ( FaceInstances::const_iterator i = m_faceInstances.begin(); i != m_faceInstances.end(); ++i )
+       {
+               ( *i ).m_lights.clear();
        }
+}
 };
 
 inline BrushInstance* Instance_getBrush( scene::Instance& instance ){
@@ -3608,17 +3628,17 @@ inline BrushInstance* Instance_getBrush( scene::Instance& instance ){
 template<typename Functor>
 class BrushSelectedVisitor : public SelectionSystem::Visitor
 {
-       const Functor& m_functor;
+const Functor& m_functor;
 public:
-       BrushSelectedVisitor( const Functor& functor ) : m_functor( functor ){
-       }
+BrushSelectedVisitor( const Functor& functor ) : m_functor( functor ){
+}
 
-       void visit( scene::Instance& instance ) const {
-               BrushInstance* brush = Instance_getBrush( instance );
-               if ( brush != 0 ) {
-                       m_functor( *brush );
-               }
+void visit( scene::Instance& instance ) const {
+       BrushInstance* brush = Instance_getBrush( instance );
+       if ( brush != 0 ) {
+               m_functor( *brush );
        }
+}
 };
 
 template<typename Functor>
@@ -3630,18 +3650,18 @@ inline const Functor& Scene_forEachSelectedBrush( const Functor& functor ){
 template<typename Functor>
 class BrushVisibleSelectedVisitor : public SelectionSystem::Visitor
 {
-       const Functor& m_functor;
+const Functor& m_functor;
 public:
-       BrushVisibleSelectedVisitor( const Functor& functor ) : m_functor( functor ){
-       }
+BrushVisibleSelectedVisitor( const Functor& functor ) : m_functor( functor ){
+}
 
-       void visit( scene::Instance& instance ) const {
-               BrushInstance* brush = Instance_getBrush( instance );
-               if ( brush != 0
-                        && instance.path().top().get().visible() ) {
-                       m_functor( *brush );
-               }
+void visit( scene::Instance& instance ) const {
+       BrushInstance* brush = Instance_getBrush( instance );
+       if ( brush != 0
+                && instance.path().top().get().visible() ) {
+               m_functor( *brush );
        }
+}
 };
 
 template<typename Functor>
@@ -3652,28 +3672,28 @@ inline const Functor& Scene_forEachVisibleSelectedBrush( const Functor& functor
 
 class BrushForEachFace
 {
-       const BrushInstanceVisitor& m_visitor;
+const BrushInstanceVisitor& m_visitor;
 public:
-       BrushForEachFace( const BrushInstanceVisitor& visitor ) : m_visitor( visitor ){
-       }
+BrushForEachFace( const BrushInstanceVisitor& visitor ) : m_visitor( visitor ){
+}
 
-       void operator()( BrushInstance& brush ) const {
-               brush.forEachFaceInstance( m_visitor );
-       }
+void operator()( BrushInstance& brush ) const {
+       brush.forEachFaceInstance( m_visitor );
+}
 };
 
 template<class Functor>
 class FaceInstanceVisitFace : public BrushInstanceVisitor
 {
-       const Functor& functor;
+const Functor& functor;
 public:
-       FaceInstanceVisitFace( const Functor& functor )
-               : functor( functor ){
-       }
+FaceInstanceVisitFace( const Functor& functor )
+       : functor( functor ){
+}
 
-       void visit( FaceInstance& face ) const {
-               functor( face.getFace() );
-       }
+void visit( FaceInstance& face ) const {
+       functor( face.getFace() );
+}
 };
 
 template<typename Functor>
@@ -3685,15 +3705,15 @@ inline const Functor& Brush_forEachFace( BrushInstance& brush, const Functor& fu
 template<class Functor>
 class FaceVisitAll : public BrushVisitor
 {
-       const Functor& functor;
+const Functor& functor;
 public:
-       FaceVisitAll( const Functor& functor )
-               : functor( functor ){
-       }
+FaceVisitAll( const Functor& functor )
+       : functor( functor ){
+}
 
-       void visit( Face& face ) const {
-               functor( face );
-       }
+void visit( Face& face ) const {
+       functor( face );
+}
 };
 
 template<typename Functor>
@@ -3711,15 +3731,15 @@ inline const Functor& Brush_forEachFace( Brush& brush, const Functor& functor ){
 template<class Functor>
 class FaceInstanceVisitAll : public BrushInstanceVisitor
 {
-       const Functor& functor;
+const Functor& functor;
 public:
-       FaceInstanceVisitAll( const Functor& functor )
-               : functor( functor ){
-       }
+FaceInstanceVisitAll( const Functor& functor )
+       : functor( functor ){
+}
 
-       void visit( FaceInstance& face ) const {
-               functor( face );
-       }
+void visit( FaceInstance& face ) const {
+       functor( face );
+}
 };
 
 template<typename Functor>
@@ -3738,33 +3758,33 @@ template<typename Type, typename Functor>
 class InstanceIfVisible : public Functor
 {
 public:
-       InstanceIfVisible( const Functor& functor ) : Functor( functor ){
-       }
+InstanceIfVisible( const Functor& functor ) : Functor( functor ){
+}
 
-       void operator()( scene::Instance& instance ){
-               if ( instance.path().top().get().visible() ) {
-                       Functor::operator()( instance );
-               }
+void operator()( scene::Instance& instance ){
+       if ( instance.path().top().get().visible() ) {
+               Functor::operator()( instance );
        }
+}
 };
 
 template<typename Functor>
 class BrushVisibleWalker : public scene::Graph::Walker
 {
-       const Functor& m_functor;
+const Functor& m_functor;
 public:
-       BrushVisibleWalker( const Functor& functor ) : m_functor( functor ){
-       }
+BrushVisibleWalker( const Functor& functor ) : m_functor( functor ){
+}
 
-       bool pre( const scene::Path& path, scene::Instance& instance ) const {
-               if ( path.top().get().visible() ) {
-                       BrushInstance* brush = Instance_getBrush( instance );
-                       if ( brush != 0 ) {
-                               m_functor( *brush );
-                       }
+bool pre( const scene::Path& path, scene::Instance& instance ) const {
+       if ( path.top().get().visible() ) {
+               BrushInstance* brush = Instance_getBrush( instance );
+               if ( brush != 0 ) {
+                       m_functor( *brush );
                }
-               return true;
        }
+       return true;
+}
 };
 
 template<typename Functor>
@@ -3801,14 +3821,14 @@ inline const Functor& Scene_ForEachSelectedBrush_ForEachFaceInstance( scene::Gra
 template<typename Functor>
 class FaceVisitorWrapper
 {
-       const Functor& functor;
+const Functor& functor;
 public:
-       FaceVisitorWrapper( const Functor& functor ) : functor( functor ){
-       }
+FaceVisitorWrapper( const Functor& functor ) : functor( functor ){
+}
 
-       void operator()( FaceInstance& faceInstance ) const {
-               functor( faceInstance.getFace() );
-       }
+void operator()( FaceInstance& faceInstance ) const {
+       functor( faceInstance.getFace() );
+}
 };
 
 template<typename Functor>