[Kst] [Bug 95022] symbols should not use the same brush as lines

Andrew Walker arwalker at sumusltd.com
Mon Dec 13 02:23:42 CET 2004


------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
        
http://bugs.kde.org/show_bug.cgi?id=95022        
arwalker sumusltd com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From arwalker sumusltd com  2004-12-13 02:23 -------
CVS commit by arwalker: 

Reset line style after use.
Remove unused variable.

CCMAIL: 95022-done bugs kde org


  M +7 -7      kst2dplot.cpp   1.314


--- kdeextragear-2/kst/kst/kst2dplot.cpp  #1.313:1.314
 @ -5066,5 +5066,4  @ void Kst2DPlot::plotCurves(QPainter& p,
   KstBaseCurvePtr c;
   QRegion clipRegion;
-  QRect clipRect;
   double maxY = 0.0, minY = 0.0;
   double rX, rY, rEX, rEY;
 @ -5078,6 +5077,5  @ void Kst2DPlot::plotCurves(QPainter& p,
   clipRegion = p.clipRegion();
   clipping = p.hasClipping();
-  p.setClipRegion(QRegion(int(Lx), int(Ly), int(Hx-Lx), int(Hy-Ly), 
-                          QRegion::Rectangle));
+  p.setClipRegion(QRegion(int(Lx), int(Ly), int(Hx-Lx), int(Hy-Ly), QRegion::Rectangle));
 
   for (int i_curve = 0; i_curve < (int)Curves.count(); i_curve++) {
 @ -5104,8 +5102,4  @ void Kst2DPlot::plotCurves(QPainter& p,
       }
 
-      p.setPen(QPen(c->color(),
-                    (c->lineWidth() < penWidth ? penWidth : c->lineWidth()),
-                    style));
-
       if (c->hasLines()) {
         QPointArray points(MAX_NUM_POLYLINES);
 @ -5116,4 +5110,8  @ void Kst2DPlot::plotCurves(QPainter& p,
         int i0Start = i0;
 
+        p.setPen(QPen(c->color(),
+                    (c->lineWidth() < penWidth ? penWidth : c->lineWidth()),
+                    style));
+
 // Optimize - isnan seems expensive, at least in gcc debug mode
 //            cachegrind backs this up.
 @ -5365,4 +5363,6  @ void Kst2DPlot::plotCurves(QPainter& p,
           overlap = false;
         }
+
+        p.setPen(QPen(c->color(), (c->lineWidth() < penWidth ? penWidth : c->lineWidth())));
       } // end if c->hasLines()



More information about the Kst mailing list