summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/datavisualization/axis/qabstract3daxis.h1
-rw-r--r--src/datavisualization/data/qbardataitem.cpp1
-rw-r--r--src/datavisualization/data/qbardataitem_p.h2
-rw-r--r--src/datavisualization/data/qitemmodelsurfacedataproxy.cpp2
-rw-r--r--src/datavisualization/data/qscatterdataitem_p.h2
-rw-r--r--src/datavisualization/data/qsurfacedataitem.cpp2
-rw-r--r--src/datavisualization/data/qsurfacedataitem_p.h2
-rw-r--r--src/datavisualization/data/scatterrenderitem_p.h6
-rw-r--r--src/datavisualization/engine/bars3drenderer.cpp15
-rw-r--r--src/datavisualization/engine/drawer.cpp2
-rw-r--r--src/datavisualization/engine/q3dscene.cpp2
-rw-r--r--src/datavisualization/engine/scatter3dcontroller.cpp4
-rw-r--r--src/datavisualization/engine/scatter3drenderer.cpp11
-rw-r--r--src/datavisualization/engine/seriesrendercache.cpp1
-rw-r--r--src/datavisualization/engine/surface3dcontroller.cpp4
-rw-r--r--src/datavisualization/engine/surface3drenderer.cpp25
-rw-r--r--src/datavisualization/input/qabstract3dinputhandler_p.h1
-rw-r--r--src/datavisualization/theme/q3dtheme.cpp1
-rw-r--r--src/datavisualization/utils/meshloader_p.h1
-rw-r--r--src/datavisualizationqml2/abstractdeclarative.cpp1
20 files changed, 1 insertions, 85 deletions
diff --git a/src/datavisualization/axis/qabstract3daxis.h b/src/datavisualization/axis/qabstract3daxis.h
index c4cb37b2..7cfde83a 100644
--- a/src/datavisualization/axis/qabstract3daxis.h
+++ b/src/datavisualization/axis/qabstract3daxis.h
@@ -55,7 +55,6 @@ public:
AxisTypeNone = 0,
AxisTypeCategory = 1,
AxisTypeValue = 2
- //AxisTypeLogValue = 6 // inherits valueaxis (4 + 2) // TODO
};
protected:
diff --git a/src/datavisualization/data/qbardataitem.cpp b/src/datavisualization/data/qbardataitem.cpp
index 792e5158..3e8ebee2 100644
--- a/src/datavisualization/data/qbardataitem.cpp
+++ b/src/datavisualization/data/qbardataitem.cpp
@@ -76,7 +76,6 @@ QBarDataItem &QBarDataItem::operator=(const QBarDataItem &other)
createExtraData();
else
d_ptr = 0;
- // TODO set extra data
return *this;
}
diff --git a/src/datavisualization/data/qbardataitem_p.h b/src/datavisualization/data/qbardataitem_p.h
index 6180c039..39ccf8c3 100644
--- a/src/datavisualization/data/qbardataitem_p.h
+++ b/src/datavisualization/data/qbardataitem_p.h
@@ -40,8 +40,6 @@ public:
QBarDataItemPrivate();
virtual ~QBarDataItemPrivate();
- // TODO stores other data for bars besides value
-
protected:
friend class QBarDataItem;
};
diff --git a/src/datavisualization/data/qitemmodelsurfacedataproxy.cpp b/src/datavisualization/data/qitemmodelsurfacedataproxy.cpp
index 8c11ba88..0753ae09 100644
--- a/src/datavisualization/data/qitemmodelsurfacedataproxy.cpp
+++ b/src/datavisualization/data/qitemmodelsurfacedataproxy.cpp
@@ -22,8 +22,6 @@
QT_BEGIN_NAMESPACE_DATAVISUALIZATION
-// TODO: CHECK DOCUMENTATION!
-
/*!
* \class QItemModelSurfaceDataProxy
* \inmodule QtDataVisualization
diff --git a/src/datavisualization/data/qscatterdataitem_p.h b/src/datavisualization/data/qscatterdataitem_p.h
index d9c7af94..730c8f89 100644
--- a/src/datavisualization/data/qscatterdataitem_p.h
+++ b/src/datavisualization/data/qscatterdataitem_p.h
@@ -40,8 +40,6 @@ public:
QScatterDataItemPrivate();
virtual ~QScatterDataItemPrivate();
- // TODO stores other data for scatter items besides position
-
protected:
friend class QScatterDataItem;
};
diff --git a/src/datavisualization/data/qsurfacedataitem.cpp b/src/datavisualization/data/qsurfacedataitem.cpp
index 41d2dff9..23f27478 100644
--- a/src/datavisualization/data/qsurfacedataitem.cpp
+++ b/src/datavisualization/data/qsurfacedataitem.cpp
@@ -72,13 +72,11 @@ QSurfaceDataItem::~QSurfaceDataItem()
QSurfaceDataItem &QSurfaceDataItem::operator=(const QSurfaceDataItem &other)
{
m_position = other.m_position;
- //m_size = other.m_size;
if (other.d_ptr)
createExtraData();
else
d_ptr = 0;
- // TODO set extra data
return *this;
}
diff --git a/src/datavisualization/data/qsurfacedataitem_p.h b/src/datavisualization/data/qsurfacedataitem_p.h
index d1e7c7da..fda3a114 100644
--- a/src/datavisualization/data/qsurfacedataitem_p.h
+++ b/src/datavisualization/data/qsurfacedataitem_p.h
@@ -40,8 +40,6 @@ public:
QSurfaceDataItemPrivate();
virtual ~QSurfaceDataItemPrivate();
- // TODO stores other data for surface items besides position
-
protected:
friend class QSurfaceDataItem;
};
diff --git a/src/datavisualization/data/scatterrenderitem_p.h b/src/datavisualization/data/scatterrenderitem_p.h
index 672eda78..1c88cbbd 100644
--- a/src/datavisualization/data/scatterrenderitem_p.h
+++ b/src/datavisualization/data/scatterrenderitem_p.h
@@ -60,19 +60,13 @@ public:
m_rotation = rotation;
}
-
inline bool isVisible() const { return m_visible; }
inline void setVisible(bool visible) { m_visible = visible; }
- // TODO: Will we ever support item specific size? If not, remove.
- //inline void setSize(float size);
- //inline float size() const { return m_size; }
-
protected:
QVector3D m_position;
QQuaternion m_rotation;
bool m_visible;
- //float m_size; // TODO in case we need a fourth variable that adjusts scatter item size
friend class QScatterDataItem;
};
diff --git a/src/datavisualization/engine/bars3drenderer.cpp b/src/datavisualization/engine/bars3drenderer.cpp
index 17f61818..7470f0cf 100644
--- a/src/datavisualization/engine/bars3drenderer.cpp
+++ b/src/datavisualization/engine/bars3drenderer.cpp
@@ -43,7 +43,7 @@ QT_BEGIN_NAMESPACE_DATAVISUALIZATION
const GLfloat labelMargin = 0.05f;
const GLfloat gridLineWidth = 0.005f;
-const bool sliceGridLabels = true; // TODO: Make this user controllable (QTRD-2546)
+const bool sliceGridLabels = true;
Bars3DRenderer::Bars3DRenderer(Bars3DController *controller)
: Abstract3DRenderer(controller),
@@ -242,7 +242,6 @@ void Bars3DRenderer::updateData()
void Bars3DRenderer::updateScene(Q3DScene *scene)
{
- // TODO: See QTRD-2374
if (m_hasNegativeValues)
scene->activeCamera()->setMinYRotation(-90.0);
else
@@ -308,7 +307,6 @@ void Bars3DRenderer::drawSlicedScene()
GLfloat barPosYAdjustment = -0.8f; // Translate to -1.0 + 0.2 for row/column labels
GLfloat scaleFactor = 0.0f;
- // TODO: When doing this correctly (QTRD-2125), use label dimensions instead of magic number
GLfloat barLabelYPos = barPosYAdjustment - 0.4f - labelMargin; // 0.4 for labels
GLfloat zeroPosAdjustment = 0.0f;
if (!m_noZeroInRange)
@@ -393,7 +391,6 @@ void Bars3DRenderer::drawSlicedScene()
}
}
- // TODO: Make user controllable (QTRD-2546)
if (sliceGridLabels) {
// Bind label shader
m_labelShader->bind();
@@ -593,7 +590,6 @@ void Bars3DRenderer::drawSlicedScene()
m_dummyBarRenderItem.setTranslation(QVector3D(item->translation().x(),
barLabelYPos,
item->translation().z()));
- // TODO: Make user controllable (QTRD-2546)
// Draw labels
m_drawer->drawLabel(m_dummyBarRenderItem, *m_sliceCache->labelItems().at(labelNo),
viewMatrix, projectionMatrix, positionComp, sliceLabelRotation,
@@ -605,7 +601,6 @@ void Bars3DRenderer::drawSlicedScene()
for (int col = 0; col < sliceItemCount; col++) {
BarRenderItem *item = m_sliceSelection.at(col);
- // TODO: Make user controllable (QTRD-2546)
if (!sliceGridLabels) {
// Draw values
if (item->height() != 0.0f || (!m_noZeroInRange && item->value() == 0.0f)) {
@@ -631,8 +626,6 @@ void Bars3DRenderer::drawSlicedScene()
}
} else {
// Only draw value for selected item when grid labels are on
- // TODO: Maybe use selection label instead of value? Should it be user controllable
- // as well? (QTRD-2546)
if (itemMode && m_visualSelectedBarPos.x() == item->position().x()
&& m_visualSelectedBarPos.y() == item->position().y()
&& item->seriesIndex() == m_visualSelectedBarSeriesIndex) {
@@ -659,7 +652,6 @@ void Bars3DRenderer::drawSlicedScene()
}
}
- // TODO: Make user controllable (QTRD-2546)
// Draw labels for axes
if (rowMode) {
if (m_sliceTitleItem) {
@@ -812,10 +804,6 @@ void Bars3DRenderer::drawScene(GLuint defaultFboHandle)
zeroVector, 0.0f, 3.5f / m_autoScaleAdjustment);
depthViewMatrix.lookAt(depthLightPos, zeroVector, upVector);
- // TODO: Why does depthViewMatrix.column(3).y() goes to zero when we're directly above?
- // That causes the scene to be not drawn from above -> must be fixed
- // qDebug() << lightPos << depthViewMatrix << depthViewMatrix.column(3);
-
// Set the depth projection matrix
depthProjectionMatrix.perspective(10.0f, viewPortRatio, 3.0f, 100.0f);
depthProjectionViewMatrix = depthProjectionMatrix * depthViewMatrix;
@@ -912,7 +900,6 @@ void Bars3DRenderer::drawScene(GLuint defaultFboHandle)
}
#endif
- // TODO: Selection must be enabled currently to support clicked signal. (QTRD-2517)
// Skip selection mode drawing if we're slicing or have no selection mode
if (!m_cachedIsSlicingActivated && m_cachedSelectionMode > QAbstract3DGraph::SelectionNone
&& m_selectionState == SelectOnScene && seriesCount > 0) {
diff --git a/src/datavisualization/engine/drawer.cpp b/src/datavisualization/engine/drawer.cpp
index dde0def2..2fb8d81f 100644
--- a/src/datavisualization/engine/drawer.cpp
+++ b/src/datavisualization/engine/drawer.cpp
@@ -210,12 +210,10 @@ void Drawer::drawLabel(const AbstractRenderItem &item, const LabelItem &labelIte
break;
}
case LabelMid: {
- // Use this for positioning with absolute item y position value
yPosition = item.translation().y();
break;
}
case LabelHigh: {
- // TODO: Fix this. Can't seem to get it right (if ok with positive-only bars, doesn't look good on +- and vice versa)
yPosition = item.translation().y() + itemHeight / 2.0f;
break;
}
diff --git a/src/datavisualization/engine/q3dscene.cpp b/src/datavisualization/engine/q3dscene.cpp
index de19e84c..d2cf9c58 100644
--- a/src/datavisualization/engine/q3dscene.cpp
+++ b/src/datavisualization/engine/q3dscene.cpp
@@ -205,7 +205,6 @@ void Q3DScene::setPrimarySubViewport(const QRect &primarySubViewport)
*/
bool Q3DScene::isPointInPrimarySubView(const QPoint &point)
{
- // TODO: Needs fixing. Doesn't respect whether slice or main view is on top or if slicing is even activated currently.
int x = point.x();
int y = point.y();
int areaMinX = d_ptr->m_primarySubViewport.x();
@@ -224,7 +223,6 @@ bool Q3DScene::isPointInPrimarySubView(const QPoint &point)
*/
bool Q3DScene::isPointInSecondarySubView(const QPoint &point)
{
- // TODO: Needs fixing. Doesn't respect whether slice or main view is on top or if slicing is even activated currently.
int x = point.x();
int y = point.y();
int areaMinX = d_ptr->m_secondarySubViewport.x();
diff --git a/src/datavisualization/engine/scatter3dcontroller.cpp b/src/datavisualization/engine/scatter3dcontroller.cpp
index 6aa10a11..8a57af26 100644
--- a/src/datavisualization/engine/scatter3dcontroller.cpp
+++ b/src/datavisualization/engine/scatter3dcontroller.cpp
@@ -132,7 +132,6 @@ void Scatter3DController::handleItemsAdded(int startIndex, int count)
{
Q_UNUSED(startIndex)
Q_UNUSED(count)
- // TODO should dirty only affected values?
QScatter3DSeries *series = static_cast<QScatterDataProxy *>(sender())->series();
if (series->isVisible()) {
adjustValueAxisRange();
@@ -145,7 +144,6 @@ void Scatter3DController::handleItemsChanged(int startIndex, int count)
{
Q_UNUSED(startIndex)
Q_UNUSED(count)
- // TODO should dirty only affected values?
QScatter3DSeries *series = static_cast<QScatterDataProxy *>(sender())->series();
if (series->isVisible()) {
adjustValueAxisRange();
@@ -158,7 +156,6 @@ void Scatter3DController::handleItemsRemoved(int startIndex, int count)
{
Q_UNUSED(startIndex)
Q_UNUSED(count)
- // TODO should dirty only affected values?
QScatter3DSeries *series = static_cast<QScatterDataProxy *>(sender())->series();
if (series == m_selectedItemSeries) {
// If items removed from selected series before the selection, adjust the selection
@@ -190,7 +187,6 @@ void Scatter3DController::handleItemsInserted(int startIndex, int count)
{
Q_UNUSED(startIndex)
Q_UNUSED(count)
- // TODO should dirty only affected values?
QScatter3DSeries *series = static_cast<QScatterDataProxy *>(sender())->series();
if (series == m_selectedItemSeries) {
// If items inserted to selected series before the selection, adjust the selection
diff --git a/src/datavisualization/engine/scatter3drenderer.cpp b/src/datavisualization/engine/scatter3drenderer.cpp
index 047b0aad..073644db 100644
--- a/src/datavisualization/engine/scatter3drenderer.cpp
+++ b/src/datavisualization/engine/scatter3drenderer.cpp
@@ -224,7 +224,6 @@ void Scatter3DRenderer::updateData()
void Scatter3DRenderer::updateScene(Q3DScene *scene)
{
- // TODO: See QTRD-2374
scene->activeCamera()->setMinYRotation(-90.0f);
if (m_hasHeightAdjustmentChanged) {
@@ -357,18 +356,12 @@ void Scatter3DRenderer::drawScene(const GLuint defaultFboHandle)
QVector3D depthLightPos = activeCamera->calculatePositionRelativeToCamera(
zeroVector, 0.0f, 2.5f / m_autoScaleAdjustment);
depthViewMatrix.lookAt(depthLightPos, zeroVector, upVector);
- // TODO: Why does depthViewMatrix.column(3).y() goes to zero when we're directly above?
- // That causes the scene to be not drawn from above -> must be fixed
- // qDebug() << lightPos << depthViewMatrix << depthViewMatrix.column(3);
// Set the depth projection matrix
#ifndef USE_WIDER_SHADOWS
// Use this for perspective shadows
depthProjectionMatrix.perspective(15.0f, viewPortRatio, 3.0f, 100.0f);
#else
// Use these for orthographic shadows
- //depthProjectionMatrix.ortho(-aspectRatio * 2.0f, aspectRatio * 2.0f,
- // -m_heightNormalizer * 2.0f, m_heightNormalizer * 2.0f,
- // 0.0f, 100.0f);
GLfloat testAspectRatio = viewPortRatio;
depthProjectionMatrix.ortho(-testAspectRatio * 2.0f, testAspectRatio * 2.0f,
-m_heightNormalizer * 2.0f, m_heightNormalizer * 2.0f,
@@ -489,7 +482,6 @@ void Scatter3DRenderer::drawScene(const GLuint defaultFboHandle)
ObjectHelper *dotObj = m_visibleSeriesList.at(series).object();
bool drawingPoints = (m_visibleSeriesList.at(series).mesh() == QAbstract3DSeries::MeshPoint);
- // TODO: Accessing series directly during rendering
float itemSize =
static_cast<QScatter3DSeries *>(m_visibleSeriesList.at(series).series())->itemSize()
/ itemScaler;
@@ -643,7 +635,6 @@ void Scatter3DRenderer::drawScene(const GLuint defaultFboHandle)
bool colorStyleIsUniform = (colorStyle == Q3DTheme::ColorStyleUniform);
bool useColor = colorStyleIsUniform || drawingPoints;
- // TODO: Accessing series directly during rendering
float itemSize =
static_cast<QScatter3DSeries *>(currentSeries.series())->itemSize()
/ itemScaler;
@@ -1267,8 +1258,6 @@ void Scatter3DRenderer::drawScene(const GLuint defaultFboHandle)
}
// Draw axis labels
- // TODO: Calculations done temporarily here. Should be done when calculating lines to avoid
- // extra for -loops?
// Bind label shader
m_labelShader->bind();
diff --git a/src/datavisualization/engine/seriesrendercache.cpp b/src/datavisualization/engine/seriesrendercache.cpp
index 2f055d46..f1165cb7 100644
--- a/src/datavisualization/engine/seriesrendercache.cpp
+++ b/src/datavisualization/engine/seriesrendercache.cpp
@@ -117,7 +117,6 @@ void SeriesRenderCache::populate(QAbstract3DSeries *series, Abstract3DRenderer *
renderer->fixMeshFileName(meshFileName, m_mesh);
}
- // TODO: Optimize by having some kind of object cache in renderer instead of having separate ObjectHelper for each series?
delete m_object;
if (meshFileName.isEmpty()) {
m_object = 0;
diff --git a/src/datavisualization/engine/surface3dcontroller.cpp b/src/datavisualization/engine/surface3dcontroller.cpp
index f0dac44b..979d615a 100644
--- a/src/datavisualization/engine/surface3dcontroller.cpp
+++ b/src/datavisualization/engine/surface3dcontroller.cpp
@@ -122,8 +122,6 @@ void Surface3DController::handlePendingClick()
QPoint position = m_renderer->clickedPosition();
QSurface3DSeries *series = static_cast<QSurface3DSeries *>(m_renderer->clickedSeries());
- // TODO: Adjust position according to inserts/removes in the series
-
setSelectedPoint(position, series);
m_renderer->resetClickedStatus();
@@ -245,8 +243,6 @@ void Surface3DController::setSelectedPoint(const QPoint &position, QSurface3DSer
scene()->setSlicingActive(false);
} else {
// If the selected point is outside data window, or there is no selected point, disable slicing
- // TODO: (QTRD-2351) This logic doesn't match the renderer logic for non straight surfaces,
- // but that logic needs to change anyway, so this is good for now.
float axisMinX = m_axisX->min();
float axisMaxX = m_axisX->max();
float axisMinZ = m_axisZ->min();
diff --git a/src/datavisualization/engine/surface3drenderer.cpp b/src/datavisualization/engine/surface3drenderer.cpp
index 135e08e5..2cf56103 100644
--- a/src/datavisualization/engine/surface3drenderer.cpp
+++ b/src/datavisualization/engine/surface3drenderer.cpp
@@ -42,9 +42,6 @@ QT_BEGIN_NAMESPACE_DATAVISUALIZATION
//#define SHOW_DEPTH_TEXTURE_SCENE
-// TODO Uniform scaling is broken on surface
-//#define USE_UNIFORM_SCALING // Scale x and z uniformly, or based on autoscaled values
-
const GLfloat aspectRatio = 2.0f; // Forced ratio of x and z to y. Dynamic will make it look odd.
const GLfloat backgroundMargin = 1.1f; // Margin for background (1.1f = make it 10% larger to avoid items being drawn inside background)
const GLfloat labelMargin = 0.05f;
@@ -227,7 +224,6 @@ void Surface3DRenderer::updateData()
m_dataArray.clear();
}
- // TODO: Handle partial surface grids on the graph edges
if (sampleSpace.width() >= 2 && sampleSpace.height() >= 2) {
if (dimensionChanged) {
m_dataArray.reserve(sampleSpace.height());
@@ -281,8 +277,6 @@ void Surface3DRenderer::updateSeries(const QList<QAbstract3DSeries *> &seriesLis
{
Abstract3DRenderer::updateSeries(seriesList, updateVisibility);
- // TODO: move to render cache when multiseries support implemented QTRD-2657
- // TODO: until then just update them always.
if (m_visibleSeriesList.size()) {
QSurface3DSeries *series = static_cast<QSurface3DSeries *>(m_visibleSeriesList.at(0).series());
updateFlatStatus(series->isFlatShadingEnabled());
@@ -348,8 +342,6 @@ void Surface3DRenderer::updateRows(const QVector<int> &rows)
void Surface3DRenderer::updateItem(const QVector<QPoint> &points)
{
- // TODO: Properly support non-straight rows and columns (QTRD-2643)
-
// Surface only supports single series for now, so we are only interested in the first series
const QSurfaceDataArray *array = 0;
if (m_visibleSeriesList.size()) {
@@ -611,7 +603,6 @@ void Surface3DRenderer::drawSlicedScene()
QMatrix4x4 modelMatrix;
QMatrix4x4 itModelMatrix;
- // TODO: Do properly when multiseries support implemented QTRD-2657
const SeriesRenderCache &series = m_visibleSeriesList.at(0);
modelMatrix.translate(offset, 0.0f, 0.0f);
@@ -912,9 +903,6 @@ void Surface3DRenderer::drawScene(GLuint defaultFboHandle)
zeroVector, 0.0f, 3.5f / m_autoScaleAdjustment);
depthViewMatrix.lookAt(depthLightPos, zeroVector, upVector);
- // TODO: Why does depthViewMatrix.column(3).y() goes to zero when we're directly above?
- // That causes the scene to be not drawn from above -> must be fixed
- // qDebug() << lightPos << depthViewMatrix << depthViewMatrix.column(3);
// Set the depth projection matrix
#ifndef USE_WIDER_SHADOWS
// Use this for perspective shadows
@@ -1096,7 +1084,6 @@ void Surface3DRenderer::drawScene(GLuint defaultFboHandle)
m_cachedTheme->ambientLightStrength());
m_surfaceShader->setUniformValue(m_surfaceShader->lightColor(), lightColor);
- // TODO: Do properly when multiseries support implemented QTRD-2657
GLuint gradientTexture;
if (m_visibleSeriesList.at(0).colorStyle() == Q3DTheme::ColorStyleUniform)
gradientTexture = m_uniformGradientTexture;
@@ -1725,7 +1712,6 @@ void Surface3DRenderer::drawScene(GLuint defaultFboHandle)
|| visiblePoint == Surface3DController::invalidSelectionPosition()) {
m_selectionActive = false;
} else {
- // TODO: Need separate selection ball for slice and main surface view QTRD-2515
if (m_cachedIsSlicingActivated)
updateSliceDataModel(visiblePoint);
if (m_cachedSelectionMode.testFlag(QAbstract3DGraph::SelectionItem))
@@ -1831,17 +1817,10 @@ void Surface3DRenderer::calculateSceneScalingFactors()
m_areaSize.setHeight(m_axisCacheZ.max() - m_axisCacheZ.min());
m_areaSize.setWidth(m_axisCacheX.max() - m_axisCacheX.min());
m_scaleFactor = qMax(m_areaSize.width(), m_areaSize.height());
-#ifndef USE_UNIFORM_SCALING // Use this if we want to use autoscaling for x and z
m_scaleX = aspectRatio * m_areaSize.width() / m_scaleFactor;
m_scaleZ = aspectRatio * m_areaSize.height() / m_scaleFactor;
m_scaleXWithBackground = m_scaleX * backgroundMargin;
m_scaleZWithBackground = m_scaleZ * backgroundMargin;
-#else // ..and this if we want uniform scaling based on largest dimension
- m_scaleX = aspectRatio / m_scaleFactor;
- m_scaleZ = aspectRatio / m_scaleFactor;
- m_scaleXWithBackground = aspectRatio * backgroundMargin;
- m_scaleZWithBackground = aspectRatio * backgroundMargin;
-#endif
}
bool Surface3DRenderer::updateFlatStatus(bool enable)
@@ -1956,7 +1935,6 @@ void Surface3DRenderer::surfacePointSelected(const QPoint &point)
m_selectionPointer->setPosition(pos);
m_selectionPointer->setLabel(createSelectionLabel(value, column, row));
- // TODO: Get pointer object from correct series once multiseries support implemented
m_selectionPointer->setPointerObject(m_visibleSeriesList.at(0).object());
m_selectionPointer->setHighlightColor(m_visibleSeriesList.at(0).singleHighlightColor());
m_selectionPointer->updateScene(m_cachedScene);
@@ -1972,7 +1950,6 @@ QPoint Surface3DRenderer::selectionIdToSurfacePoint(uint id)
QString Surface3DRenderer::createSelectionLabel(float value, int column, int row)
{
- // TODO: Get from correct series once multiple series supported
QString labelText = m_visibleSeriesList[0].itemLabelFormat();
static const QString xTitleTag(QStringLiteral("@xTitle"));
static const QString yTitleTag(QStringLiteral("@yTitle"));
@@ -2010,7 +1987,6 @@ QString Surface3DRenderer::createSelectionLabel(float value, int column, int row
labelText.replace(zLabelTag, valueLabelText);
}
- // TODO: Get from correct series once multiple series supported
labelText.replace(seriesNameTag, m_visibleSeriesList[0].name());
m_selectionLabelDirty = false;
@@ -2203,7 +2179,6 @@ void Surface3DRenderer::updateDepthBuffer()
void Surface3DRenderer::generateUniformGradient(const QVector3D newColor)
{
if (m_visibleSeriesList.size()) {
- // TODO: move uniform gradient to render cache when multiseries support implemented QTRD-2657
QColor newQColor = Utils::colorFromVector(newColor);
m_uniformGradientTextureColor = newColor;
QLinearGradient newGradient;
diff --git a/src/datavisualization/input/qabstract3dinputhandler_p.h b/src/datavisualization/input/qabstract3dinputhandler_p.h
index 001040b8..4faffc5e 100644
--- a/src/datavisualization/input/qabstract3dinputhandler_p.h
+++ b/src/datavisualization/input/qabstract3dinputhandler_p.h
@@ -64,7 +64,6 @@ private:
QPoint m_inputPosition;
QRect m_mainViewPort;
- // TODO: Check if this could be avoided with signals/slots or some other way.
Q3DScene *m_scene;
bool m_isDefaultHandler;
diff --git a/src/datavisualization/theme/q3dtheme.cpp b/src/datavisualization/theme/q3dtheme.cpp
index 34c86108..1f9155b7 100644
--- a/src/datavisualization/theme/q3dtheme.cpp
+++ b/src/datavisualization/theme/q3dtheme.cpp
@@ -422,7 +422,6 @@ Q3DTheme::~Q3DTheme()
{
}
-// TODO: Add needRenders if necessary after color api has been added to series (QTRD-2200/2557)
/*!
* \property Q3DTheme::baseColors
*
diff --git a/src/datavisualization/utils/meshloader_p.h b/src/datavisualization/utils/meshloader_p.h
index e609c45e..be88dbdb 100644
--- a/src/datavisualization/utils/meshloader_p.h
+++ b/src/datavisualization/utils/meshloader_p.h
@@ -43,7 +43,6 @@ class MeshLoader
QVector<QVector3D> &out_vertices,
QVector<QVector2D> &out_uvs,
QVector<QVector3D> &out_normals);
- // TODO: add loaders for other formats?
};
QT_END_NAMESPACE_DATAVISUALIZATION
diff --git a/src/datavisualizationqml2/abstractdeclarative.cpp b/src/datavisualizationqml2/abstractdeclarative.cpp
index 093897be..e573b6f2 100644
--- a/src/datavisualizationqml2/abstractdeclarative.cpp
+++ b/src/datavisualizationqml2/abstractdeclarative.cpp
@@ -205,7 +205,6 @@ void AbstractDeclarative::render()
glClear(GL_COLOR_BUFFER_BIT);
}
- // TODO: Store the state of these and restore before returning
glDepthMask(GL_TRUE);
glEnable(GL_DEPTH_TEST);
glDepthFunc(GL_LESS);