[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    koffice/plugins/chartshape
From:       Johannes Simon <johannes.simon () gmail ! com>
Date:       2010-11-08 15:35:22
Message-ID: 20101108153522.969B1AC89B () svn ! kde ! org
[Download RAW message or body]

SVN commit 1194297 by jsimon:

Follow-up to boud's warning fixes

 M  +3 -7      PlotArea.cpp  


--- trunk/koffice/plugins/chartshape/PlotArea.cpp #1194296:1194297
@@ -77,7 +77,6 @@
 #include "ChartProxyModel.h"
 #include "ScreenConversions.h"
 #include "Layout.h"
-#include "OdfLoadingHelper.h"
 
 using namespace KChart;
 
@@ -236,10 +235,9 @@
             automaticallyHiddenAxisTitles.removeAll( axis->title() );
         delete axis;
     }
-    // There need to be at least these two axes. Do not delete, but
-    // hide them instead.
-    Axis *xAxis = new Axis( q, XAxisDimension );
-    Q_UNUSED(xAxis);
+    // There need to be at least these two axes. Their constructor will
+    // automatically add them to the plot area as child shape.
+    new Axis( q, XAxisDimension );
     Axis *yAxis = new Axis( q, YAxisDimension );
     yAxis->setShowMajorGrid( true );
 }
@@ -607,8 +605,6 @@
 
     loadOdfAttributes( plotAreaElement, context, OdfAllAttributes );
 
-//    OdfLoadingHelper *helper = (OdfLoadingHelper*)context.sharedData( OdfLoadingHelperId );
-
     // First step is to load the axis. Datasets are attached to an
     // axis and we need the axis to check for categories.
 
[prev in list] [next in list] [prev in thread] [next in thread]