# HG changeset patch
# User Santeri Piippo <crimsondusk64@gmail.com>
# Date 1398194257 -10800
# Node ID 9a53d9a379de6c726434d9177ce54132b87eff3c
# Parent  37b71daf4cb55ccb517af356a3db6cdd075723fe
- refactored the gl compiler a tad, made selection blend work with BFC/random colors

diff -r 37b71daf4cb5 -r 9a53d9a379de src/glCompiler.cc
--- a/src/glCompiler.cc	Tue Apr 22 20:40:19 2014 +0300
+++ b/src/glCompiler.cc	Tue Apr 22 22:17:37 2014 +0300
@@ -127,27 +127,52 @@
 
 // =============================================================================
 //
-QColor GLCompiler::getColorForPolygon (LDPolygon& poly, LDObject* topobj) const
+QColor GLCompiler::getColorForPolygon (LDPolygon& poly, LDObject* topobj,
+									   EVBOComplement complement) const
 {
 	QColor qcol;
 
-	if (poly.color == maincolor)
-	{
-		if (topobj->color() == maincolor)
-			qcol = GLRenderer::getMainColor();
-		else
-			qcol = getColor (topobj->color())->faceColor;
-	}
-	elif (poly.color == edgecolor)
+	switch (complement)
 	{
-		qcol = luma (QColor (gl_bgcolor)) > 40 ? Qt::black : Qt::white;
-	}
-	else
-	{
-		LDColor* col = getColor (poly.color);
+		case VBOCM_Surfaces:
+		case VBOCM_NumComplements:
+			return QColor();
+
+		case VBOCM_BFCFrontColors:
+			qcol = g_BFCFrontColor;
+			break;
+
+		case VBOCM_BFCBackColors:
+			qcol = g_BFCBackColor;
+			break;
+
+		case VBOCM_PickColors:
+			return indexColorForID (topobj->id());
+
+		case VBOCM_RandomColors:
+			qcol = topobj->randomColor();
+			break;
 
-		if (col)
-			qcol = col->faceColor;
+		case VBOCM_NormalColors:
+			if (poly.color == maincolor)
+			{
+				if (topobj->color() == maincolor)
+					qcol = GLRenderer::getMainColor();
+				else
+					qcol = getColor (topobj->color())->faceColor;
+			}
+			elif (poly.color == edgecolor)
+			{
+				qcol = luma (QColor (gl_bgcolor)) > 40 ? Qt::black : Qt::white;
+			}
+			else
+			{
+				LDColor* col = getColor (poly.color);
+
+				if (col)
+					qcol = col->faceColor;
+			}
+			break;
 	}
 
 	if (not qcol.isValid())
@@ -284,7 +309,40 @@
 	ObjectVBOInfo info;
 	info.isChanged = true;
 	dropObject (obj);
-	compileSubObject (obj, obj, &info);
+
+	switch (obj->type())
+	{
+		// Note: We cannot split quads into triangles here, it would mess up the
+		// wireframe view. Quads must go into separate vbos.
+		case LDObject::ETriangle:
+		case LDObject::EQuad:
+		case LDObject::ELine:
+		case LDObject::ECondLine:
+		{
+			LDPolygon* poly = obj->getPolygon();
+			poly->id = obj->id();
+			compilePolygon (*poly, obj, &info);
+			delete poly;
+			break;
+		}
+
+		case LDObject::ESubfile:
+		{
+			LDSubfile* ref = static_cast<LDSubfile*> (obj);
+			auto data = ref->inlinePolygons();
+
+			for (LDPolygon& poly : data)
+			{
+				poly.id = obj->id();
+				compilePolygon (poly, obj, &info);
+			}
+			break;
+		}
+
+		default:
+			break;
+	}
+
 	m_objectInfo[obj] = info;
 	needMerge();
 }
@@ -309,103 +367,24 @@
 	{
 		const int vbonum			= vboNumber (surface, (EVBOComplement) complement);
 		QVector<GLfloat>& vbodata	= objinfo->data[vbonum];
-		const QColor normalColor	= getColorForPolygon (poly, topobj);
-		const QColor pickColor		= indexColorForID (topobj->id());
+		const QColor color			= getColorForPolygon (poly, topobj, (EVBOComplement) complement);
 
 		for (int vert = 0; vert < numverts; ++vert)
 		{
-			switch ((EVBOComplement) complement)
+			if (complement == VBOCM_Surfaces)
 			{
-				case VBOCM_Surfaces:
-				{
-					// Write coordinates. Apparently Z must be flipped too?
-					vbodata	<< poly.vertices[vert].x()
-							<< -poly.vertices[vert].y()
-							<< -poly.vertices[vert].z();
-					break;
-				}
-
-				case VBOCM_NormalColors:
-				{
-					writeColor (vbodata, normalColor);
-					break;
-				}
-
-				case VBOCM_PickColors:
-				{
-					writeColor (vbodata, pickColor);
-					break;
-				}
-
-				case VBOCM_BFCFrontColors:
-				{
-					writeColor (vbodata, g_BFCFrontColor);
-					break;
-				}
-
-				case VBOCM_BFCBackColors:
-				{
-					writeColor (vbodata, g_BFCBackColor);
-					break;
-				}
-
-				case VBOCM_RandomColors:
-				{
-					writeColor (vbodata, topobj->randomColor());
-					break;
-				}
-
-				case VBOCM_NumComplements:
-					break;
+				// Write coordinates. Apparently Z must be flipped too?
+				vbodata	<< poly.vertices[vert].x()
+						<< -poly.vertices[vert].y()
+						<< -poly.vertices[vert].z();
+			}
+			else
+			{
+				vbodata	<< ((GLfloat) color.red()) / 255.0f
+						<< ((GLfloat) color.green()) / 255.0f
+						<< ((GLfloat) color.blue()) / 255.0f
+						<< ((GLfloat) color.alpha()) / 255.0f;
 			}
 		}
 	}
 }
-
-// =============================================================================
-//
-void GLCompiler::compileSubObject (LDObject* obj, LDObject* topobj, ObjectVBOInfo* objinfo)
-{
-	switch (obj->type())
-	{
-		// Note: We cannot split quads into triangles here, it would mess up the
-		// wireframe view. Quads must go into separate vbos.
-		case LDObject::ETriangle:
-		case LDObject::EQuad:
-		case LDObject::ELine:
-		case LDObject::ECondLine:
-		{
-			LDPolygon* poly = obj->getPolygon();
-			poly->id = topobj->id();
-			compilePolygon (*poly, topobj, objinfo);
-			delete poly;
-			break;
-		}
-
-		case LDObject::ESubfile:
-		{
-			LDSubfile* ref = static_cast<LDSubfile*> (obj);
-			auto data = ref->inlinePolygons();
-
-			for (LDPolygon& poly : data)
-			{
-				poly.id = topobj->id();
-				compilePolygon (poly, topobj, objinfo);
-			}
-			break;
-		}
-
-		default:
-			break;
-	}
-}
-
-// =============================================================================
-//
-void GLCompiler::writeColor (QVector<GLfloat>& array, const QColor& color)
-{
-	array	<< ((GLfloat) color.red()) / 255.0f
-			<< ((GLfloat) color.green()) / 255.0f
-			<< ((GLfloat) color.blue()) / 255.0f
-			<< ((GLfloat) color.alpha()) / 255.0f;
-}
diff -r 37b71daf4cb5 -r 9a53d9a379de src/glCompiler.h
--- a/src/glCompiler.h	Tue Apr 22 20:40:19 2014 +0300
+++ b/src/glCompiler.h	Tue Apr 22 22:17:37 2014 +0300
@@ -40,7 +40,8 @@
 	void				compileDocument (LDDocument* doc);
 	void				dropObject (LDObject* obj);
 	void				initialize();
-	QColor				getColorForPolygon (LDPolygon& poly, LDObject* topobj) const;
+	QColor				getColorForPolygon (LDPolygon& poly, LDObject* topobj,
+											EVBOComplement complement) const;
 	QColor				indexColorForID (int id) const;
 	void				needMerge();
 	void				prepareVBO (int vbonum);
@@ -67,8 +68,6 @@
 private:
 	void			compileStaged();
 	void			compileObject (LDObject* obj);
-	void			compileSubObject (LDObject* obj, LDObject* topobj, GLCompiler::ObjectVBOInfo* objinfo);
-	void			writeColor (QVector<float>& array, const QColor& color);
 	void			compilePolygon (LDPolygon& poly, LDObject* topobj, GLCompiler::ObjectVBOInfo* objinfo);
 
 	QMap<LDObject*, ObjectVBOInfo>		m_objectInfo;