From kde-commits Mon Nov 08 15:35:22 2010 From: Johannes Simon Date: Mon, 08 Nov 2010 15:35:22 +0000 To: kde-commits Subject: koffice/plugins/chartshape Message-Id: <20101108153522.969B1AC89B () svn ! kde ! org> X-MARC-Message: https://marc.info/?l=kde-commits&m=128923055607050 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.