[Libreoffice-commits] core.git: 6 commits - chart2/Library_chartcore.mk chart2/source include/vcl oox/source sc/source vcl/source
Markus Mohrhard
markus.mohrhard at collabora.co.uk
Fri Jun 6 19:54:08 PDT 2014
chart2/Library_chartcore.mk | 1
chart2/source/inc/ChartDebugTrace.hxx | 63 --------------------
chart2/source/model/main/ChartModel_Persistence.cxx | 6 -
chart2/source/model/template/ChartTypeTemplate.cxx | 10 ---
chart2/source/tools/ChartDebugTrace.cxx | 61 -------------------
include/vcl/metric.hxx | 4 -
oox/source/drawingml/shape.cxx | 3
sc/source/core/data/column.cxx | 4 -
sc/source/filter/oox/workbookhelper.cxx | 2
vcl/source/window/openglwin.cxx | 1
10 files changed, 8 insertions(+), 147 deletions(-)
New commits:
commit 9ea3e3a2fbe9c200a2b7fe11e48765d1d8389eb3
Author: Markus Mohrhard <markus.mohrhard at collabora.co.uk>
Date: Sat Jun 7 04:46:39 2014 +0200
column storage might be changed, related lp#990868
Change-Id: Ibd196484c8c5199e7398726a250082c266a89229
diff --git a/sc/source/core/data/column.cxx b/sc/source/core/data/column.cxx
index 7d6ebb1..f0dc03a 100644
--- a/sc/source/core/data/column.cxx
+++ b/sc/source/core/data/column.cxx
@@ -1709,7 +1709,6 @@ void ScColumn::CopyUpdated( const ScColumn& rPosCol, ScColumn& rDestCol ) const
void ScColumn::CopyScenarioFrom( const ScColumn& rSrcCol )
{
// This is the scenario table, the data is copied into it
- sc::CopyToDocContext aCxt(*pDocument);
ScAttrIterator aAttrIter( pAttrArray, 0, MAXROW );
SCROW nStart = -1, nEnd = -1;
const ScPatternAttr* pPattern = aAttrIter.Next( nStart, nEnd );
@@ -1718,6 +1717,7 @@ void ScColumn::CopyScenarioFrom( const ScColumn& rSrcCol )
if ( ((ScMergeFlagAttr&)pPattern->GetItem( ATTR_MERGE_FLAG )).IsScenario() )
{
DeleteArea( nStart, nEnd, IDF_CONTENTS );
+ sc::CopyToDocContext aCxt(*pDocument);
((ScColumn&)rSrcCol).
CopyToColumn(aCxt, nStart, nEnd, IDF_CONTENTS, false, *this);
@@ -1741,7 +1741,6 @@ void ScColumn::CopyScenarioFrom( const ScColumn& rSrcCol )
void ScColumn::CopyScenarioTo( ScColumn& rDestCol ) const
{
// This is the scenario table, the data is copied to the other
- sc::CopyToDocContext aCxt(*rDestCol.pDocument);
ScAttrIterator aAttrIter( pAttrArray, 0, MAXROW );
SCROW nStart = -1, nEnd = -1;
const ScPatternAttr* pPattern = aAttrIter.Next( nStart, nEnd );
@@ -1750,6 +1749,7 @@ void ScColumn::CopyScenarioTo( ScColumn& rDestCol ) const
if ( ((ScMergeFlagAttr&)pPattern->GetItem( ATTR_MERGE_FLAG )).IsScenario() )
{
rDestCol.DeleteArea( nStart, nEnd, IDF_CONTENTS );
+ sc::CopyToDocContext aCxt(*rDestCol.pDocument);
CopyToColumn(aCxt, nStart, nEnd, IDF_CONTENTS, false, rDestCol);
// UpdateUsed not needed, is already done in TestCopyScenario (obsolete comment ?)
commit 86f8772131a0588f3bfa4f09cc03746a2afe8cc8
Author: Markus Mohrhard <markus.mohrhard at collabora.co.uk>
Date: Sat Jun 7 03:56:56 2014 +0200
don't try to import invalid document, fdo#79135
Change-Id: I3b601d629550078a8fa353bf838619f8d7b34718
diff --git a/sc/source/filter/oox/workbookhelper.cxx b/sc/source/filter/oox/workbookhelper.cxx
index 4648c25..f59c478 100644
--- a/sc/source/filter/oox/workbookhelper.cxx
+++ b/sc/source/filter/oox/workbookhelper.cxx
@@ -433,6 +433,8 @@ ScRangeData* WorkbookGlobals::createLocalNamedRangeObject(
{
ScDocument& rDoc = getScDocument();
ScRangeName* pNames = rDoc.GetRangeName( nTab );
+ if(!pNames)
+ throw RuntimeException("invalid sheet index used");
// find an unused name
orName = findUnusedName( pNames, orName );
// create the named range
commit 127b11cbf944d51640c45801693b93ab3b18b14b
Author: Markus Mohrhard <markus.mohrhard at collabora.co.uk>
Date: Sat Jun 7 03:43:47 2014 +0200
make sure all objects exist, related fdo#75692
fdo75692-2.xlsx and fdo#75692-3.xlsx crash.
Change-Id: I56353e7da1850a49e18d3a570641843600d34b1c
diff --git a/oox/source/drawingml/shape.cxx b/oox/source/drawingml/shape.cxx
index 8c97b87..0737830 100644
--- a/oox/source/drawingml/shape.cxx
+++ b/oox/source/drawingml/shape.cxx
@@ -1249,7 +1249,8 @@ void Shape::finalizeXShape( XmlFilterBase& rFilter, const Reference< XShapes >&
{
Reference< chart2::data::XDataReceiver > xDataRec( xChartDoc, UNO_QUERY );
Reference< chart2::data::XDataSource > xData( xDataRec->getUsedData(), UNO_QUERY );
- if( xData->getDataSequences().getLength() <= 0 || xData->getDataSequences()[0]->getValues()->getData().getLength() <= 0 )
+ if( xData->getDataSequences().getLength() <= 0 || !xData->getDataSequences()[0]->getValues().is() ||
+ xData->getDataSequences()[0]->getValues()->getData().getLength() <= 0 )
{
rFilter.useInternalChartDataTable( true );
rFilter.getChartConverter()->convertFromModel( rFilter, aModel, xChartDoc, xExternalPage, mxShape->getPosition(), mxShape->getSize() );
commit af8ebb607c5c4a37d549cf8a49be02fe7830c55a
Author: Markus Mohrhard <markus.mohrhard at collabora.co.uk>
Date: Thu Jun 5 06:26:37 2014 +0200
remove debug output
Change-Id: I7c56adf0cd91f6474145708d91988c974e1ea854
diff --git a/vcl/source/window/openglwin.cxx b/vcl/source/window/openglwin.cxx
index f7b3cf3..e0c2c59 100644
--- a/vcl/source/window/openglwin.cxx
+++ b/vcl/source/window/openglwin.cxx
@@ -72,7 +72,6 @@ void OpenGLWindow::MouseButtonUp( const MouseEvent& rMEvt )
if(aPoint == maStartPoint)
{
Color aColor = GetPixel(aPoint);
- SAL_WARN("vcl.opengl", aColor.GetColor());
mpRenderer->clickedAt(aPoint, rMEvt.GetButtons());
}
else
commit 4d12fecb5b3891a316d405a9e61280465172a319
Author: Markus Mohrhard <markus.mohrhard at collabora.co.uk>
Date: Wed Jun 4 23:37:40 2014 +0200
make that variable protected again
Change-Id: I17235397e490458c250165c36bec97bf1ce326d7
diff --git a/include/vcl/metric.hxx b/include/vcl/metric.hxx
index 160b9b3..a0ef288 100644
--- a/include/vcl/metric.hxx
+++ b/include/vcl/metric.hxx
@@ -34,9 +34,10 @@ namespace vcl {
// avoid problems where poppler's FontInfo is picked up
class VCL_DLLPUBLIC FontInfo : public Font
{
- friend class OutputDevice;
+ friend class ::OutputDevice;
protected:
+ ImplFontMetric* mpImplMetric; // Implementation
public:
FontInfo();
@@ -49,7 +50,6 @@ public:
bool operator==( const FontInfo& ) const;
bool operator!=( const FontInfo& rInfo ) const
{ return !operator==( rInfo ); }
- ImplFontMetric* mpImplMetric; // Implementation
};
}
commit 439550c9f1c848023b4b95266f9d78671c468ee3
Author: Markus Mohrhard <markus.mohrhard at collabora.co.uk>
Date: Wed Jun 4 23:37:06 2014 +0200
remove unnecessary and unused chart debug functionality
Change-Id: Ic8100b2afebe469709723b324394e08b7669d911
diff --git a/chart2/Library_chartcore.mk b/chart2/Library_chartcore.mk
index 97245ca..094762d 100644
--- a/chart2/Library_chartcore.mk
+++ b/chart2/Library_chartcore.mk
@@ -183,7 +183,6 @@ $(eval $(call gb_Library_add_exception_objects,chartcore,\
chart2/source/tools/BaseGFXHelper \
chart2/source/tools/CachedDataSequence \
chart2/source/tools/CharacterProperties \
- chart2/source/tools/ChartDebugTrace \
chart2/source/tools/ChartModelHelper \
chart2/source/tools/ChartTypeHelper \
chart2/source/tools/ChartViewHelper \
diff --git a/chart2/source/inc/ChartDebugTrace.hxx b/chart2/source/inc/ChartDebugTrace.hxx
deleted file mode 100644
index f8eb950..0000000
--- a/chart2/source/inc/ChartDebugTrace.hxx
+++ /dev/null
@@ -1,63 +0,0 @@
-/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
-/*
- * This file is part of the LibreOffice project.
- *
- * This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/.
- *
- * This file incorporates work covered by the following license notice:
- *
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed
- * with this work for additional information regarding copyright
- * ownership. The ASF licenses this file to you under the Apache
- * License, Version 2.0 (the "License"); you may not use this file
- * except in compliance with the License. You may obtain a copy of
- * the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-#ifndef INCLUDED_CHART2_SOURCE_INC_CHARTDEBUGTRACE_HXX
-#define INCLUDED_CHART2_SOURCE_INC_CHARTDEBUGTRACE_HXX
-
-#include <com/sun/star/chart2/XChartDocument.hpp>
-#include <com/sun/star/chart2/XDiagram.hpp>
-#include "charttoolsdllapi.hxx"
-
-#define CHART_TRACE_OSL_DEBUG_LEVEL 2
-
-namespace chart
-{
-namespace debug
-{
-
-#if OSL_DEBUG_LEVEL >= CHART_TRACE_OSL_DEBUG_LEVEL
-/** Does several OSL_TRACEs that show information about the given chart document
- (e.g. the chart-type manager, diagrams etc.)
-
- If OSL_DEBUG_LEVEL is less than CHART_TRACE_OSL_DEBUG_LEVEL this
- function does nothing.
- */
-OOO_DLLPUBLIC_CHARTTOOLS void ChartDebugTraceDocument(
- const ::com::sun::star::uno::Reference<
- ::com::sun::star::chart2::XChartDocument > & xDoc,
- int nIndent = 0 );
-
-/** Does several OSL_TRACEs that show information about the given diagram
- (e.g. coordinate systems, data series etc.)
-
- If OSL_DEBUG_LEVEL is less than CHART_TRACE_OSL_DEBUG_LEVEL this
- function does nothing.
- */
-OOO_DLLPUBLIC_CHARTTOOLS void ChartDebugTraceDiagram(
- const ::com::sun::star::uno::Reference<
- ::com::sun::star::chart2::XDiagram > & xDiagram,
- int nIndent = 0 );
-#endif
-
-} // namespace debug
-} // namespace chart
-
-// INCLUDED_CHART2_SOURCE_INC_CHARTDEBUGTRACE_HXX
-#endif
-
-/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/chart2/source/model/main/ChartModel_Persistence.cxx b/chart2/source/model/main/ChartModel_Persistence.cxx
index 80ddf7a..f976a30 100644
--- a/chart2/source/model/main/ChartModel_Persistence.cxx
+++ b/chart2/source/model/main/ChartModel_Persistence.cxx
@@ -19,7 +19,6 @@
#include "ChartModel.hxx"
#include "MediaDescriptorHelper.hxx"
-#include "ChartDebugTrace.hxx"
#include "macros.hxx"
#include "ChartViewHelper.hxx"
#include "ChartModelHelper.hxx"
@@ -480,11 +479,6 @@ void SAL_CALL ChartModel::initNew()
}
setModified( sal_False );
unlockControllers();
-
-#if OSL_DEBUG_LEVEL >= CHART_TRACE_OSL_DEBUG_LEVEL
- OSL_TRACE( "ChartModel::initNew: Showing ChartDocument structure" );
- ::chart::debug::ChartDebugTraceDocument( Reference< chart2::XChartDocument >( this ));
-#endif
}
void SAL_CALL ChartModel::load(
diff --git a/chart2/source/model/template/ChartTypeTemplate.cxx b/chart2/source/model/template/ChartTypeTemplate.cxx
index 1940305..8be651a 100644
--- a/chart2/source/model/template/ChartTypeTemplate.cxx
+++ b/chart2/source/model/template/ChartTypeTemplate.cxx
@@ -30,7 +30,6 @@
#include "AxisHelper.hxx"
#include "LegendHelper.hxx"
#include "DiagramHelper.hxx"
-#include "ChartDebugTrace.hxx"
#include "AxisIndexDefines.hxx"
#include <cppuhelper/component_context.hxx>
#include <com/sun/star/chart/ChartSolidType.hpp>
@@ -304,10 +303,6 @@ void SAL_CALL ChartTypeTemplate::changeDiagramData(
Reference< XDataSeriesContainer > xDSCnt( aChartTypes[i], uno::UNO_QUERY_THROW );
xDSCnt->setDataSeries( aSeriesSeq[i] );
}
-#if OSL_DEBUG_LEVEL >= CHART_TRACE_OSL_DEBUG_LEVEL
- OSL_TRACE( "ChartTypeTemplate::changeDiagramData: Showing Diagram structure" );
- ::chart::debug::ChartDebugTraceDiagram( xDiagram );
-#endif
}
catch( const uno::Exception & ex )
{
@@ -805,11 +800,6 @@ void ChartTypeTemplate::FillDiagram(
{
ASSERT_EXCEPTION( ex );
}
-
-#if OSL_DEBUG_LEVEL >= CHART_TRACE_OSL_DEBUG_LEVEL
- OSL_TRACE( "ChartTypeTemplate::FillDiagram: Showing Diagram structure" );
- ::chart::debug::ChartDebugTraceDiagram( xDiagram );
-#endif
}
void ChartTypeTemplate::createChartTypes(
diff --git a/chart2/source/tools/ChartDebugTrace.cxx b/chart2/source/tools/ChartDebugTrace.cxx
deleted file mode 100644
index 6c6ddd1..0000000
--- a/chart2/source/tools/ChartDebugTrace.cxx
+++ /dev/null
@@ -1,61 +0,0 @@
-/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
-/*
- * This file is part of the LibreOffice project.
- *
- * This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/.
- *
- * This file incorporates work covered by the following license notice:
- *
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed
- * with this work for additional information regarding copyright
- * ownership. The ASF licenses this file to you under the Apache
- * License, Version 2.0 (the "License"); you may not use this file
- * except in compliance with the License. You may obtain a copy of
- * the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-#include "ChartDebugTrace.hxx"
-#include "macros.hxx"
-#include <com/sun/star/lang/XMultiServiceFactory.hpp>
-#include <com/sun/star/chart2/AxisType.hpp>
-#include <com/sun/star/chart2/XCoordinateSystemContainer.hpp>
-#include <com/sun/star/chart2/XChartTypeContainer.hpp>
-#include <com/sun/star/chart2/XDataSeriesContainer.hpp>
-#include <com/sun/star/chart2/StackingDirection.hpp>
-#include <rtl/math.hxx>
-
-using namespace ::com::sun::star;
-using namespace ::com::sun::star::chart2;
-
-using ::com::sun::star::uno::Reference;
-using ::com::sun::star::uno::Sequence;
-
-namespace chart
-{
-namespace debug
-{
-
-#if OSL_DEBUG_LEVEL >= CHART_TRACE_OSL_DEBUG_LEVEL
-
-void ChartDebugTraceDocument(
- const Reference< XChartDocument > & /*xDoc*/,
- int /*nIndent*/ )
-{
-
-}
-
-void ChartDebugTraceDiagram(
- const Reference< XDiagram > & /*xDiagram*/,
- int /*nIndent*/ )
-{
-
-}
-#endif
-
-} // namespace debug
-} // namespace chart
-
-/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
More information about the Libreoffice-commits
mailing list