> Is this ok to call the Qt function with qreal, then copy this qreal
> into a double buffer. I do not think this will introduce to much
> overhead. Should I simply open a bug report instead ?

Yup that sounds totally fine. I've committed a fix to CVS head.

Utkarsh
Index: Qt/Components/pqChartSeriesEditorModel.cxx
===================================================================
RCS file: /cvsroot/ParaView3/ParaView3/Qt/Components/pqChartSeriesEditorModel.cxx,v
retrieving revision 1.8
diff -u -3 -p -r1.8 pqChartSeriesEditorModel.cxx
--- Qt/Components/pqChartSeriesEditorModel.cxx	31 Jul 2009 15:02:53 -0000	1.8
+++ Qt/Components/pqChartSeriesEditorModel.cxx	25 Jan 2010 14:19:46 -0000
@@ -293,7 +293,11 @@ void pqChartSeriesEditorModel::setSeries
   if (row >= 0 && row < this->rowCount(QModelIndex()))
     {
     double double_color[3];
-    color.getRgbF(double_color, double_color+1, double_color+2);
+    qreal qreal_color[3];
+    color.getRgbF(qreal_color, qreal_color+1, qreal_color+2);
+    double_color[0] = static_cast<double>(qreal_color[0]);
+    double_color[1] = static_cast<double>(qreal_color[1]);
+    double_color[2] = static_cast<double>(qreal_color[2]);
     vtkSMPropertyHelper(this->RepresentationProxy,
       "SeriesColor").SetStatus(
       this->getSeriesName(row), double_color, 3);
_______________________________________________
Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the ParaView Wiki at: 
http://paraview.org/Wiki/ParaView

Follow this link to subscribe/unsubscribe:
http://www.paraview.org/mailman/listinfo/paraview

Reply via email to