[Libreoffice-commits] .: xmloff/source

2013-02-01 Thread Radek Doulik
 xmloff/source/draw/shapeexport4.cxx|   24 
 xmloff/source/draw/ximpcustomshape.cxx |   28 
 2 files changed, 52 insertions(+)

New commits:
commit 143ecebc73e84dfcafdc2577f811152b4a415fde
Author: Radek Doulik r...@novell.com
Date:   Fri Feb 1 12:25:23 2013 +0100

export darken/darkeless/lighten/lightenless commands

Change-Id: I4531ab82a52e3da41f12409185f7c0730381aa23

diff --git a/xmloff/source/draw/shapeexport4.cxx 
b/xmloff/source/draw/shapeexport4.cxx
index 1bb5481..ce504fe 100644
--- a/xmloff/source/draw/shapeexport4.cxx
+++ b/xmloff/source/draw/shapeexport4.cxx
@@ -413,6 +413,30 @@ void ImpExportEnhancedPath( SvXMLExport rExport,
 i += 2;
 }
 break;
+case 
com::sun::star::drawing::EnhancedCustomShapeSegmentCommand::DARKEN :
+if ( bExtended )
+aStrBuffer.append( (sal_Unicode)'H' );
+else
+bNeedExtended = true;
+break;
+case 
com::sun::star::drawing::EnhancedCustomShapeSegmentCommand::DARKENLESS :
+if ( bExtended )
+aStrBuffer.append( (sal_Unicode)'I' );
+else
+bNeedExtended = true;
+break;
+case 
com::sun::star::drawing::EnhancedCustomShapeSegmentCommand::LIGHTEN :
+if ( bExtended )
+aStrBuffer.append( (sal_Unicode)'J' );
+else
+bNeedExtended = true;
+break;
+case 
com::sun::star::drawing::EnhancedCustomShapeSegmentCommand::LIGHTENLESS :
+if ( bExtended )
+aStrBuffer.append( (sal_Unicode)'K' );
+else
+bNeedExtended = true;
+break;
 default : // ups, seems to be something wrong
 {
 aSegment.Count = 1;
diff --git a/xmloff/source/draw/ximpcustomshape.cxx 
b/xmloff/source/draw/ximpcustomshape.cxx
index a3406ed..1a280ef 100644
--- a/xmloff/source/draw/ximpcustomshape.cxx
+++ b/xmloff/source/draw/ximpcustomshape.cxx
@@ -702,6 +702,34 @@ void GetEnhancedPath( std::vector 
com::sun::star::beans::PropertyValue  rDest
 nIndex++;
 }
 break;
+case 'H' :
+{
+nLatestSegmentCommand = 
com::sun::star::drawing::EnhancedCustomShapeSegmentCommand::DARKEN;
+nParametersNeeded = 0;
+nIndex++;
+}
+break;
+case 'I' :
+{
+nLatestSegmentCommand = 
com::sun::star::drawing::EnhancedCustomShapeSegmentCommand::DARKENLESS;
+nParametersNeeded = 0;
+nIndex++;
+}
+break;
+case 'J' :
+{
+nLatestSegmentCommand = 
com::sun::star::drawing::EnhancedCustomShapeSegmentCommand::LIGHTEN;
+nParametersNeeded = 0;
+nIndex++;
+}
+break;
+case 'K' :
+{
+nLatestSegmentCommand = 
com::sun::star::drawing::EnhancedCustomShapeSegmentCommand::LIGHTENLESS;
+nParametersNeeded = 0;
+nIndex++;
+}
+break;
 case 'W' :
 {
 nLatestSegmentCommand = 
com::sun::star::drawing::EnhancedCustomShapeSegmentCommand::CLOCKWISEARCTO;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2013-01-29 Thread Libreoffice Gerrit user
 xmloff/source/xforms/XFormsBindContext.cxx   |   18 ++---
 xmloff/source/xforms/XFormsModelContext.cxx  |2 -
 xmloff/source/xforms/XFormsSubmissionContext.cxx |   30 +++
 xmloff/source/xforms/xformsapi.cxx   |4 +--
 xmloff/source/xforms/xformsapi.hxx   |3 --
 xmloff/source/xforms/xformsexport.cxx|   28 ++---
 6 files changed, 41 insertions(+), 44 deletions(-)

New commits:
commit c4770788cba4bc3658285d955f5753e437b88998
Author: Miklos Vajna vmik...@suse.cz
Date:   Tue Jan 29 09:43:53 2013 +0100

xmloff: no longer necessary OUTSTRING() macro

Change-Id: I60f588efb35d5968c21b7b0e0fc29bcbbd2dd50d

diff --git a/xmloff/source/xforms/XFormsBindContext.cxx 
b/xmloff/source/xforms/XFormsBindContext.cxx
index 3929317..eee251b 100644
--- a/xmloff/source/xforms/XFormsBindContext.cxx
+++ b/xmloff/source/xforms/XFormsBindContext.cxx
@@ -90,28 +90,28 @@ void XFormsBindContext::HandleAttribute( sal_uInt16 nToken,
 switch( nToken )
 {
 case XML_NODESET:
-xforms_setValue( mxBinding, OUSTRING(BindingExpression), rValue );
+xforms_setValue( mxBinding, BindingExpression, rValue );
 break;
 case XML_ID:
-xforms_setValue( mxBinding, OUSTRING(BindingID), rValue );
+xforms_setValue( mxBinding, BindingID, rValue );
 break;
 case XML_READONLY:
-xforms_setValue( mxBinding, OUSTRING(ReadonlyExpression), rValue );
+xforms_setValue( mxBinding, ReadonlyExpression, rValue );
 break;
 case XML_RELEVANT:
-xforms_setValue( mxBinding, OUSTRING(RelevantExpression), rValue );
+xforms_setValue( mxBinding, RelevantExpression, rValue );
 break;
 case XML_REQUIRED:
-xforms_setValue( mxBinding, OUSTRING(RequiredExpression), rValue );
+xforms_setValue( mxBinding, RequiredExpression, rValue );
 break;
 case XML_CONSTRAINT:
-xforms_setValue( mxBinding, OUSTRING(ConstraintExpression), rValue );
+xforms_setValue( mxBinding, ConstraintExpression, rValue );
 break;
 case XML_CALCULATE:
-xforms_setValue( mxBinding, OUSTRING(CalculateExpression), rValue );
+xforms_setValue( mxBinding, CalculateExpression, rValue );
 break;
 case XML_TYPE:
-xforms_setValue( mxBinding, OUSTRING(Type),
+xforms_setValue( mxBinding, Type,
   makeAny( xforms_getTypeName( 
mxModel-getDataTypeRepository(),
GetImport().GetNamespaceMap(),
rValue ) ) );
@@ -127,7 +127,7 @@ void XFormsBindContext::StartElement(
 {
 // we need to register the namespaces
 ReferenceXNameContainer xContainer(
-mxBinding-getPropertyValue( OUSTRING(BindingNamespaces) ),
+mxBinding-getPropertyValue( BindingNamespaces ),
 UNO_QUERY );
 
 DBG_ASSERT( xContainer.is(), binding should have a namespace container );
diff --git a/xmloff/source/xforms/XFormsModelContext.cxx 
b/xmloff/source/xforms/XFormsModelContext.cxx
index 0a09c23..3a47de5 100644
--- a/xmloff/source/xforms/XFormsModelContext.cxx
+++ b/xmloff/source/xforms/XFormsModelContext.cxx
@@ -91,7 +91,7 @@ void XFormsModelContext::HandleAttribute(
 switch( nToken )
 {
 case XML_ID:
-mxModel-setPropertyValue( OUSTRING(ID), makeAny( rValue ) );
+mxModel-setPropertyValue( ID, makeAny( rValue ) );
 break;
 case XML_SCHEMA:
 GetImport().SetError( XMLERROR_XFORMS_NO_SCHEMA_SUPPORT );
diff --git a/xmloff/source/xforms/XFormsSubmissionContext.cxx 
b/xmloff/source/xforms/XFormsSubmissionContext.cxx
index b783427..7ad50a6 100644
--- a/xmloff/source/xforms/XFormsSubmissionContext.cxx
+++ b/xmloff/source/xforms/XFormsSubmissionContext.cxx
@@ -106,50 +106,50 @@ void XFormsSubmissionContext::HandleAttribute( sal_uInt16 
nToken,
 switch( nToken )
 {
 case XML_ID:
-xforms_setValue( mxSubmission, OUSTRING(ID), rValue );
+xforms_setValue( mxSubmission, ID, rValue );
 break;
 case XML_BIND:
-xforms_setValue( mxSubmission, OUSTRING(Bind), rValue );
+xforms_setValue( mxSubmission, Bind, rValue );
 break;
 case XML_REF:
-xforms_setValue( mxSubmission, OUSTRING(Ref), rValue );
+xforms_setValue( mxSubmission, Ref, rValue );
 break;
 case XML_ACTION:
-xforms_setValue( mxSubmission, OUSTRING(Action), rValue );
+xforms_setValue( mxSubmission, Action, rValue );
 break;
 case XML_METHOD:
-xforms_setValue( mxSubmission, OUSTRING(Method), rValue );
+xforms_setValue( mxSubmission, Method, rValue );
 break;
 case XML_VERSION:
-xforms_setValue( mxSubmission, OUSTRING(Version), rValue );
+xforms_setValue( mxSubmission, Version, rValue );
 break;
 case XML_INDENT:
-xforms_setValue( mxSubmission, 

[Libreoffice-commits] .: xmloff/source

2013-01-24 Thread Libreoffice Gerrit user
 xmloff/source/chart/SchXMLTools.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit c1760a241b3f0362338d09460ac7dd2e1f2b25aa
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Jan 24 13:27:17 2013 +0200

WaE: 'bVal' may be used uninitialized in this function

Seen in the MacOSX-Intel_1-built_no-moz_on_10.6.8 tinderbox build log,
not in an own build.

Change-Id: I1b5c87d3c1876821981d8f8a7b089b81659e5c7b

diff --git a/xmloff/source/chart/SchXMLTools.cxx 
b/xmloff/source/chart/SchXMLTools.cxx
index 2e469a4..5752fad 100644
--- a/xmloff/source/chart/SchXMLTools.cxx
+++ b/xmloff/source/chart/SchXMLTools.cxx
@@ -383,7 +383,7 @@ Reference chart2::data::XDataSequence  CreateDataSequence(
 {
 try
 {
-sal_Bool bVal;
+sal_Bool bVal = sal_False;
 uno::Any any = 
xPropSet-getPropertyValue(UseInternalDataProvider);
 if (any = bVal)
 bUseInternal = static_castbool(bVal);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2013-01-17 Thread Libreoffice Gerrit user
 xmloff/source/text/txtexppr.cxx |8 
 xmloff/source/text/txtimppr.cxx |   23 ++-
 2 files changed, 30 insertions(+), 1 deletion(-)

New commits:
commit 23c099a0236062724430efb986251637095c8d4f
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Jan 17 12:41:11 2013 +

Resolves: fdo#58730 workaround [UL|LR]Space 100% problem

If all paragraph margins are 100% on import, ignore that as being the 
implicit
default. That avoids explicit 100% being set onto the awesome [UL|LR]Space
which takes a relative propsize of 100% as a flag that its value field is
absolute and so rejected by SwTxtFmtColl::Modify as a candidate for getting 
its
true value initialized relative to its parent, so it ends up as an absolute 0

Always elide the property on export because writing individual
margin-foos provides better backward compatibility with older versions
anyway.

Trigged by 3c5facfce42a0dbe362d6b9fa5ac374fd76f51a1

Change-Id: I55f6ceeae287b7d8e99befa4bd3cc06738a21299

diff --git a/xmloff/source/text/txtexppr.cxx b/xmloff/source/text/txtexppr.cxx
index ceff1d0..56e36e7 100644
--- a/xmloff/source/text/txtexppr.cxx
+++ b/xmloff/source/text/txtexppr.cxx
@@ -455,6 +455,8 @@ void XMLTextExportPropertySetMapper::ContextFilter(
 XMLPropertyState* pClip11State = NULL;
 XMLPropertyState* pClipState = NULL;
 
+// filter fo:margin
+XMLPropertyState* pAllParaMarginRel = NULL;
 XMLPropertyState* pAllParaMargin = NULL;
 XMLPropertyState* pAllMargin = NULL;
 
@@ -572,6 +574,7 @@ void XMLTextExportPropertySetMapper::ContextFilter(
 case CTF_NUMBERINGSTYLENAME:pListStyleName = propertie; break;
 case CTF_TEXT_CLIP11:   pClip11State = propertie; break;
 case CTF_TEXT_CLIP: pClipState = propertie; break;
+case CTF_PARAMARGINALL_REL: pAllParaMarginRel = propertie; break;
 case CTF_PARAMARGINALL: pAllParaMargin = propertie; break;
 case CTF_MARGINALL: pAllMargin = propertie; break;
 }
@@ -625,6 +628,11 @@ void XMLTextExportPropertySetMapper::ContextFilter(
 lcl_checkMultiProperty(pParaBottomMarginState, pParaBottomMarginRelState);
 lcl_checkMultiProperty(pParaFirstLineState, pParaFirstLineRelState);
 
+if (pAllParaMarginRel)
+{   // because older OOo/LO versions can't read fo:margin:
+pAllParaMarginRel-mnIndex = -1; // just export individual 
attributes...
+pAllParaMarginRel-maValue.clear();
+}
 if (pAllParaMargin)
 {
 pAllParaMargin-mnIndex = -1; // just export individual attributes...
diff --git a/xmloff/source/text/txtimppr.cxx b/xmloff/source/text/txtimppr.cxx
index be8ef01..ef9758a 100644
--- a/xmloff/source/text/txtimppr.cxx
+++ b/xmloff/source/text/txtimppr.cxx
@@ -257,6 +257,26 @@ void XMLTextImportPropertyMapper::FontDefaultsCheck(
 }
 }
 
+//fdo#58730 The [UL|LR]Space class has a deficiency where 100% also serves as
+//a flag that the value is an absolute value so we can't truly handle an
+//up/lower space property which wants to specify its 200% upper but 100% lower
+//of its parent (try typing 100% vs 200% into the edit style dialog and revisit
+//your style). So on xml load that ends up meaning 200%, 0 lower. This is a
+//crock.
+//
+//On import clear 100% all-margins relative sizes.
+static bool
+isNotDefaultRelSize(const XMLPropertyState* pRelState, const 
UniReferenceXMLPropertySetMapper rPrMap)
+{
+if (rPrMap-GetEntryContextId(pRelState-mnIndex) == CTF_PARAMARGINALL_REL)
+{
+sal_Int32 nTemp = 0;
+pRelState-maValue = nTemp;
+return nTemp != 100;
+}
+return true;
+}
+
 void XMLTextImportPropertyMapper::finished(
 ::std::vector XMLPropertyState  rProperties,
 sal_Int32 /*nStartIndex*/, sal_Int32 /*nEndIndex*/ ) const
@@ -425,7 +445,8 @@ void XMLTextImportPropertyMapper::finished(
 
 for (sal_uInt16 i = 0; i  4; i++)
 {
-if (pAllParaMargin  !pParaMargins[i])
+if (pAllParaMargin  !pParaMargins[i]
+ isNotDefaultRelSize(pAllParaMargin, getPropertySetMapper()))
 {
 #if OSL_DEBUG_LEVEL  0
 sal_Int16 nTmp = getPropertySetMapper()-GetEntryContextId(
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2013-01-02 Thread Libreoffice Gerrit user
 xmloff/source/style/xmlnumfi.cxx |4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 5693934456f917861af7cb02734578b049a46940
Author: Markus Mohrhard markus.mohrh...@googlemail.com
Date:   Thu Jan 3 02:18:25 2013 +0100

limit the number of imported digits, fdo#58539

This should fix the crash with gnome#627420.

Change-Id: Ibfff498282dc1c6fe9124ced645392107ef8829f

diff --git a/xmloff/source/style/xmlnumfi.cxx b/xmloff/source/style/xmlnumfi.cxx
index 7b0799c..8b57734 100644
--- a/xmloff/source/style/xmlnumfi.cxx
+++ b/xmloff/source/style/xmlnumfi.cxx
@@ -916,6 +916,8 @@ static void lcl_EnquoteIfNecessary( rtl::OUStringBuffer 
rContent, const SvXMLNu
 //  SvXMLNumFmtElementContext
 //
 
+const sal_Int32 MAX_SECOND_DIGITS = 20; // fdo#58539  gnome#627420: limit 
number of digits during import
+
 SvXMLNumFmtElementContext::SvXMLNumFmtElementContext( SvXMLImport rImport,
 sal_uInt16 nPrfx, const rtl::OUString 
rLName,
 SvXMLNumFormatContext rParentContext, 
sal_uInt16 nNewType,
@@ -948,7 +950,7 @@ SvXMLNumFmtElementContext::SvXMLNumFmtElementContext( 
SvXMLImport rImport,
 {
 case XML_TOK_ELEM_ATTR_DECIMAL_PLACES:
 if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 ))
-aNumInfo.nDecimals = nAttrVal;
+aNumInfo.nDecimals = std::minsal_Int32(nAttrVal, 
MAX_SECOND_DIGITS);
 break;
 case XML_TOK_ELEM_ATTR_MIN_INTEGER_DIGITS:
 if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 ))
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-12-28 Thread Libreoffice Gerrit user
 xmloff/source/chart/SchXMLAxisContext.cxx  |2 -
 xmloff/source/chart/SchXMLChartContext.cxx |2 -
 xmloff/source/chart/SchXMLExport.cxx   |2 -
 xmloff/source/chart/SchXMLPlotAreaContext.cxx  |3 -
 xmloff/source/chart/SchXMLSeries2Context.cxx   |4 +-
 xmloff/source/chart/SchXMLTools.cxx|   50 -
 xmloff/source/draw/animationimport.cxx |2 -
 xmloff/source/draw/eventimp.cxx|4 +-
 xmloff/source/draw/shapeexport2.cxx|4 +-
 xmloff/source/forms/elementimport.cxx  |2 -
 xmloff/source/forms/propertyexport.cxx |3 -
 xmloff/source/style/xmlnumfi.cxx   |4 +-
 xmloff/source/text/txtflde.cxx |6 +--
 xmloff/source/transform/EventOASISTContext.cxx |4 +-
 xmloff/source/transform/TransformerBase.cxx|4 +-
 15 files changed, 47 insertions(+), 49 deletions(-)

New commits:
commit 4fd5136cbd1baa86361e84e3e3b146ba29d7d316
Author: Miklos Vajna vmik...@suse.cz
Date:   Fri Dec 28 13:36:36 2012 +0100

xmloff: mostly kill CONSTASCII_STRINGPARAM usage

diff --git a/xmloff/source/chart/SchXMLAxisContext.cxx 
b/xmloff/source/chart/SchXMLAxisContext.cxx
index 3ddf60b..b14618f 100644
--- a/xmloff/source/chart/SchXMLAxisContext.cxx
+++ b/xmloff/source/chart/SchXMLAxisContext.cxx
@@ -779,7 +779,7 @@ void SchXMLAxisContext::CorrectAxisPositions( const 
Reference chart2::XChartDoc
 if( xMainXAxisProp.is()  xMainYAxisProp.is() )
 {
 chart2::ScaleData aMainXScale = 
xMainXAxis-getScaleData();
-if( 0 == rChartTypeServiceName.reverseCompareToAsciiL( 
RTL_CONSTASCII_STRINGPARAM( com.sun.star.chart2.ScatterChartType ) ) )
+if( 0 == rChartTypeServiceName.reverseCompareTo( 
com.sun.star.chart2.ScatterChartType ) )
 {
 xMainYAxisProp-setPropertyValue( 
OUString(CrossoverPosition)
 , uno::makeAny( 
::com::sun::star::chart::ChartAxisPosition_VALUE) );
diff --git a/xmloff/source/chart/SchXMLChartContext.cxx 
b/xmloff/source/chart/SchXMLChartContext.cxx
index 608c342..0632b6b 100644
--- a/xmloff/source/chart/SchXMLChartContext.cxx
+++ b/xmloff/source/chart/SchXMLChartContext.cxx
@@ -862,7 +862,7 @@ void SchXMLChartContext::EndElement()
 bool bLinesOn = true;
 if( (maSeriesDefaultsAndStyles.maLinesOnProperty = bLinesOn)  
!bLinesOn )
 {
-if( 0 == maChartTypeServiceName.reverseCompareToAsciiL( 
RTL_CONSTASCII_STRINGPARAM( com.sun.star.chart2.ScatterChartType ) ) )
+if( 0 == maChartTypeServiceName.reverseCompareTo( 
com.sun.star.chart2.ScatterChartType ) )
 {
 bSwitchOffLinesForScatter = true;
 SchXMLSeries2Context::switchSeriesLinesOff( 
maSeriesDefaultsAndStyles.maSeriesStyleList );
diff --git a/xmloff/source/chart/SchXMLExport.cxx 
b/xmloff/source/chart/SchXMLExport.cxx
index a84f714..e15819b 100644
--- a/xmloff/source/chart/SchXMLExport.cxx
+++ b/xmloff/source/chart/SchXMLExport.cxx
@@ -2088,7 +2088,7 @@ void SchXMLExportHelper_Impl::exportPlotArea(
 
 // stock-chart elements
 OUString sChartType ( xDiagram-getDiagramType());
-if( 0 == sChartType.reverseCompareToAsciiL( RTL_CONSTASCII_STRINGPARAM( 
com.sun.star.chart.StockDiagram )))
+if( 0 == sChartType.reverseCompareTo( com.sun.star.chart.StockDiagram ))
 {
 Reference chart::XStatisticDisplay  xStockPropProvider( xDiagram, 
uno::UNO_QUERY );
 if( xStockPropProvider.is())
diff --git a/xmloff/source/chart/SchXMLPlotAreaContext.cxx 
b/xmloff/source/chart/SchXMLPlotAreaContext.cxx
index 0697e8e..8d10bf1 100644
--- a/xmloff/source/chart/SchXMLPlotAreaContext.cxx
+++ b/xmloff/source/chart/SchXMLPlotAreaContext.cxx
@@ -598,8 +598,7 @@ void SchXMLPlotAreaContext::EndElement()
 }
 
 // #i32366# stock has volume
-if( ( 0 == mxDiagram-getDiagramType().reverseCompareToAsciiL(
-  RTL_CONSTASCII_STRINGPARAM( 
com.sun.star.chart.StockDiagram ))) 
+if( ( 0 == 
mxDiagram-getDiagramType().reverseCompareTo(com.sun.star.chart.StockDiagram 
)) 
 mbStockHasVolume )
 {
 try
diff --git a/xmloff/source/chart/SchXMLSeries2Context.cxx 
b/xmloff/source/chart/SchXMLSeries2Context.cxx
index e3188b9..2764efc 100644
--- a/xmloff/source/chart/SchXMLSeries2Context.cxx
+++ b/xmloff/source/chart/SchXMLSeries2Context.cxx
@@ -275,9 +275,9 @@ SchXMLSeries2Context::SchXMLSeries2Context(
 mbSymbolSizeIsMissingInFile(false),
 maChartSize( rChartSize )
 {
-if( 0 == aGlobalChartTypeName.reverseCompareToAsciiL( 
RTL_CONSTASCII_STRINGPARAM( com.sun.star.chart2.DonutChartType ) ) )
+if( 0 == aGlobalChartTypeName.reverseCompareTo( 
com.sun.star.chart2.DonutChartType ) )
 {
-

[Libreoffice-commits] .: xmloff/source

2012-12-26 Thread Libreoffice Gerrit user
 xmloff/source/draw/shapeexport2.cxx |8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

New commits:
commit 9e41982ac165092ed024b8ab318eedc7fcf4c6a9
Author: Markus Mohrhard markus.mohrh...@googlemail.com
Date:   Wed Dec 26 19:57:34 2012 +0100

text:p is not allowed for chart objects, fdo#58571

Change-Id: I24c485d31ea3f0540164ff65074b392be1e557e6

diff --git a/xmloff/source/draw/shapeexport2.cxx 
b/xmloff/source/draw/shapeexport2.cxx
index d4f1e23..2ebd96a 100644
--- a/xmloff/source/draw/shapeexport2.cxx
+++ b/xmloff/source/draw/shapeexport2.cxx
@@ -1662,8 +1662,12 @@ void XMLShapeExport::ImpExportOLE2Shape(
 if( !sClassId.isEmpty() )
 mrExport.AddAttribute(XML_NAMESPACE_DRAW, XML_CLASS_ID, 
sClassId );
 
-// #i118485# Add text export, the draw OLE shape allows text 
now
-ImpExportText( xShape );
+if(eShapeType != XmlShapeTypePresChartShape  eShapeType != 
XmlShapeTypeDrawChartShape)
+{
+// #i118485# Add text export, the draw OLE shape allows 
text now
+// fdo#58571 chart objects don't allow text:p
+ImpExportText( xShape );
+}
 
 if(!bExportEmbedded)
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-12-08 Thread Libreoffice Gerrit user
 xmloff/source/style/xmlstyle.cxx |6 ++---
 xmloff/source/text/XMLLineNumberingImportContext.cxx |   22 +--
 2 files changed, 14 insertions(+), 14 deletions(-)

New commits:
commit da8af42a2d783588359d33f79d15b94fa0d91bb4
Author: Chr. Rossmanith chrrossman...@gmx.de
Date:   Sat Nov 24 16:34:13 2012 +0100

Removed RTL_CONSTASCII_USTRINGPARAM

Change-Id: Ifb14257ef20c14f8e1551a1470d0dbea1dc0759b

diff --git a/xmloff/source/style/xmlstyle.cxx b/xmloff/source/style/xmlstyle.cxx
index 433b140..b69f3f6 100644
--- a/xmloff/source/style/xmlstyle.cxx
+++ b/xmloff/source/style/xmlstyle.cxx
@@ -418,7 +418,7 @@ SvXMLStyleContext 
*SvXMLStylesContext::CreateStyleChildContext( sal_uInt16 p_nPr
 {
 const SvXMLTokenMap rTokenMap = GetStyleStylesElemTokenMap();
 sal_uInt16 nToken = rTokenMap.Get( p_nPrefix, rLocalName );
-switch( nToken  )
+switch( nToken )
 {
 case XML_TOK_STYLE_STYLE:
 case XML_TOK_STYLE_DEFAULT_STYLE:
@@ -804,8 +804,8 @@ SvXMLStylesContext::SvXMLStylesContext( SvXMLImport 
rImport, sal_uInt16 nPrfx,
 const OUString rLName,
 const uno::Reference 
xml::sax::XAttributeList  , sal_Bool bAuto ) :
 SvXMLImportContext( rImport, nPrfx, rLName ),
-msParaStyleServiceName( RTL_CONSTASCII_USTRINGPARAM( 
com.sun.star.style.ParagraphStyle ) ),
-msTextStyleServiceName( RTL_CONSTASCII_USTRINGPARAM( 
com.sun.star.style.CharacterStyle ) ),
+msParaStyleServiceName( com.sun.star.style.ParagraphStyle ),
+msTextStyleServiceName( com.sun.star.style.CharacterStyle ),
 mpImpl( new SvXMLStylesContext_Impl( bAuto ) ),
 mpStyleStylesElemTokenMap( 0 )
 {
diff --git a/xmloff/source/text/XMLLineNumberingImportContext.cxx 
b/xmloff/source/text/XMLLineNumberingImportContext.cxx
index 103e5d6..77dd3ba 100644
--- a/xmloff/source/text/XMLLineNumberingImportContext.cxx
+++ b/xmloff/source/text/XMLLineNumberingImportContext.cxx
@@ -51,17 +51,17 @@ 
XMLLineNumberingImportContext::XMLLineNumberingImportContext(
 const OUString rLocalName,
 const ReferenceXAttributeList  xAttrList)
 :   SvXMLStyleContext(rImport, nPrfx, rLocalName, xAttrList, 
XML_STYLE_FAMILY_TEXT_LINENUMBERINGCONFIG)
-,   sCharStyleName(RTL_CONSTASCII_USTRINGPARAM(CharStyleName))
-,   sCountEmptyLines(RTL_CONSTASCII_USTRINGPARAM(CountEmptyLines))
-,   sCountLinesInFrames(RTL_CONSTASCII_USTRINGPARAM(CountLinesInFrames))
-,   sDistance(RTL_CONSTASCII_USTRINGPARAM(Distance))
-,   sInterval(RTL_CONSTASCII_USTRINGPARAM(Interval))
-,   sSeparatorText(RTL_CONSTASCII_USTRINGPARAM(SeparatorText))
-,   sNumberPosition(RTL_CONSTASCII_USTRINGPARAM(NumberPosition))
-,   sNumberingType(RTL_CONSTASCII_USTRINGPARAM(NumberingType))
-,   sIsOn(RTL_CONSTASCII_USTRINGPARAM(IsOn))
-,   sRestartAtEachPage(RTL_CONSTASCII_USTRINGPARAM(RestartAtEachPage))
-,   sSeparatorInterval(RTL_CONSTASCII_USTRINGPARAM(SeparatorInterval))
+,   sCharStyleName(CharStyleName)
+,   sCountEmptyLines(CountEmptyLines)
+,   sCountLinesInFrames(CountLinesInFrames)
+,   sDistance(Distance)
+,   sInterval(Interval)
+,   sSeparatorText(SeparatorText)
+,   sNumberPosition(NumberPosition)
+,   sNumberingType(NumberingType)
+,   sIsOn(IsOn)
+,   sRestartAtEachPage(RestartAtEachPage)
+,   sSeparatorInterval(SeparatorInterval)
 ,   sNumFormat(GetXMLToken(XML_1))
 ,   sNumLetterSync(GetXMLToken(XML_FALSE))
 ,   nOffset(-1)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-12-08 Thread Libreoffice Gerrit user
 xmloff/source/core/DocumentSettingsContext.cxx|   42 +--
 xmloff/source/core/DomExport.cxx  |4 
 xmloff/source/core/SettingsExportHelper.cxx   |   54 ++--
 xmloff/source/core/XMLEmbeddedObjectImportContext.cxx |2 
 xmloff/source/core/xmlexp.cxx |   10 
 xmloff/source/core/xmlimp.cxx |  229 ++
 xmloff/source/core/xmlmultiimagehelper.cxx|   16 -
 7 files changed, 140 insertions(+), 217 deletions(-)

New commits:
commit 649c6b38911e9b77d21efba5fa1d6affa9fc4b0e
Author: Chr. Rossmanith chrrossman...@gmx.de
Date:   Tue Dec 4 14:35:40 2012 +0100

RTL_CONSTASCII_USTRINGPARAM clean up

Change-Id: I4d57a449c2bd61be0cb7b7ed9016b3ee71825b57

diff --git a/xmloff/source/core/DocumentSettingsContext.cxx 
b/xmloff/source/core/DocumentSettingsContext.cxx
index 0af9f1b..6ec025c 100644
--- a/xmloff/source/core/DocumentSettingsContext.cxx
+++ b/xmloff/source/core/DocumentSettingsContext.cxx
@@ -106,7 +106,7 @@ uno::Referencecontainer::XNameContainer 
XMLMyList::GetNameContainer()
 
 if( mxServiceFactory.is() )
 {
-rtl::OUString 
sName(RTL_CONSTASCII_USTRINGPARAM(com.sun.star.document.NamedPropertyValues));
+rtl::OUString sName(com.sun.star.document.NamedPropertyValues);
 xNameContainer = 
uno::Referencecontainer::XNameContainer(mxServiceFactory-createInstance(sName),
 uno::UNO_QUERY);
 if (xNameContainer.is())
 {
@@ -128,7 +128,7 @@ uno::Referencecontainer::XIndexContainer 
XMLMyList::GetIndexContainer()
 
 if( mxServiceFactory.is() )
 {
-rtl::OUString 
sName(RTL_CONSTASCII_USTRINGPARAM(com.sun.star.document.IndexedPropertyValues));
+rtl::OUString sName(com.sun.star.document.IndexedPropertyValues);
 xIndexContainer = 
uno::Referencecontainer::XIndexContainer(mxServiceFactory-createInstance(sName),
 uno::UNO_QUERY);
 if (xIndexContainer.is())
 {
@@ -683,16 +683,14 @@ void XMLConfigItemContext::EndElement()
  * manipulate the values accordingly. */
 void XMLConfigItemContext::ManipulateConfigItem()
 {
-if( mrItemName.equalsAsciiL(
-RTL_CONSTASCII_STRINGPARAM( PrinterIndependentLayout ) ) )
+if( mrItemName == PrinterIndependentLayout )
 {
 rtl::OUString sValue;
 mrAny = sValue;
 
 sal_Int16 nTmp = document::PrinterIndependentLayout::HIGH_RESOLUTION;
 
-if( sValue.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM(enabled)) ||
-sValue.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM(low-resolution)) )
+if( sValue == enabled || sValue == low-resolution )
 {
 nTmp = document::PrinterIndependentLayout::LOW_RESOLUTION;
 }
@@ -797,7 +795,7 @@ void XMLConfigItemMapIndexedContext::EndElement()
 uno::Reference lang::XMultiServiceFactory  xFac( 
GetImport().GetModel(), uno::UNO_QUERY );
 if( xFac.is() )
 {
-uno::Reference beans::XPropertySet  xProps( 
xFac-createInstance( ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( 
com.sun.star.document.Settings ) ) ), uno::UNO_QUERY );
+uno::Reference beans::XPropertySet  xProps( 
xFac-createInstance( com.sun.star.document.Settings ), uno::UNO_QUERY );
 if( xProps.is()  
xProps-getPropertySetInfo()-hasPropertyByName( maConfigItemName ) )
 {
 xProps-getPropertyValue( maConfigItemName ) = 
xForbChars;
@@ -818,11 +816,11 @@ void XMLConfigItemMapIndexedContext::EndElement()
 beans::PropertyValue *pForChar = aProps.getArray();
 i18n::ForbiddenCharacters aForbid;
 lang::Locale aLocale;
-const rtl::OUString sLanguage  ( 
RTL_CONSTASCII_USTRINGPARAM ( Language ) );
-const rtl::OUString sCountry   ( 
RTL_CONSTASCII_USTRINGPARAM ( Country ) );
-const rtl::OUString sVariant   ( 
RTL_CONSTASCII_USTRINGPARAM ( Variant ) );
-const rtl::OUString sBeginLine ( 
RTL_CONSTASCII_USTRINGPARAM ( BeginLine ) );
-const rtl::OUString sEndLine   ( 
RTL_CONSTASCII_USTRINGPARAM ( EndLine ) );
+const rtl::OUString sLanguage  ( Language );
+const rtl::OUString sCountry   ( Country );
+const rtl::OUString sVariant   ( Variant );
+const rtl::OUString sBeginLine ( BeginLine );
+const rtl::OUString sEndLine   ( EndLine );
 sal_Bool bHaveLanguage = sal_False, bHaveCountry = 
sal_False, bHaveVariant = sal_False,
  bHaveBegin = sal_False, bHaveEnd = sal_False;
 
@@ -886,16 +884,16 @@ void XMLConfigItemMapIndexedContext::EndElement()
 
 formula::SymbolDescriptor *pDescriptor = aSymbolList.getArray();
 
-const rtl::OUString sName 

Re: [Libreoffice-commits] .: xmloff/source

2012-12-08 Thread Lubos Lunak
On Saturday 08 of December 2012, Libreoffice Gerrit user wrote:
  xmloff/source/core/DocumentSettingsContext.cxx|   42 +--
  xmloff/source/core/DomExport.cxx  |4
  xmloff/source/core/SettingsExportHelper.cxx   |   54 ++--
  xmloff/source/core/XMLEmbeddedObjectImportContext.cxx |2
  xmloff/source/core/xmlexp.cxx |   10
  xmloff/source/core/xmlimp.cxx |  229
 ++ xmloff/source/core/xmlmultiimagehelper.cxx| 
  16 -
  7 files changed, 140 insertions(+), 217 deletions(-)

 New commits:
 commit 649c6b38911e9b77d21efba5fa1d6affa9fc4b0e
 Author: Chr. Rossmanith chrrossman...@gmx.de
 Date:   Tue Dec 4 14:35:40 2012 +0100

 RTL_CONSTASCII_USTRINGPARAM clean up
...
 diff --git a/xmloff/source/core/xmlmultiimagehelper.cxx
 b/xmloff/source/core/xmlmultiimagehelper.cxx index c2214f2..e060de1 100644
 --- a/xmloff/source/core/xmlmultiimagehelper.cxx
 +++ b/xmloff/source/core/xmlmultiimagehelper.cxx
 @@ -33,37 +33,37 @@ namespace
  sal_uInt32 nRetval(0);

  // pixel formats first
 -if(rString.endsWithAsciiL(RTL_CONSTASCII_STRINGPARAM(.bmp)))
 +if(rString.endsWithAsciiL(.bmp, 4))
  {
  return 10;
  }
 -if(rString.endsWithAsciiL(RTL_CONSTASCII_STRINGPARAM(.gif)))
 +if(rString.endsWithAsciiL(.gif, 4))
...

 These should be rString.endsWith( .gif ) . Specifying the length manually 
is error-prone and unnecessary.

-- 
 Lubos Lunak
 l.lu...@suse.cz
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: xmloff/source

2012-11-28 Thread Libreoffice Gerrit user
 xmloff/source/style/XMLFontAutoStylePool.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 5f82ec11789a201e38565f59ce7d598b3d17827f
Author: Tor Lillqvist t...@iki.fi
Date:   Wed Nov 28 21:13:46 2012 +0200

OUString::trim() does not modify in-place

Change-Id: I9efc0ef7e8fa433352dcbc3d3173b95fdf838e20

diff --git a/xmloff/source/style/XMLFontAutoStylePool.cxx 
b/xmloff/source/style/XMLFontAutoStylePool.cxx
index ca1d8a5..a496e79 100644
--- a/xmloff/source/style/XMLFontAutoStylePool.cxx
+++ b/xmloff/source/style/XMLFontAutoStylePool.cxx
@@ -165,7 +165,7 @@ OUString XMLFontAutoStylePool::Add(
 else if( nLen  0 )
 {
 sName = rFamilyName.copy( 0, nLen );
-sName.trim();
+sName = sName.trim();
 }
 
 if( sName.isEmpty() )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-11-28 Thread Libreoffice Gerrit user
 xmloff/source/text/XMLTextFrameContext.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 9ebc1ab13854f44af64b2d63717e631542d9374c
Author: Tor Lillqvist t...@iki.fi
Date:   Wed Nov 28 21:26:01 2012 +0200

OUString::trim() does not modify in-place

Change-Id: I68227dcf77b9082708503d2c0d9bf829d78a442e

diff --git a/xmloff/source/text/XMLTextFrameContext.cxx 
b/xmloff/source/text/XMLTextFrameContext.cxx
index f82811c..6e8fc2e 100644
--- a/xmloff/source/text/XMLTextFrameContext.cxx
+++ b/xmloff/source/text/XMLTextFrameContext.cxx
@@ -1044,7 +1044,7 @@ XMLTextFrameContext_Impl::XMLTextFrameContext_Impl(
 case XML_TOK_TEXT_FRAME_TRANSFORM:
 {
 OUString sValue( rValue );
-sValue.trim();
+sValue = sValue.trim();
 const OUString aRotate(GetXMLToken(XML_ROTATE));
 const sal_Int32 nRotateLen(aRotate.getLength());
 sal_Int32 nLen = sValue.getLength();
@@ -1054,7 +1054,7 @@ XMLTextFrameContext_Impl::XMLTextFrameContext_Impl(
 ')' == sValue[nLen-1] )
 {
 sValue = sValue.copy( nRotateLen+1, nLen-(nRotateLen+2) );
-sValue.trim();
+sValue = sValue.trim();
 sal_Int32 nVal;
 if (::sax::Converter::convertNumber( nVal, sValue ))
 nRotation = (sal_Int16)(nVal % 360 );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-11-16 Thread Libreoffice Gerrit user
 xmloff/source/style/xmlnumfe.cxx |   15 ---
 xmloff/source/style/xmlnumfi.cxx |   12 ++--
 2 files changed, 14 insertions(+), 13 deletions(-)

New commits:
commit ec2e175384f89d183def1d53f33a6bb394278c87
Author: Eike Rathke er...@redhat.com
Date:   Fri Nov 16 18:55:33 2012 +0100

use LanguageTag

Change-Id: I568af42912ce1ec7701dc0043dfbd9d69243d533

diff --git a/xmloff/source/style/xmlnumfe.cxx b/xmloff/source/style/xmlnumfe.cxx
index 295b2c9..a2f85a4 100644
--- a/xmloff/source/style/xmlnumfe.cxx
+++ b/xmloff/source/style/xmlnumfe.cxx
@@ -22,6 +22,7 @@
 #include svl/zformat.hxx
 #include svl/numuno.hxx
 #include i18npool/mslangid.hxx
+#include i18npool/languagetag.hxx
 #include tools/debug.hxx
 #include rtl/math.hxx
 #include unotools/calendarwrapper.hxx
@@ -243,7 +244,7 @@ SvXMLNumFmtExport::SvXMLNumFmtExport(
 }
 else
 {
-lang::Locale aLocale( MsLangId::convertLanguageToLocale( 
MsLangId::getSystemLanguage() ) );
+lang::Locale aLocale( LanguageTag( MsLangId::getSystemLanguage() 
).getLocale() );
 
 pCharClass = new CharClass( 
comphelper::getComponentContext(rExport.getServiceFactory()), aLocale );
 pLocaleData = new LocaleDataWrapper( 
comphelper::getComponentContext(rExport.getServiceFactory()), aLocale );
@@ -278,7 +279,7 @@ SvXMLNumFmtExport::SvXMLNumFmtExport(
 }
 else
 {
-lang::Locale aLocale( MsLangId::convertLanguageToLocale( 
MsLangId::getSystemLanguage() ) );
+lang::Locale aLocale( LanguageTag( MsLangId::getSystemLanguage() 
).getLocale() );
 
 pCharClass = new CharClass( 
comphelper::getComponentContext(rExport.getServiceFactory()), aLocale );
 pLocaleData = new LocaleDataWrapper( 
comphelper::getComponentContext(rExport.getServiceFactory()), aLocale );
@@ -342,7 +343,7 @@ void SvXMLNumFmtExport::AddLanguageAttr_Impl( sal_Int32 
nLang )
 if ( nLang != LANGUAGE_SYSTEM )
 {
 OUString aLangStr, aCountryStr;
-MsLangId::convertLanguageToIsoNames( (LanguageType)nLang, aLangStr, 
aCountryStr );
+LanguageTag( (LanguageType)nLang ).getIsoLanguageCountry( aLangStr, 
aCountryStr );
 
 if (!aLangStr.isEmpty())
 rExport.AddAttribute( XML_NAMESPACE_NUMBER, XML_LANGUAGE, aLangStr 
);
@@ -798,7 +799,7 @@ sal_Bool SvXMLNumFmtExport::WriteTextWithCurrency_Impl( 
const OUString rString,
 
 sal_Bool bRet = sal_False;
 
-LanguageType nLang = MsLangId::convertLocaleToLanguage( rLocale );
+LanguageType nLang = LanguageTag( rLocale ).getLanguageType( false);
 pFormatter-ChangeIntl( nLang );
 String sCurString, sDummy;
 pFormatter-GetCompatibilityCurrency( sCurString, sDummy );
@@ -841,7 +842,7 @@ static OUString lcl_GetDefaultCalendar( SvNumberFormatter* 
pFormatter, LanguageT
 CalendarWrapper* pCalendar = pFormatter-GetCalendar();
 if (pCalendar)
 {
-lang::Locale aLocale( MsLangId::convertLanguageToLocale( nLang ) );
+lang::Locale aLocale( LanguageTag( nLang ).getLocale() );
 
 uno::SequenceOUString aCals = pCalendar-getAllCalendars( aLocale );
 sal_Int32 nCnt = aCals.getLength();
@@ -1311,7 +1312,7 @@ void SvXMLNumFmtExport::ExportPart_Impl( const 
SvNumberformat rFormat, sal_uInt
 //  automatic currency symbol is implemented as 
part of
 //  normal text - search for the symbol
 bCurrencyWritten = WriteTextWithCurrency_Impl( 
*pElemStr,
-MsLangId::convertLanguageToLocale( nLang ) );
+LanguageTag( nLang ).getLocale() );
 bAnyContent = sal_True;
 }
 else
@@ -1486,7 +1487,7 @@ void SvXMLNumFmtExport::ExportPart_Impl( const 
SvNumberformat rFormat, sal_uInt
 if ( nElemType == NF_KEY_ )
 {
 //  write additional text element for separator
-pLocaleData-setLocale( 
MsLangId::convertLanguageToLocale( nLang ) );
+pLocaleData-setLocale( LanguageTag( nLang 
).getLocale() );
 AddToTextElement_Impl( 
pLocaleData-getLongDateDayOfWeekSep() );
 }
 }
diff --git a/xmloff/source/style/xmlnumfi.cxx b/xmloff/source/style/xmlnumfi.cxx
index abfbc36..f5b8c21 100644
--- a/xmloff/source/style/xmlnumfi.cxx
+++ b/xmloff/source/style/xmlnumfi.cxx
@@ -26,7 +26,7 @@
 #include svl/zformat.hxx
 #include svl/numuno.hxx
 #include rtl/math.hxx
-#include i18npool/mslangid.hxx
+#include i18npool/languagetag.hxx
 #include tools/color.hxx
 #include tools/debug.hxx
 #include rtl/ustrbuf.hxx
@@ -602,9 +602,9 @@ const LocaleDataWrapper SvXMLNumImpData::GetLocaleData( 
LanguageType nLang )
 pLocaleData = new LocaleDataWrapper(
 comphelper::getComponentContext(

[Libreoffice-commits] .: xmloff/source

2012-11-15 Thread Libreoffice Gerrit user
 xmloff/source/core/xmlimp.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 42782fddff98eeab5c8249918e9ba000f08c22e8
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Nov 15 18:28:24 2012 +0200

WaE: 'const' type qualifier on return type has no effect

Change-Id: I06cc8b77e49ec2a22534e1f45d93dfa0d0a0689f

diff --git a/xmloff/source/core/xmlimp.cxx b/xmloff/source/core/xmlimp.cxx
index 922fe8a..d6e29df 100644
--- a/xmloff/source/core/xmlimp.cxx
+++ b/xmloff/source/core/xmlimp.cxx
@@ -203,7 +203,7 @@ namespace
 ~DocumentInfo()
 {}
 
-const sal_uInt16 getGeneratorVersion() const
+sal_uInt16 getGeneratorVersion() const
 {
 return mnGeneratorVersion;
 }
@@ -265,7 +265,7 @@ public:
 DestroyFontToSubsFontConverter( hMathFontConv );
 }
 
-const sal_uInt16 getGeneratorVersion( const SvXMLImport rImport )
+sal_uInt16 getGeneratorVersion( const SvXMLImport rImport )
 {
 if ( !mpDocumentInfo.get() )
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-11-09 Thread Libreoffice Gerrit user
 xmloff/source/text/XMLRedlineExport.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit d34e2f8bf150210355d6717be7b2ad95c869af25
Author: jailletc36 christophe.jail...@wanadoo.fr
Date:   Fri Nov 9 14:28:58 2012 +0100

s/aIter++/++aIter/


Change-Id: Id80013cbab8f1633db59d2d4492c7363224871bd
Signed-off-by: jailletc36 christophe.jail...@wanadoo.fr
Reviewed-on: https://gerrit.libreoffice.org/1016
Reviewed-by: Eike Rathke er...@redhat.com
Tested-by: Eike Rathke er...@redhat.com

diff --git a/xmloff/source/text/XMLRedlineExport.cxx 
b/xmloff/source/text/XMLRedlineExport.cxx
index 917fb95..11d6e2e 100644
--- a/xmloff/source/text/XMLRedlineExport.cxx
+++ b/xmloff/source/text/XMLRedlineExport.cxx
@@ -107,7 +107,7 @@ XMLRedlineExport::~XMLRedlineExport()
 // delete changes lists
 for( ChangesMapType::iterator aIter = aChangeMap.begin();
  aIter != aChangeMap.end();
- aIter++ )
+ ++aIter )
 {
 delete aIter-second;
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-11-08 Thread Libreoffice Gerrit user
 xmloff/source/style/XMLBackgroundImageContext.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit faff21dcaa3f04be5aedbb966f0a6794404dfcaa
Author: jailletc36 christophe.jail...@wanadoo.fr
Date:   Thu Nov 8 21:15:26 2012 +0100

Same, cut and paste typo ?

Change-Id: I34e1de922544e5e0f0ea0e03be5d238d600aa833
Signed-off-by: jailletc36 christophe.jail...@wanadoo.fr
Reviewed-on: https://gerrit.libreoffice.org/1010
Reviewed-by: Tor Lillqvist t...@iki.fi
Tested-by: Tor Lillqvist t...@iki.fi

diff --git a/xmloff/source/style/XMLBackgroundImageContext.cxx 
b/xmloff/source/style/XMLBackgroundImageContext.cxx
index 4d74120..a209d5b 100644
--- a/xmloff/source/style/XMLBackgroundImageContext.cxx
+++ b/xmloff/source/style/XMLBackgroundImageContext.cxx
@@ -154,7 +154,6 @@ static void lcl_xmlbic_MergeVertPos( GraphicLocation ePos,
 : (GraphicLocation_MIDDLE_MIDDLE==eVert
 ? GraphicLocation_LEFT_MIDDLE
 : GraphicLocation_LEFT_BOTTOM);
-ePos = eVert;
 break;
 
 case GraphicLocation_MIDDLE_TOP:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-11-08 Thread Libreoffice Gerrit user
 xmloff/source/style/backhdl.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit e9fc811e2bc995220954b4c471b93a7f06a82359
Author: jailletc36 christophe.jail...@wanadoo.fr
Date:   Thu Nov 8 21:11:45 2012 +0100

Cut and paste typo ?

Corrected according to surrounding code. I don't know if or when
this code is called.

Change-Id: I4ed63439e47a89cc982d792c619c4c58335daf93
Signed-off-by: jailletc36 christophe.jail...@wanadoo.fr
Reviewed-on: https://gerrit.libreoffice.org/1009
Reviewed-by: Tor Lillqvist t...@iki.fi
Tested-by: Tor Lillqvist t...@iki.fi

diff --git a/xmloff/source/style/backhdl.cxx b/xmloff/source/style/backhdl.cxx
index 9ce7150..9cfe176 100644
--- a/xmloff/source/style/backhdl.cxx
+++ b/xmloff/source/style/backhdl.cxx
@@ -236,7 +236,6 @@ void XMLBackGraphicPositionPropHdl::MergeXMLVertPos( 
style::GraphicLocation ePo
   (style::GraphicLocation_MIDDLE_MIDDLE==eVert ?
style::GraphicLocation_LEFT_MIDDLE :
style::GraphicLocation_LEFT_BOTTOM);
-ePos = eVert;
 break;
 
 case style::GraphicLocation_MIDDLE_TOP:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-11-03 Thread Libreoffice Gerrit user
 xmloff/source/chart/SchXMLAutoStylePoolP.cxx   |1 -
 xmloff/source/chart/SchXMLPlotAreaContext.cxx  |2 --
 xmloff/source/core/xmlexp.cxx  |1 -
 xmloff/source/draw/shapeexport2.cxx|2 --
 xmloff/source/meta/MetaExportComponent.cxx |4 ++--
 xmloff/source/style/MarkerStyle.cxx|3 ---
 xmloff/source/style/xmlaustp.cxx   |2 --
 xmloff/source/style/xmlexppr.cxx   |3 ---
 xmloff/source/style/xmlnumi.cxx|1 -
 xmloff/source/text/txtparae.cxx|9 -
 xmloff/source/transform/EventOASISTContext.cxx |1 -
 11 files changed, 2 insertions(+), 27 deletions(-)

New commits:
commit 7fe8d9328c619523897aa0e84b4bb9f5ab11b743
Author: Miklos Vajna vmik...@suse.cz
Date:   Sat Nov 3 18:35:52 2012 +0100

xmloff: drop unused OUStrings

Change-Id: I040e9a5c9e60a27aa82fd624cc01edfecf5cc8de

diff --git a/xmloff/source/chart/SchXMLAutoStylePoolP.cxx 
b/xmloff/source/chart/SchXMLAutoStylePoolP.cxx
index 5f59c34..5660ec0 100644
--- a/xmloff/source/chart/SchXMLAutoStylePoolP.cxx
+++ b/xmloff/source/chart/SchXMLAutoStylePoolP.cxx
@@ -54,7 +54,6 @@ void SchXMLAutoStylePoolP::exportStyleAttributes(
 const SvXMLNamespaceMap rNamespaceMap
 ) const
 {
-const rtl::OUString sCDATA( GetXMLToken( XML_CDATA ));
 SvXMLAutoStylePoolP::exportStyleAttributes( rAttrList, nFamily, 
rProperties,
 rPropExp, rUnitConverter, 
rNamespaceMap );
 
diff --git a/xmloff/source/chart/SchXMLPlotAreaContext.cxx 
b/xmloff/source/chart/SchXMLPlotAreaContext.cxx
index 51f8cd2..e22934f 100644
--- a/xmloff/source/chart/SchXMLPlotAreaContext.cxx
+++ b/xmloff/source/chart/SchXMLPlotAreaContext.cxx
@@ -673,7 +673,6 @@ SchXMLDataPointContext::~SchXMLDataPointContext()
 void SchXMLDataPointContext::StartElement( const uno::Reference 
xml::sax::XAttributeList  xAttrList )
 {
 sal_Int16 nAttrCount = xAttrList.is()? xAttrList-getLength(): 0;
-::rtl::OUString aValue;
 ::rtl::OUString sAutoStyleName;
 sal_Int32 nRepeat = 1;
 
@@ -1050,7 +1049,6 @@ 
SchXMLStatisticsObjectContext::~SchXMLStatisticsObjectContext()
 void SchXMLStatisticsObjectContext::StartElement( const uno::Reference 
xml::sax::XAttributeList  xAttrList )
 {
 sal_Int16 nAttrCount = xAttrList.is()? xAttrList-getLength(): 0;
-::rtl::OUString aValue;
 ::rtl::OUString sAutoStyleName;
 rtl::OUString aPosRange;
 rtl::OUString aNegRange;
diff --git a/xmloff/source/core/xmlexp.cxx b/xmloff/source/core/xmlexp.cxx
index e3bdf9f..05a33bd 100644
--- a/xmloff/source/core/xmlexp.cxx
+++ b/xmloff/source/core/xmlexp.cxx
@@ -2207,7 +2207,6 @@ OUString SvXMLExport::GetRelativeReference(const 
OUString rValue)
 {
 }
 }
-OUString sRet = sValue;
 if( xUriRef.is() )//no conversion for empty values or for fragments
 {
 //conversion for matching schemes only
diff --git a/xmloff/source/draw/shapeexport2.cxx 
b/xmloff/source/draw/shapeexport2.cxx
index 0614995..988137e 100644
--- a/xmloff/source/draw/shapeexport2.cxx
+++ b/xmloff/source/draw/shapeexport2.cxx
@@ -239,8 +239,6 @@ sal_Bool XMLShapeExport::ImpExportPresentationAttributes( 
const uno::Reference
 {
 sal_Bool bIsEmpty = sal_False;
 
-OUStringBuffer sStringBuffer;
-
 // write presentation class entry
 mrExport.AddAttribute(XML_NAMESPACE_PRESENTATION, XML_CLASS, rClass);
 
diff --git a/xmloff/source/meta/MetaExportComponent.cxx 
b/xmloff/source/meta/MetaExportComponent.cxx
index 9fb1976..511eb6a 100644
--- a/xmloff/source/meta/MetaExportComponent.cxx
+++ b/xmloff/source/meta/MetaExportComponent.cxx
@@ -163,8 +163,8 @@ sal_uInt32 XMLMetaExportComponent::exportDoc( enum 
XMLTokenEnum )
 AddAttribute( XML_NAMESPACE_OFFICE, XML_VERSION,
 ::rtl::OUString::createFromAscii(pVersion) );
 
-SvXMLElementExport aDocElem( *this, XML_NAMESPACE_OFFICE, 
XML_DOCUMENT_META,
-sal_True, sal_True );
+SvXMLElementExport aDocElem( *this, XML_NAMESPACE_OFFICE, 
XML_DOCUMENT_META,
+sal_True, sal_True );
 
 // NB: office:meta is now written by _ExportMeta
 _ExportMeta();
diff --git a/xmloff/source/style/MarkerStyle.cxx 
b/xmloff/source/style/MarkerStyle.cxx
index 5c0c0ec..5b129b4 100644
--- a/xmloff/source/style/MarkerStyle.cxx
+++ b/xmloff/source/style/MarkerStyle.cxx
@@ -183,9 +183,6 @@ sal_Bool XMLMarkerStyleExport::exportXML(
 
 if(rValue = aBezier)
 {
-OUString aStrValue;
-OUStringBuffer aOut;
-
 /
 // Name
 sal_Bool bEncoded = sal_False;
diff --git a/xmloff/source/style/xmlaustp.cxx b/xmloff/source/style/xmlaustp.cxx
index 46c37ed..25a1a48 100644
--- a/xmloff/source/style/xmlaustp.cxx
+++ b/xmloff/source/style/xmlaustp.cxx
@@ -195,8 +195,6 @@ void 

[Libreoffice-commits] .: xmloff/source

2012-10-12 Thread Libreoffice Gerrit user
 xmloff/source/chart/SchXMLTableContext.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit dcaf8a871c3ce47e4b51b5c026e47d1ad03f5696
Author: Julien Nabet serval2...@yahoo.fr
Date:   Fri Oct 12 23:29:59 2012 +0200

uninit variable

Change-Id: I94870693fc269ab7a40e9ded482d259921316c38

diff --git a/xmloff/source/chart/SchXMLTableContext.cxx 
b/xmloff/source/chart/SchXMLTableContext.cxx
index c9c5857..5e0a67a 100644
--- a/xmloff/source/chart/SchXMLTableContext.cxx
+++ b/xmloff/source/chart/SchXMLTableContext.cxx
@@ -79,7 +79,8 @@ struct lcl_ApplyCellToData : public ::std::unary_function 
SchXMLCell, void 
 lcl_ApplyCellToData( Sequence double   rOutData ) :
 m_rData( rOutData ),
 m_nIndex( 0 ),
-m_nSize( rOutData.getLength())
+m_nSize( rOutData.getLength()),
+m_fNaN( 0.0 )
 {
 ::rtl::math::setNan( m_fNaN );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-10-02 Thread Libreoffice Gerrit user
 xmloff/source/text/XMLTextMasterPageExport.cxx |   14 --
 1 file changed, 8 insertions(+), 6 deletions(-)

New commits:
commit d92345561c998f7382cf9ef0fdcd29096f978435
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Oct 2 14:41:59 2012 +0100

don't blow up on .odt export with !hasValue footers/headers

Change-Id: I2507c23b7b17b9172f720548d32cdfe299d4fa20

diff --git a/xmloff/source/text/XMLTextMasterPageExport.cxx 
b/xmloff/source/text/XMLTextMasterPageExport.cxx
index a449f92..ab7f854 100644
--- a/xmloff/source/text/XMLTextMasterPageExport.cxx
+++ b/xmloff/source/text/XMLTextMasterPageExport.cxx
@@ -127,20 +127,21 @@ void XMLTextMasterPageExport::exportMasterPageContent(
 else
 {
 aAny = rPropSet-getPropertyValue( sHeaderOn );
-sal_Bool bHeader = *(sal_Bool *)aAny.getValue();
+sal_Bool bHeader = sal_False;
+aAny = bHeader;
 
 sal_Bool bHeaderFirst = sal_False;
 if( bHeader )
 {
 aAny = rPropSet-getPropertyValue( sFirstShareContent );
-bHeaderFirst = !*(sal_Bool *)aAny.getValue();
+aAny = bHeaderFirst;
 }
 
 sal_Bool bHeaderLeft = sal_False;
 if( bHeader )
 {
 aAny = rPropSet-getPropertyValue( sHeaderShareContent );
-bHeaderLeft = !*(sal_Bool *)aAny.getValue();
+aAny = bHeaderLeft;
 }
 
 if( xHeaderText.is() )
@@ -174,20 +175,21 @@ void XMLTextMasterPageExport::exportMasterPageContent(
 }
 
 aAny = rPropSet-getPropertyValue( sFooterOn );
-sal_Bool bFooter = *(sal_Bool *)aAny.getValue();
+sal_Bool bFooter = sal_False;
+aAny = bFooter;
 
 sal_Bool bFooterFirst = sal_False;
 if( bFooter )
 {
 aAny = rPropSet-getPropertyValue( sFirstShareContent );
-bFooterFirst = !*(sal_Bool *)aAny.getValue();
+aAny = bFooterFirst;
 }
 
 sal_Bool bFooterLeft = sal_False;
 if( bFooter )
 {
 aAny = rPropSet-getPropertyValue( sFooterShareContent );
-bFooterLeft = !*(sal_Bool *)aAny.getValue();
+aAny = bFooterLeft;
 }
 
 if( xFooterText.is() )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-08-03 Thread Caolán McNamara
 xmloff/source/style/impastp4.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 4e1629ff792adeff05e4a0e04f5263a242d167ff
Author: Caolán McNamara caol...@redhat.com
Date:   Fri Aug 3 08:32:03 2012 +0100

additional const_iterator in const method

Change-Id: I8cad958e1b6ba5eec000244a31e1ee98e0b5d123

diff --git a/xmloff/source/style/impastp4.cxx b/xmloff/source/style/impastp4.cxx
index 194c516..6131466 100644
--- a/xmloff/source/style/impastp4.cxx
+++ b/xmloff/source/style/impastp4.cxx
@@ -300,13 +300,13 @@ void SvXMLAutoStylePoolP_Impl::exportXML(
 {
 // Get list of parents for current family (nFamily)
 XMLFamilyData_Impl aTmp( nFamily );
-XMLFamilyDataList_Impl::iterator aFind = maFamilyList.find(aTmp);
+XMLFamilyDataList_Impl::const_iterator aFind = maFamilyList.find(aTmp);
 DBG_ASSERT( aFind != maFamilyList.end(),
 SvXMLAutoStylePool_Impl::exportXML: unknown family );
 if (aFind == maFamilyList.end())
 return;
 
-XMLFamilyData_Impl rFamily = *aFind;
+const XMLFamilyData_Impl rFamily = *aFind;
 sal_uInt32 nCount = rFamily.mnCount;
 
 if (!nCount)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-08-01 Thread Caolán McNamara
 xmloff/source/style/xmlstyle.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 0fc413a327242adc1eb4c196179fdf660229edc9
Author: Caolán McNamara caol...@redhat.com
Date:   Wed Aug 1 15:04:02 2012 +0100

const SvXMLStyleIndexCmp_Impl up

Change-Id: I5bd75e73f2584364bbb749767529efc75aff3d15

diff --git a/xmloff/source/style/xmlstyle.cxx b/xmloff/source/style/xmlstyle.cxx
index f881dec..380dd48 100644
--- a/xmloff/source/style/xmlstyle.cxx
+++ b/xmloff/source/style/xmlstyle.cxx
@@ -246,7 +246,7 @@ public:
 
 struct SvXMLStyleIndexCmp_Impl
 {
-bool operator()(const SvXMLStyleIndex_Impl r1, const 
SvXMLStyleIndex_Impl r2)
+bool operator()(const SvXMLStyleIndex_Impl r1, const 
SvXMLStyleIndex_Impl r2) const
 {
 sal_Int32 nRet;
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-07-19 Thread Michael Stahl
 xmloff/source/style/PageMasterImportPropMapper.cxx |   51 +
 1 file changed, 51 insertions(+)

New commits:
commit 7f9928bfa561ccb6ed4e2baacc7d6960bc1ce231
Author: Michael Stahl mst...@redhat.com
Date:   Thu Jul 19 12:12:14 2012 +0200

fdo#38056: ODF import: fix page style attributes:

PageMasterImportPropMapper: in the case of a single fo:border
and style:border-line-width attribute, it is possible that the border
is imported wrongly, like this:

1. pAllBorderProperty is set from the imported value, with name TopBorder
2. individual pNewBorder[i] are created as copies from pAllBorderProperty,
   one of which also with name TopBorder
3. pNewBorder[i] is updated with widths from pBorderWidths[i]
4. the individual pNewBorder[i] are added to the property vector
5. the property vector is sorted by property name
6. the properites are applied in order; if the pNewBorder[TOP]
   happens to precede the pAllBorderProperty (which is indeterminate
   as they both have name TopBorder), then the pAllBorderProperty
   will overwrite the border widths computed in step 3.

Thus, nerf the various pAllFoo properties so they do not override
the individual Foo properties later on.

Change-Id: I87755f1184d59da2aa72ac053e6f77d7295d6958

diff --git a/xmloff/source/style/PageMasterImportPropMapper.cxx 
b/xmloff/source/style/PageMasterImportPropMapper.cxx
index ad45cdb..005c732 100644
--- a/xmloff/source/style/PageMasterImportPropMapper.cxx
+++ b/xmloff/source/style/PageMasterImportPropMapper.cxx
@@ -408,6 +408,57 @@ void 
PageMasterImportPropertyMapper::finished(::std::vector XMLPropertyState 
 rProperties.push_back(*pFooterDynamic);
 delete pFooterDynamic;
 }
+
+// fdo#38056: nerf the various AllFoo properties so they do not override
+// the individual Foo properties later on
+if (pAllPaddingProperty)
+{
+pAllPaddingProperty-mnIndex = -1;
+}
+if (pAllBorderProperty)
+{
+pAllBorderProperty-mnIndex = -1;
+}
+if (pAllBorderWidthProperty)
+{
+pAllBorderWidthProperty-mnIndex = -1;
+}
+if (pAllHeaderPaddingProperty)
+{
+pAllHeaderPaddingProperty-mnIndex = -1;
+}
+if (pAllHeaderBorderProperty)
+{
+pAllHeaderBorderProperty-mnIndex = -1;
+}
+if (pAllHeaderBorderWidthProperty)
+{
+pAllHeaderBorderWidthProperty-mnIndex = -1;
+}
+if (pAllFooterPaddingProperty)
+{
+pAllFooterPaddingProperty-mnIndex = -1;
+}
+if (pAllFooterBorderProperty)
+{
+pAllFooterBorderProperty-mnIndex = -1;
+}
+if (pAllFooterBorderWidthProperty)
+{
+pAllFooterBorderWidthProperty-mnIndex = -1;
+}
+if (pAllMarginProperty)
+{
+pAllMarginProperty-mnIndex = -1;
+}
+if (pAllHeaderMarginProperty)
+{
+pAllHeaderMarginProperty-mnIndex = -1;
+}
+if (pAllFooterMarginProperty)
+{
+pAllFooterMarginProperty-mnIndex = -1;
+}
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-07-13 Thread Lubos Lunak
 xmloff/source/style/fonthdl.cxx |6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

New commits:
commit fc508908f55cc1fe5a22adcba710cebb75fc979c
Author: Luboš Luňák l.lu...@suse.cz
Date:   Fri Jul 13 15:15:57 2012 +0200

fix a crash with empty string

If it's named nLast, then it's supposed to point to the last character,
which, with an empty string, logically means it should point before
the start of the string.

Change-Id: I248fbe80c1c4b1c2c8d39f0bcf772f9b85415e4a

diff --git a/xmloff/source/style/fonthdl.cxx b/xmloff/source/style/fonthdl.cxx
index 3d58197..f4d1705 100644
--- a/xmloff/source/style/fonthdl.cxx
+++ b/xmloff/source/style/fonthdl.cxx
@@ -85,12 +85,10 @@ sal_Bool XMLFontFamilyNamePropHdl::importXML( const 
OUString rStrImpValue, uno:
 {
 sal_Int32 nFirst = nPos;
 nPos = ::sax::Converter::indexOfComma( rStrImpValue, nPos );
-sal_Int32 nLast = (-1 == nPos ? rStrImpValue.getLength() : nPos);
-if( nLast  0 )
-nLast--;
+sal_Int32 nLast = (-1 == nPos ? rStrImpValue.getLength() - 1 : nPos - 
1);
 
 // skip trailing blanks
-while( sal_Unicode(' ') == rStrImpValue[nLast]  nLast  nFirst )
+while( nLast  nFirst  sal_Unicode(' ') == rStrImpValue[nLast] )
 nLast--;
 
 // skip leading blanks
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-07-10 Thread Miklos Vajna
 xmloff/source/core/unointerfacetouniqueidentifiermapper.cxx |2 
 xmloff/source/draw/XMLGraphicsDefaultStyle.cxx  |4 -
 xmloff/source/script/XMLScriptContextFactory.cxx|6 -
 xmloff/source/script/XMLScriptExportHandler.cxx |2 
 xmloff/source/script/XMLStarBasicContextFactory.cxx |   10 +-
 xmloff/source/style/shdwdhdl.cxx|2 
 xmloff/source/text/XMLAutoMarkFileContext.cxx   |3 
 xmloff/source/text/XMLAutoTextEventImport.cxx   |3 
 xmloff/source/text/XMLIndexTemplateContext.cxx  |   44 ++--
 xmloff/source/text/XMLIndexTitleTemplateContext.cxx |8 --
 xmloff/source/text/XMLTextMasterPageExport.cxx  |   16 ++--
 xmloff/source/text/XMLTextShapeStyleContext.cxx |2 
 12 files changed, 48 insertions(+), 54 deletions(-)

New commits:
commit a8fb1a123f6c46eed611dbaea699624ea0b7962b
Author: Miklos Vajna vmik...@suse.cz
Date:   Tue Jul 10 11:01:00 2012 +0200

xmloff: kill RTL_CONSTASCII_USTRINGPARAM in rebased code

Change-Id: I3d5227c299d78d4492c47e29385c2c5ca25cfe49

diff --git a/xmloff/source/core/unointerfacetouniqueidentifiermapper.cxx 
b/xmloff/source/core/unointerfacetouniqueidentifiermapper.cxx
index fc330d2..4e9a715 100644
--- a/xmloff/source/core/unointerfacetouniqueidentifiermapper.cxx
+++ b/xmloff/source/core/unointerfacetouniqueidentifiermapper.cxx
@@ -44,7 +44,7 @@ const OUString 
UnoInterfaceToUniqueIdentifierMapper::registerReference( const R
 }
 else
 {
-OUString aId( RTL_CONSTASCII_USTRINGPARAM( id ) );
+OUString aId( id );
 aId += OUString::valueOf( mnNextId++ );
 return (*maEntries.insert( IdMap_t::value_type( aId, rInterface ) 
).first).first;
 }
diff --git a/xmloff/source/draw/XMLGraphicsDefaultStyle.cxx 
b/xmloff/source/draw/XMLGraphicsDefaultStyle.cxx
index 8c157a5..c6216a4 100644
--- a/xmloff/source/draw/XMLGraphicsDefaultStyle.cxx
+++ b/xmloff/source/draw/XMLGraphicsDefaultStyle.cxx
@@ -88,7 +88,7 @@ void XMLGraphicsDefaultStyle::SetDefaults()
 if( !xFact.is() )
 return;
 
-Reference XPropertySet  xDefaults( xFact-createInstance( 
OUString(RTL_CONSTASCII_USTRINGPARAM(com.sun.star.drawing.Defaults) ) ), 
UNO_QUERY );
+Reference XPropertySet  xDefaults( xFact-createInstance( 
com.sun.star.drawing.Defaults ), UNO_QUERY );
 if( !xDefaults.is() )
 return;
 // SJ: #i114750#
@@ -105,7 +105,7 @@ void XMLGraphicsDefaultStyle::SetDefaults()
 ) )
 bWordWrapDefault = sal_False;
 
-const OUString sTextWordWrap( RTL_CONSTASCII_USTRINGPARAM( TextWordWrap 
) );
+const OUString sTextWordWrap( TextWordWrap );
 Reference XPropertySetInfo  xInfo( xDefaults-getPropertySetInfo() );
 if ( xInfo-hasPropertyByName( sTextWordWrap ) )
 xDefaults-setPropertyValue( sTextWordWrap, Any( bWordWrapDefault ) );
diff --git a/xmloff/source/script/XMLScriptContextFactory.cxx 
b/xmloff/source/script/XMLScriptContextFactory.cxx
index 5f671ea..1990d67 100644
--- a/xmloff/source/script/XMLScriptContextFactory.cxx
+++ b/xmloff/source/script/XMLScriptContextFactory.cxx
@@ -35,9 +35,9 @@ using ::com::sun::star::uno::Sequence;
 using ::com::sun::star::uno::Any;
 
 XMLScriptContextFactory::XMLScriptContextFactory() :
-sEventType(RTL_CONSTASCII_USTRINGPARAM(EventType)),
-sScript(RTL_CONSTASCII_USTRINGPARAM(Script)),
-sURL(RTL_CONSTASCII_USTRINGPARAM(Script))
+sEventType(EventType),
+sScript(Script),
+sURL(Script)
 {
 }
 
diff --git a/xmloff/source/script/XMLScriptExportHandler.cxx 
b/xmloff/source/script/XMLScriptExportHandler.cxx
index 9e0c096..51057b7 100644
--- a/xmloff/source/script/XMLScriptExportHandler.cxx
+++ b/xmloff/source/script/XMLScriptExportHandler.cxx
@@ -34,7 +34,7 @@ using ::com::sun::star::beans::PropertyValue;
 
 
 XMLScriptExportHandler::XMLScriptExportHandler() :
-sURL(RTL_CONSTASCII_USTRINGPARAM(Script))
+sURL(Script)
 {
 }
 
diff --git a/xmloff/source/script/XMLStarBasicContextFactory.cxx 
b/xmloff/source/script/XMLStarBasicContextFactory.cxx
index 3b3faa3..b9825be 100644
--- a/xmloff/source/script/XMLStarBasicContextFactory.cxx
+++ b/xmloff/source/script/XMLStarBasicContextFactory.cxx
@@ -36,10 +36,10 @@ using ::com::sun::star::uno::Any;
 
 
 XMLStarBasicContextFactory::XMLStarBasicContextFactory() :
-sEventType(RTL_CONSTASCII_USTRINGPARAM(EventType)),
-sLibrary(RTL_CONSTASCII_USTRINGPARAM(Library)),
-sMacroName(RTL_CONSTASCII_USTRINGPARAM(MacroName)),
-sStarBasic(RTL_CONSTASCII_USTRINGPARAM(StarBasic))
+sEventType(EventType),
+sLibrary(Library),
+sMacroName(MacroName),
+sStarBasic(StarBasic)
 {
 }
 
@@ -83,7 +83,7 @@ SvXMLImportContext* XMLStarBasicContextFactory::CreateContext(
 sMacroNameVal.copy(0,rApp.getLength()).equalsIgnoreAsciiCase( rApp ) 
 ':' == sMacroNameVal[rApp.getLength()] 

[Libreoffice-commits] .: xmloff/source

2012-07-05 Thread Miklos Vajna
 xmloff/source/text/txtimp.cxx |   22 --
 1 file changed, 22 deletions(-)

New commits:
commit a99e3134aa253977b5112cc903d4097907bf3801
Author: Miklos Vajna vmik...@suse.cz
Date:   Thu Jul 5 11:49:19 2012 +0200

xmloff: unused includes in txtimp

Change-Id: I0a4feed51067f6b222d60f569dd794f2439fc790

diff --git a/xmloff/source/text/txtimp.cxx b/xmloff/source/text/txtimp.cxx
index d80ed05..e751058 100644
--- a/xmloff/source/text/txtimp.cxx
+++ b/xmloff/source/text/txtimp.cxx
@@ -30,40 +30,21 @@
 #include vector
 
 #include boost/scoped_array.hpp
-#include boost/tuple/tuple.hpp
 
-#include tools/solar.h
-#include tools/debug.hxx
 #include svl/svstdarr.hxx
-#include com/sun/star/beans/XPropertySet.hpp
-#include com/sun/star/beans/PropertyValue.hpp
 #include com/sun/star/container/XEnumerationAccess.hpp
-#include com/sun/star/container/XNameContainer.hpp
 #include com/sun/star/style/XStyleFamiliesSupplier.hpp
-#include com/sun/star/text/XTextCursor.hpp
 #include com/sun/star/text/XChapterNumberingSupplier.hpp
 #include com/sun/star/text/XTextFramesSupplier.hpp
 #include com/sun/star/text/XTextGraphicObjectsSupplier.hpp
 #include com/sun/star/text/XTextEmbeddedObjectsSupplier.hpp
-#include com/sun/star/text/TextContentAnchorType.hpp
-#include com/sun/star/text/XTextFrame.hpp
 #include com/sun/star/text/XFormField.hpp
-#include com/sun/star/drawing/XShapes.hpp
-#include com/sun/star/util/DateTime.hpp
-#include com/sun/star/lang/XMultiServiceFactory.hpp
 #include com/sun/star/ucb/XAnyCompareFactory.hpp
-#include com/sun/star/container/XNamed.hpp
-#include xmloff/xmltoken.hxx
 #include xmloff/xmlnmspe.hxx
 #include xmloff/txtstyli.hxx
-#include xmloff/families.hxx
-#include xmloff/xmlnumfi.hxx
 #include xmloff/xmlnumi.hxx
-#include xmloff/xmlictxt.hxx
-#include xmloff/xmlimppr.hxx
 
 #include txtparai.hxx
-#include xmloff/txtimp.hxx
 #include xmloff/txtprmap.hxx
 #include xmloff/txtimppr.hxx
 #include xmloff/xmlimp.hxx
@@ -72,13 +53,10 @@
 #include XMLTextListItemContext.hxx
 #include XMLTextListBlockContext.hxx
 #include XMLTextFrameContext.hxx
-#include XMLPropertyBackpatcher.hxx
 #include XMLTextFrameHyperlinkContext.hxx
 #include XMLSectionImportContext.hxx
 #include XMLIndexTOCContext.hxx
 #include xmloff/XMLFontStylesContext.hxx
-#include xmloff/ProgressBarHelper.hxx
-#include xmloff/nmspmap.hxx
 #include xmloff/XMLEventsImportContext.hxx
 #include XMLTrackedChangesImportContext.hxx
 #include XMLChangeImportContext.hxx
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-06-27 Thread Takeshi Abe
 xmloff/source/chart/SchXMLChartContext.hxx|5 -
 xmloff/source/chart/SchXMLParagraphContext.hxx|2 --
 xmloff/source/chart/SchXMLTableContext.hxx|4 
 xmloff/source/draw/layerimp.hxx   |2 --
 xmloff/source/draw/sdxmlexp_impl.hxx  |9 -
 xmloff/source/draw/sdxmlimp_impl.hxx  |4 
 xmloff/source/draw/ximpcustomshape.hxx|1 -
 xmloff/source/forms/callbacks.hxx |4 
 xmloff/source/forms/elementimport.hxx |1 -
 xmloff/source/forms/formattributes.hxx|1 -
 xmloff/source/forms/layerimport.hxx   |4 +---
 xmloff/source/style/impastpl.hxx  |1 -
 xmloff/source/text/XMLRedlineExport.hxx   |1 -
 xmloff/source/text/XMLSectionImportContext.hxx|1 -
 xmloff/source/text/XMLTextFrameHyperlinkContext.hxx   |3 ---
 xmloff/source/text/XMLTrackedChangesImportContext.hxx |4 
 xmloff/source/transform/CreateElemTContext.hxx|2 --
 xmloff/source/transform/ProcAddAttrTContext.hxx   |2 --
 xmloff/source/transform/ProcAttrTContext.hxx  |2 --
 19 files changed, 1 insertion(+), 52 deletions(-)

New commits:
commit 6013882bdff6265548a1511799611243a9d0e083
Author: Takeshi Abe t...@fixedpoint.jp
Date:   Wed Jun 27 20:03:57 2012 +0900

removed unnecessary forward declarations of class

except moving SvXMLStyleContext into xmloff/source/forms/layerimport.hxx

Change-Id: I5e16b54f98ccdd54bc0bb373d3fe92ed1bef3c5e

diff --git a/xmloff/source/chart/SchXMLChartContext.hxx 
b/xmloff/source/chart/SchXMLChartContext.hxx
index 249cbdb..4e39113 100644
--- a/xmloff/source/chart/SchXMLChartContext.hxx
+++ b/xmloff/source/chart/SchXMLChartContext.hxx
@@ -40,18 +40,13 @@
 #include list
 #include map
 
-class SchXMLImport;
 class SchXMLImportHelper;
 
 namespace com { namespace sun { namespace star {
 namespace chart {
 class XChartDocument;
-class XDiagram;
 struct ChartSeriesAddress;
 }
-namespace util {
-class XStringMapping;
-}
 namespace xml { namespace sax {
 class XAttributeList;
 }}
diff --git a/xmloff/source/chart/SchXMLParagraphContext.hxx 
b/xmloff/source/chart/SchXMLParagraphContext.hxx
index 31b8337..ba246ee 100644
--- a/xmloff/source/chart/SchXMLParagraphContext.hxx
+++ b/xmloff/source/chart/SchXMLParagraphContext.hxx
@@ -23,8 +23,6 @@
 #include rtl/ustring.hxx
 #include rtl/ustrbuf.hxx
 
-class SchXMLImport;
-
 namespace com { namespace sun { namespace star { namespace xml { namespace sax 
{
 class XAttributeList;
 }
diff --git a/xmloff/source/chart/SchXMLTableContext.hxx 
b/xmloff/source/chart/SchXMLTableContext.hxx
index 005fda9..3285849 100644
--- a/xmloff/source/chart/SchXMLTableContext.hxx
+++ b/xmloff/source/chart/SchXMLTableContext.hxx
@@ -28,15 +28,11 @@
 #include transporttypes.hxx
 
 namespace com { namespace sun { namespace star {
-namespace frame {
-class XModel;
-}
 namespace xml { namespace sax {
 class XAttributeList;
 }}
 namespace chart {
 class XChartDocument;
-struct ChartSeriesAddress;
 
 
 // 
diff --git a/xmloff/source/draw/layerimp.hxx b/xmloff/source/draw/layerimp.hxx
index 47eb16a..09464a9 100644
--- a/xmloff/source/draw/layerimp.hxx
+++ b/xmloff/source/draw/layerimp.hxx
@@ -23,8 +23,6 @@
 #include xmloff/xmlictxt.hxx
 #include com/sun/star/container/XNameAccess.hpp
 
-class AnimImpImpl;
-
 //
 // presentations:animations
 
diff --git a/xmloff/source/draw/sdxmlexp_impl.hxx 
b/xmloff/source/draw/sdxmlexp_impl.hxx
index 0558fae..bb95104 100644
--- a/xmloff/source/draw/sdxmlexp_impl.hxx
+++ b/xmloff/source/draw/sdxmlexp_impl.hxx
@@ -41,20 +41,11 @@
 
 //
 
-class SvXMLUnitConverter;
-class SvXMLExportItemMapper;
-class SfxPoolItem;
-class SfxItemSet;
-class OUStrings_Impl;
 class Rectangle;
 
-class ImpPresPageDrawStylePropMapper;
 class ImpXMLEXPPageMasterInfo;
-class ImpXMLDrawPageInfoList;
 class ImpXMLAutoLayoutInfo;
-class SvXMLAutoStylePoolP;
 class XMLSdPropHdlFactory;
-class ImpXMLShapeStyleInfo;
 class XMLShapeExportPropertyMapper;
 class XMLPageExportPropertyMapper;
 
diff --git a/xmloff/source/draw/sdxmlimp_impl.hxx 
b/xmloff/source/draw/sdxmlimp_impl.hxx
index d6bd917..f13524a 100644
--- a/xmloff/source/draw/sdxmlimp_impl.hxx
+++ b/xmloff/source/draw/sdxmlimp_impl.hxx
@@ -169,10 +169,6 @@ typedef std::map sal_Int32, 
com::sun::star::uno::Reference com::sun::star::dra
 
 class SvXMLUnitConverter;
 class SvXMLTokenMap;
-class XMLSdPropHdlFactory;
-class XMLPropertySetMapper;
-class XMLPropStyleContext;
-class SdXMLStylesContext;
 class 

[Libreoffice-commits] .: xmloff/source

2012-06-12 Thread Eike Rathke
 xmloff/source/chart/SchXMLChartContext.cxx |8 
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 0f79440a86dc788a38ef4b3be63bffd2cd396ddc
Author: Eike Rathke er...@redhat.com
Date:   Tue Jun 12 11:51:55 2012 +0200

work around spurious signed overflow warnings

gcc (GCC) 4.6.3 20120306 (Red Hat 4.6.3-2)
com/sun/star/uno/Sequence.hxx:178:10: error: assuming signed overflow does 
not
occur when assuming that (X - c)  X is always false 
[-Werror=strict-overflow]

Change-Id: Ibbc2f3533097ad379e1707f77039c675a0d00133

diff --git a/xmloff/source/chart/SchXMLChartContext.cxx 
b/xmloff/source/chart/SchXMLChartContext.cxx
index 801d412..cdadaf4 100644
--- a/xmloff/source/chart/SchXMLChartContext.cxx
+++ b/xmloff/source/chart/SchXMLChartContext.cxx
@@ -636,7 +636,7 @@ void lcl_ApplyDataFromRectangularRangeToDiagram(
 if( !sColTrans.isEmpty() || !sRowTrans.isEmpty() )
 {
 aArgs.realloc( aArgs.getLength() + 1 );
-aArgs[ aArgs.getLength() - 1 ] = beans::PropertyValue(
+aArgs[ sal::static_int_castsal_uInt32(aArgs.getLength()) - 1 ] = 
beans::PropertyValue(
 ::rtl::OUString( SequenceMapping ),
 -1, uno::makeAny( !sColTrans.isEmpty()
 ? lcl_getNumberSequenceFromString( sColTrans, bHasCateories  
!xNewDoc-hasInternalDataProvider() )
@@ -662,7 +662,7 @@ void lcl_ApplyDataFromRectangularRangeToDiagram(
 if( !aChartOleObjectName.isEmpty() )
 {
 aArgs.realloc( aArgs.getLength() + 1 );
-aArgs[ aArgs.getLength() - 1 ] = beans::PropertyValue(
+aArgs[ sal::static_int_castsal_uInt32(aArgs.getLength()) - 1 ] = 
beans::PropertyValue(
 ::rtl::OUString( ChartOleObjectName ),
 -1, uno::makeAny( aChartOleObjectName ),
 beans::PropertyState_DIRECT_VALUE );
@@ -674,11 +674,11 @@ void lcl_ApplyDataFromRectangularRangeToDiagram(
 xDataProvider-createDataSource( aArgs ));
 
 aArgs.realloc( aArgs.getLength() + 2 );
-aArgs[ aArgs.getLength() - 2 ] = beans::PropertyValue(
+aArgs[ sal::static_int_castsal_uInt32(aArgs.getLength()) - 2 ] = 
beans::PropertyValue(
 ::rtl::OUString( HasCategories ),
 -1, uno::makeAny( bHasCateories ),
 beans::PropertyState_DIRECT_VALUE );
-aArgs[ aArgs.getLength() - 1 ] = beans::PropertyValue(
+aArgs[ sal::static_int_castsal_uInt32(aArgs.getLength()) - 1 ] = 
beans::PropertyValue(
 ::rtl::OUString(UseCategoriesAsX),
 -1, uno::makeAny( sal_False ),//categories in ODF files are not to be 
used as x values (independent from what is offered in our ui)
 beans::PropertyState_DIRECT_VALUE );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-06-05 Thread Thorsten Behrens
 xmloff/source/chart/PropertyMap.hxx  |2 +-
 xmloff/source/chart/SchXMLExport.cxx |4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 0e9598fa6c388024fa788358e26ef77b5fc3f074
Author: Thorsten Behrens tbehr...@suse.com
Date:   Tue Jun 5 10:52:17 2012 +0200

Catch more remnant ODFVER_LATEST instances.

This extends commit d0af38eaa029c2f2c2617544c8b6798c463381b9,
converting more latest-ver to 1.2-extended cases.

diff --git a/xmloff/source/chart/PropertyMap.hxx 
b/xmloff/source/chart/PropertyMap.hxx
index b295eed..5ef9021 100644
--- a/xmloff/source/chart/PropertyMap.hxx
+++ b/xmloff/source/chart/PropertyMap.hxx
@@ -94,7 +94,7 @@
 #define MAP_FULL( ApiName, NameSpace, XMLTokenName, XMLType, ContextId, 
EarliestODFVersionForExport ) { ApiName, sizeof(ApiName)-1, 
XML_NAMESPACE_##NameSpace, xmloff::token::XMLTokenName, 
XMLType|XML_TYPE_PROP_CHART, ContextId, EarliestODFVersionForExport }
 #define MAP_ENTRY( a, ns, nm, t ){ a, sizeof(a)-1, 
XML_NAMESPACE_##ns, xmloff::token::nm, t|XML_TYPE_PROP_CHART, 0, 
SvtSaveOptions::ODFVER_010 }
 #define MAP_ENTRY_ODF12( a, ns, nm, t )  { a, sizeof(a)-1, 
XML_NAMESPACE_##ns, xmloff::token::nm, t|XML_TYPE_PROP_CHART, 0, 
SvtSaveOptions::ODFVER_012 }
-#define MAP_ENTRY_ODF_EXT( a, ns, nm, t ){ a, sizeof(a)-1, 
XML_NAMESPACE_##ns, xmloff::token::nm, t|XML_TYPE_PROP_CHART, 0, 
SvtSaveOptions::ODFVER_LATEST }
+#define MAP_ENTRY_ODF_EXT( a, ns, nm, t ){ a, sizeof(a)-1, 
XML_NAMESPACE_##ns, xmloff::token::nm, t|XML_TYPE_PROP_CHART, 0, 
SvtSaveOptions::ODFVER_012_EXT_COMPAT }
 #define MAP_CONTEXT( a, ns, nm, t, c )   { a, sizeof(a)-1, 
XML_NAMESPACE_##ns, xmloff::token::nm, t|XML_TYPE_PROP_CHART, c, 
SvtSaveOptions::ODFVER_010 }
 #define MAP_SPECIAL( a, ns, nm, t, c )   { a, sizeof(a)-1, 
XML_NAMESPACE_##ns, xmloff::token::nm, t|XML_TYPE_PROP_CHART | 
MID_FLAG_SPECIAL_ITEM, c, SvtSaveOptions::ODFVER_010 }
 #define MAP_SPECIAL_ODF12( a, ns, nm, t, c ) { a, sizeof(a)-1, 
XML_NAMESPACE_##ns, xmloff::token::nm, t|XML_TYPE_PROP_CHART | 
MID_FLAG_SPECIAL_ITEM, c, SvtSaveOptions::ODFVER_012 }
diff --git a/xmloff/source/chart/SchXMLExport.cxx 
b/xmloff/source/chart/SchXMLExport.cxx
index 9cb5bc5..83a3491 100755
--- a/xmloff/source/chart/SchXMLExport.cxx
+++ b/xmloff/source/chart/SchXMLExport.cxx
@@ -3119,7 +3119,7 @@ void SchXMLExportHelper_Impl::exportErrorBar( const 
Referencebeans::XPropertySe
 const SvtSaveOptions::ODFDefaultVersion nCurrentVersion( 
SvtSaveOptions().GetODFDefaultVersion() );
 
 /// Dont export X ErrorBars for older ODF versions.
-if ( !bYError  nCurrentVersion != SvtSaveOptions::ODFVER_LATEST )
+if ( !bYError  nCurrentVersion = SvtSaveOptions::ODFVER_012 )
 return;
 
 if (xSeriesProp.is())
@@ -3168,7 +3168,7 @@ void SchXMLExportHelper_Impl::exportErrorBar( const 
Referencebeans::XPropertySe
 for( ::std::vector Reference chart2::data::XDataSequence  
::const_iterator aIt(
  aErrorBarSequences.begin()); aIt != 
aErrorBarSequences.end(); ++aIt )
 {
-if ( nCurrentVersion == SvtSaveOptions::ODFVER_LATEST )
+if ( nCurrentVersion  SvtSaveOptions::ODFVER_012 )
 {
 rtl::OUString aRole, aRange;
 Reference beans::XPropertySet  xSeqProp( *aIt, 
uno::UNO_QUERY_THROW );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-06-05 Thread Michael Stahl
 xmloff/source/text/txtparae.cxx |   49 
 1 file changed, 20 insertions(+), 29 deletions(-)

New commits:
commit 4190e9351e69730f0097e6cdf4dad2fafa10577e
Author: Michael Stahl mst...@redhat.com
Date:   Tue Jun 5 13:51:09 2012 +0200

fdo#30711: fix some problems in 7a1c5e54:

The elements are exported without checking for bAutoStyles, and the Any
conversion is overly verbose.

Change-Id: Iebb52c8cd2638424dd48d28ec5a12c10a41f6cbf

diff --git a/xmloff/source/text/txtparae.cxx b/xmloff/source/text/txtparae.cxx
index e7b370b..04b008c 100644
--- a/xmloff/source/text/txtparae.cxx
+++ b/xmloff/source/text/txtparae.cxx
@@ -2295,16 +2295,15 @@ void XMLTextParagraphExport::exportTextRangeEnumeration(
 if (xParameters.is()  xParameters-hasByName(Name))
 {
 const Any aValue = xParameters-getByName(Name);
-const Type aValueType = aValue.getValueType();
-if (aValueType == ::getCppuType((OUString*)0))
+OUString sValue;
+if (aValue = sValue)
 {
-OUString sValue;
-aValue = sValue;
 GetExport().AddAttribute(XML_NAMESPACE_TEXT, 
XML_NAME, sValue);
 }
 }
-GetExport().StartElement(XML_NAMESPACE_TEXT, 
XML_BOOKMARK_START, sal_False);
-GetExport().EndElement(XML_NAMESPACE_TEXT, 
XML_BOOKMARK_START, sal_False);
+SvXMLElementExport aElem( GetExport(), !bAutoStyles,
+XML_NAMESPACE_TEXT, XML_BOOKMARK_START,
+sal_False, sal_False );
 const OUString sFieldType = xFormField-getFieldType();
 if (sFieldType ==  ODF_FORMTEXT)
 {
@@ -2319,19 +2318,15 @@ void XMLTextParagraphExport::exportTextRangeEnumeration(
 openFieldMark = NONE;
 }
 }
-else
-{
-GetExport().StartElement(XML_NAMESPACE_TEXT, 
XML_BOOKMARK_START, sal_False);
-GetExport().EndElement(XML_NAMESPACE_TEXT, 
XML_BOOKMARK_START, sal_False);
-}
 }
 }
 else if (sType.equals(sTextFieldEnd))
 {
 if ( GetExport().getDefaultVersion()  
SvtSaveOptions::ODFVER_012 )
 {
-GetExport().StartElement(XML_NAMESPACE_FIELD, 
XML_FIELDMARK_END, sal_False);
-GetExport().EndElement(XML_NAMESPACE_FIELD, 
XML_FIELDMARK_END, sal_False);
+SvXMLElementExport aElem( GetExport(), !bAutoStyles,
+XML_NAMESPACE_FIELD, XML_FIELDMARK_END,
+sal_False, sal_False );
 }
 else
 {
@@ -2342,17 +2337,16 @@ void XMLTextParagraphExport::exportTextRangeEnumeration(
 if (xParameters.is()  xParameters-hasByName(Name))
 {
 const Any aValue = xParameters-getByName(Name);
-const Type aValueType = aValue.getValueType();
-if (aValueType == ::getCppuType((OUString*)0))
+OUString sValue;
+if (aValue = sValue)
 {
-OUString sValue;
-aValue = sValue;
 GetExport().AddAttribute(XML_NAMESPACE_TEXT, 
XML_NAME, sValue);
 }
 }
 }
-GetExport().StartElement(XML_NAMESPACE_TEXT, 
XML_BOOKMARK_END, sal_False);
-
GetExport().EndElement(XML_NAMESPACE_TEXT,XML_BOOKMARK_END, sal_False);
+SvXMLElementExport aElem( GetExport(), !bAutoStyles,
+XML_NAMESPACE_TEXT, XML_BOOKMARK_END,
+sal_False, sal_False );
 }
 }
 else if (sType.equals(sTextFieldStartEnd))
@@ -2383,8 +2377,9 @@ void XMLTextParagraphExport::exportTextRangeEnumeration(
 {
 GetExport().AddAttribute(XML_NAMESPACE_TEXT, XML_NAME, 
xBookmark-getName());
 }
-GetExport().StartElement(XML_NAMESPACE_TEXT, XML_BOOKMARK, 
sal_False);
-GetExport().EndElement(XML_NAMESPACE_TEXT, XML_BOOKMARK, 
sal_False);
+SvXMLElementExport aElem( GetExport(), !bAutoStyles,
+XML_NAMESPACE_TEXT, XML_BOOKMARK,
+  

[Libreoffice-commits] .: xmloff/source

2012-05-22 Thread Tor Lillqvist
 xmloff/source/style/impastp4.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 8e18130bba49082cc1416e1a2631bd5a4cb3cb8f
Author: Tor Lillqvist t...@iki.fi
Date:   Tue May 22 14:19:19 2012 +0300

WaE: unused parameter 'rStrName'

Change-Id: Ia760fe7ae66f8caadb10947c3d743a9f64a55dc5

diff --git a/xmloff/source/style/impastp4.cxx b/xmloff/source/style/impastp4.cxx
index 3fda512..32cd3f8 100644
--- a/xmloff/source/style/impastp4.cxx
+++ b/xmloff/source/style/impastp4.cxx
@@ -106,7 +106,7 @@ void SvXMLAutoStylePoolP_Impl::AddFamily(
 
 void SvXMLAutoStylePoolP_Impl::SetFamilyPropSetMapper(
 sal_Int32 nFamily,
-const OUString rStrName,
+const OUString /* rStrName */,
 const UniReference  SvXMLExportPropertyMapper   rMapper )
 {
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-05-17 Thread Caolán McNamara
 xmloff/source/forms/elementimport.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 5e0b52c1893ffe04a68333026afa7a557c48d534
Author: Caolán McNamara caol...@redhat.com
Date:   Thu May 17 14:51:04 2012 +0100

some more ConstAsciiString behind OSL_ENSURE

Change-Id: Ic93ec84ce62234364d233a9188a65439e88a79fe

diff --git a/xmloff/source/forms/elementimport.cxx 
b/xmloff/source/forms/elementimport.cxx
index 5885914..8d29aaa 100644
--- a/xmloff/source/forms/elementimport.cxx
+++ b/xmloff/source/forms/elementimport.cxx
@@ -993,8 +993,8 @@ namespace xmloff
 {
 // we have exactly 2 properties where this type class is allowed:
 OSL_ENSURE(
-(0 == 
_rPropValue.Name.compareToAscii(PROPERTY_EFFECTIVE_VALUE))
-||  (0 == 
_rPropValue.Name.compareToAscii(PROPERTY_EFFECTIVE_DEFAULT)),
+(0 == 
_rPropValue.Name.equalsAsciiL(PROPERTY_EFFECTIVE_VALUE.ascii, 
PROPERTY_EFFECTIVE_VALUE.length))
+||  (0 == 
_rPropValue.Name.equalsAsciiL(PROPERTY_EFFECTIVE_DEFAULT.ascii, 
PROPERTY_EFFECTIVE_DEFAULT.length)),
 OControlImport::implTranslateValueProperty: invalid property 
type/name combination!);
 
 // Both properties are allowed to have a double or a string value,
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-05-16 Thread Caolán McNamara
 xmloff/source/draw/sdxmlimp.cxx |4 
 1 file changed, 4 deletions(-)

New commits:
commit cc92cb54e246e48dc62ef149aeaa550e14ec3a9e
Author: Caolán McNamara caol...@redhat.com
Date:   Wed May 16 15:16:53 2012 +0100

completely nutso merge bustage since 2004

useless unused fat global objects since
146f73bcfa48d18a1eb449aaa5ddd43d2dd2

Change-Id: I2a5f54d34b498a64b658f158f98269f16e2e71f8

diff --git a/xmloff/source/draw/sdxmlimp.cxx b/xmloff/source/draw/sdxmlimp.cxx
index d1f960c..2687a18 100644
--- a/xmloff/source/draw/sdxmlimp.cxx
+++ b/xmloff/source/draw/sdxmlimp.cxx
@@ -1029,10 +1029,6 @@ OUString SAL_CALL SdXMLImport::getImplementationName() 
throw( uno::RuntimeExcept
 }
 }
 
-HeaderFooterDeclMap maHeaderDeclsMap;
-HeaderFooterDeclMap maFooterDeclsMap;
-DateTimeDeclMap maDateTimeDeclsMap;
-
 void SdXMLImport::AddHeaderDecl( const ::rtl::OUString rName, const 
::rtl::OUString rText )
 {
 if( !rName.isEmpty()  !rText.isEmpty() )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-04-11 Thread Ivan Timofeev
 xmloff/source/chart/SchXMLExport.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 9c323a4bbe0f7dec33523e87e1c1010dd832d4bf
Author: Ivan Timofeev timofeev@gmail.com
Date:   Wed Apr 11 22:18:22 2012 +0400

fix windows build

diff --git a/xmloff/source/chart/SchXMLExport.cxx 
b/xmloff/source/chart/SchXMLExport.cxx
index 0d253d2..6632b47 100644
--- a/xmloff/source/chart/SchXMLExport.cxx
+++ b/xmloff/source/chart/SchXMLExport.cxx
@@ -3137,7 +3137,7 @@ void SchXMLExportHelper_Impl::exportErrorBar( const 
Referencebeans::XPropertySe
 {
 Any aAny;
 
-aAny = xSeriesProp-getPropertyValue( bYError ? ErrorBarY : 
ErrorBarX );
+aAny = xSeriesProp-getPropertyValue( bYError ? 
rtl::OUString(ErrorBarY) : rtl::OUString(ErrorBarX) );
 aAny = xErrorBarProp;
 
 if ( xErrorBarProp.is() )
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-03-26 Thread Thorsten Behrens
 xmloff/source/draw/xexptran.cxx |   10 ++
 1 file changed, 10 insertions(+)

New commits:
commit a2ee8055e9c136923f0244fe289cac6377933c31
Author: Thorsten Behrens tbehr...@suse.com
Date:   Fri Mar 23 17:46:43 2012 +0100

Fix fdo#47406 incorrect relative moves after closePath

I missed the fact that odf export still uses the old exporter in
xmloff/source/draw/xexptran.cxx, which was not fixed wrt. the update
of current position in basegfx/source/polygon/b2dsvgpolypolygon.cxx

diff --git a/xmloff/source/draw/xexptran.cxx b/xmloff/source/draw/xexptran.cxx
index 15ef848..6032984 100644
--- a/xmloff/source/draw/xexptran.cxx
+++ b/xmloff/source/draw/xexptran.cxx
@@ -1606,6 +1606,7 @@ void SdXMLImExSvgDElement::AddPolygon(
 
 // bezier poly, handle curves
 bool  bDidWriteStart(false);
+sal_Int32 nStartX(0), nStartY(0);
 
 for(sal_Int32 a(0L); a  nCnt; a++)
 {
@@ -2080,6 +2081,8 @@ void SdXMLImExSvgDElement::AddPolygon(
 
 // remember start written
 bDidWriteStart = true;
+nStartX = nX;
+nStartY = nY;
 }
 
 // remember new last position
@@ -2100,6 +2103,13 @@ void SdXMLImExSvgDElement::AddPolygon(
 aNewString += rtl::OUString(static_castsal_Unicode('z'));
 else
 aNewString += rtl::OUString(static_castsal_Unicode('Z'));
+
+// update current point - we're back at the start
+if( bDidWriteStart )
+{
+mnLastX = nStartX;
+mnLastY = nStartY;
+}
 }
 
 // append new string
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-03-18 Thread Julien Nabet
 xmloff/source/chart/MultiPropertySetHandler.hxx |8 
 xmloff/source/core/RDFaImportHelper.cxx |2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 9032c7f63e28e5a9de00b3b868452581f6f5f145
Author: Julien Nabet serval2...@yahoo.fr
Date:   Sun Mar 18 17:55:24 2012 +0100

Some cppcheck cleaning

diff --git a/xmloff/source/chart/MultiPropertySetHandler.hxx 
b/xmloff/source/chart/MultiPropertySetHandler.hxx
index ab0ce3a..953e943 100644
--- a/xmloff/source/chart/MultiPropertySetHandler.hxx
+++ b/xmloff/source/chart/MultiPropertySetHandler.hxx
@@ -209,7 +209,7 @@ MultiPropertySetHandler::MultiPropertySetHandler 
(::com::sun::star::uno::Referen
 MultiPropertySetHandler::~MultiPropertySetHandler (void)
 {
 ::std::map ::rtl::OUString, PropertyWrapperBase*, 
OUStringComparison::iterator I;
-for (I=aPropertyList.begin(); I!=aPropertyList.end(); I++)
+for (I=aPropertyList.begin(); I!=aPropertyList.end(); ++I)
 delete I-second;
 }
 
@@ -220,7 +220,7 @@ sal_BoolMultiPropertySetHandler::GetProperties  (void)
 ::std::map ::rtl::OUString, PropertyWrapperBase*, 
OUStringComparison::iterator I;
 ::com::sun::star::uno::Sequence ::rtl::OUString aNameList 
(aPropertyList.size());
 int i;
-for (I=aPropertyList.begin(),i=0; I!=aPropertyList.end(); I++)
+for (I=aPropertyList.begin(),i=0; I!=aPropertyList.end(); ++I)
 aNameList[i++] = I-second-msName;
 if ( ! MultiGet(aNameList))
 if ( ! SingleGet(aNameList))
@@ -243,7 +243,7 @@ sal_BoolMultiPropertySetHandler::MultiGet   (const 
::com::sun::star::uno::Se
 int i;
 ::com::sun::star::uno::Sequence ::com::sun::star::uno::Any 
aValueList =
 xMultiSet-getPropertyValues (rNameList);
-for (I=aPropertyList.begin(),i=0; I!=aPropertyList.end(); I++)
+for (I=aPropertyList.begin(),i=0; I!=aPropertyList.end(); ++I)
 I-second-SetValue (aValueList[i++]);
 }
 catch (const ::com::sun::star::beans::UnknownPropertyException)
@@ -269,7 +269,7 @@ sal_BoolMultiPropertySetHandler::SingleGet  (const 
::com::sun::star::uno::Se
 {
 ::std::map ::rtl::OUString, PropertyWrapperBase*, 
OUStringComparison::iterator I;
 int i;
-for (I=aPropertyList.begin(),i=0; I!=aPropertyList.end(); I++)
+for (I=aPropertyList.begin(),i=0; I!=aPropertyList.end(); ++I)
 I-second-SetValue (xSingleSet-getPropertyValue 
(rNameList[i++]));
 }
 catch (const ::com::sun::star::beans::UnknownPropertyException)
diff --git a/xmloff/source/core/RDFaImportHelper.cxx 
b/xmloff/source/core/RDFaImportHelper.cxx
index 6b980d4..be466c8 100644
--- a/xmloff/source/core/RDFaImportHelper.cxx
+++ b/xmloff/source/core/RDFaImportHelper.cxx
@@ -236,7 +236,7 @@ RDFaReader::ReadCURIEs(::rtl::OUString const  i_rCURIEs) 
const
   }
 }
 while (!CURIEs.isEmpty());
-if (!vec.size())
+if (vec.empty())
 {
 OSL_TRACE( ReadCURIEs: invalid CURIEs );
 }
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-03-18 Thread Julien Nabet
 xmloff/source/text/XMLRedlineExport.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b1b4f699fc47d2c40034453754d01a1bbb0a7734
Author: Julien Nabet serval2...@yahoo.fr
Date:   Sun Mar 18 17:58:35 2012 +0100

Prefer prefix ++/-- operators for non-primitive types

diff --git a/xmloff/source/text/XMLRedlineExport.cxx 
b/xmloff/source/text/XMLRedlineExport.cxx
index 007aac1..917fb95 100644
--- a/xmloff/source/text/XMLRedlineExport.cxx
+++ b/xmloff/source/text/XMLRedlineExport.cxx
@@ -177,7 +177,7 @@ void XMLRedlineExport::ExportChangesList(
 // iterate over changes list
 for( ChangesListType::iterator aIter = pChangesList-begin();
  aIter != pChangesList-end();
- aIter++ )
+ ++aIter )
 {
 ExportChangedRegion( *aIter );
 }
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-03-08 Thread Stephan Bergmann
 xmloff/source/chart/transporttypes.hxx |   15 +--
 1 file changed, 1 insertion(+), 14 deletions(-)

New commits:
commit c361ef86504e79a1e1b56fa3a7eba2f893386e2c
Author: Stephan Bergmann sberg...@redhat.com
Date:   Thu Mar 8 20:39:35 2012 +0100

Compiler-generated code is just fine

...now that member is Sequence instead of pointer to Sequence.

diff --git a/xmloff/source/chart/transporttypes.hxx 
b/xmloff/source/chart/transporttypes.hxx
index cb03cbc..7ecf086 100644
--- a/xmloff/source/chart/transporttypes.hxx
+++ b/xmloff/source/chart/transporttypes.hxx
@@ -50,21 +50,8 @@ struct SchXMLCell
 SchXMLCellType eType;
 rtl::OUString aRangeId;
 
-SchXMLCell() : aComplexString(), fValue( 0.0 ), eType( 
SCH_CELL_TYPE_UNKNOWN )
+SchXMLCell(): fValue( 0.0 ), eType( SCH_CELL_TYPE_UNKNOWN )
 {}
-
-SchXMLCell( const SchXMLCell rOther )
-: aString( rOther.aString )
-, aComplexString( rOther.aComplexString )
-, fValue( rOther.fValue )
-, eType( rOther.eType )
-, aRangeId( rOther.aRangeId )
-{}
-
-~SchXMLCell()
-{
-
-}
 };
 
 struct SchXMLTable
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-02-10 Thread Jan Holesovsky
 xmloff/source/text/txtparai.cxx |   14 ++
 1 file changed, 10 insertions(+), 4 deletions(-)

New commits:
commit 9951de4a10386a85dc1bfa465304cd0005414b8e
Author: Jan Holesovsky ke...@suse.cz
Date:   Fri Feb 10 17:54:03 2012 +0100

fdo#42771: Fix crash when loading an invalid .fodt.

createTextCursorByRange() likes to throw runtime exception, even though it
just means 'we were unable to create the cursor'.

diff --git a/xmloff/source/text/txtparai.cxx b/xmloff/source/text/txtparai.cxx
index 721e03b..70848c4 100644
--- a/xmloff/source/text/txtparai.cxx
+++ b/xmloff/source/text/txtparai.cxx
@@ -2031,10 +2031,16 @@ XMLParaContext::~XMLParaContext()
 xTxtImport-InsertControlCharacter( ControlCharacter::APPEND_PARAGRAPH );
 
 // create a cursor that select the whole last paragraph
-Reference  XTextCursor  xAttrCursor(
-xTxtImport-GetText()-createTextCursorByRange( xStart ));
-if( !xAttrCursor.is() )
-return; // Robust (defect file)
+Reference  XTextCursor  xAttrCursor;
+try {
+xAttrCursor = xTxtImport-GetText()-createTextCursorByRange( xStart );
+if( !xAttrCursor.is() )
+return; // Robust (defect file)
+} catch (uno::Exception ) {
+// createTextCursorByRange() likes to throw runtime exception, even
+// though it just means 'we were unable to create the cursor'
+return;
+}
 xAttrCursor-gotoRange( xEnd, sal_True );
 
 // xml:id for RDF metadata
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-01-25 Thread Olivier Hallot
 xmloff/source/forms/controlpropertyhdl.cxx |6 +-
 xmloff/source/forms/elementexport.cxx  |   24 +-
 xmloff/source/forms/elementimport.cxx  |   48 ++---
 xmloff/source/forms/eventexport.cxx|4 -
 xmloff/source/forms/eventimport.cxx|2 
 xmloff/source/forms/formcellbinding.cxx|4 -
 xmloff/source/forms/layerexport.cxx|4 -
 xmloff/source/forms/layerimport.cxx|   10 ++--
 xmloff/source/forms/property_meta_data.cxx |   10 ++--
 xmloff/source/forms/propertyexport.cxx |   20 
 xmloff/source/forms/propertyimport.cxx |   10 ++--
 xmloff/source/meta/xmlmetae.cxx|   14 +++---
 xmloff/source/meta/xmlmetai.cxx|4 -
 xmloff/source/meta/xmlversion.cxx  |4 -
 xmloff/source/script/XMLStarBasicExportHandler.cxx |2 
 xmloff/source/script/xmlbasici.cxx |2 
 xmloff/source/style/DashStyle.cxx  |4 -
 xmloff/source/style/FillStyleContext.cxx   |4 -
 xmloff/source/style/GradientStyle.cxx  |4 -
 xmloff/source/style/HatchStyle.cxx |4 -
 xmloff/source/style/ImageStyle.cxx |8 +--
 xmloff/source/style/MarkerStyle.cxx|4 -
 xmloff/source/style/PageMasterImportContext.cxx|2 
 xmloff/source/style/PageMasterPropHdl.cxx  |   12 ++---
 xmloff/source/style/TransGradientStyle.cxx |4 -
 xmloff/source/style/XMLBackgroundImageContext.cxx  |6 +-
 xmloff/source/style/XMLBackgroundImageExport.cxx   |8 +--
 xmloff/source/style/XMLFontAutoStylePool.cxx   |4 -
 xmloff/source/style/XMLPageExport.cxx  |4 -
 xmloff/source/style/bordrhdl.cxx   |2 
 xmloff/source/style/cdouthdl.cxx   |2 
 xmloff/source/style/chrhghdl.cxx   |6 +-
 xmloff/source/style/chrlohdl.cxx   |4 -
 xmloff/source/style/csmaphdl.cxx   |2 
 34 files changed, 126 insertions(+), 126 deletions(-)

New commits:
commit 3dbb89e5a39e7811d2fc0c1fbad012c3d565396b
Author: Olivier Hallot olivier.hal...@alta.org.br
Date:   Wed Jan 25 21:59:48 2012 -0200

Fix for fdo43460 Part XLVIII getLength() to isEmpty()

Part XLVIII
Modules
xmloff (part 2)

diff --git a/xmloff/source/forms/controlpropertyhdl.cxx 
b/xmloff/source/forms/controlpropertyhdl.cxx
index 3c8b0fc..5442a55 100644
--- a/xmloff/source/forms/controlpropertyhdl.cxx
+++ b/xmloff/source/forms/controlpropertyhdl.cxx
@@ -245,7 +245,7 @@ namespace xmloff
 sal_uInt16 nStyle = 1;
 
 while   (   aTokens.getNextToken(sToken)// have a new token
-  (0 != sToken.getLength())   // really have a new token
+  (!sToken.isEmpty())   // really have a new token
 )
 {
 // try interpreting the token as border style
@@ -304,7 +304,7 @@ namespace xmloff
 if ( !bSuccess )
 return sal_False;
 
-if ( _rStrExpValue.getLength() )
+if ( !_rStrExpValue.isEmpty() )
 _rStrExpValue += ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM(   
) );
 _rStrExpValue += aOut.makeStringAndClear();
 
@@ -343,7 +343,7 @@ namespace xmloff
 }
 _rStrExpValue = aResult.makeStringAndClear();
 
-return _rStrExpValue.getLength() != 0;
+return !_rStrExpValue.isEmpty();
 }
 
 //=
diff --git a/xmloff/source/forms/elementexport.cxx 
b/xmloff/source/forms/elementexport.cxx
index fcc3efa..863d99a 100644
--- a/xmloff/source/forms/elementexport.cxx
+++ b/xmloff/source/forms/elementexport.cxx
@@ -316,7 +316,7 @@ namespace xmloff
 // the control id
 if (CCA_CONTROL_ID  m_nIncludeCommon)
 {
-OSL_ENSURE(m_sControlId.getLength(), 
OControlExport::exportInnerAttributes: have no control id for the control!);
+OSL_ENSURE(!m_sControlId.isEmpty(), 
OControlExport::exportInnerAttributes: have no control id for the control!);
 m_rContext.getGlobalContext().AddAttributeIdLegacy(
 XML_NAMESPACE_FORM, m_sControlId);
 #if OSL_DEBUG_LEVEL  0
@@ -799,7 +799,7 @@ namespace xmloff
 // the target frame
 if (m_nIncludeCommon  CCA_FOR)
 {
-if (m_sReferringControls.getLength())
+if (!m_sReferringControls.isEmpty())
 {   // there is at least one control referring to the one we're 
handling currently
 AddAttribute(
 
OAttributeMetaData::getCommonControlAttributeNamespace(CCA_FOR),
@@ -1095,7 +1095,7 @@ namespace xmloff
 else
 OSL_FAIL( OControlExport::exportSpecialAttributes: not 
property which can be mapped to 

[Libreoffice-commits] .: xmloff/source

2012-01-16 Thread Stephan Bergmann
 xmloff/source/text/txtparae.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 69d91300547f6ed45a9a499ffae71c2c0f91c441
Author: Stephan Bergmann sberg...@redhat.com
Date:   Mon Jan 16 21:42:09 2012 +0100

WaE signed/unsigned comparison.

diff --git a/xmloff/source/text/txtparae.cxx b/xmloff/source/text/txtparae.cxx
index e32203a..96316d1 100644
--- a/xmloff/source/text/txtparae.cxx
+++ b/xmloff/source/text/txtparae.cxx
@@ -856,7 +856,7 @@ void XMLTextParagraphExport::exportListChange(
 
 if ( nListLevelsToBeClosed  0 
  pListElements 
- pListElements-size() = ( 2 * nListLevelsToBeClosed ) )
+ pListElements-size() = ( 2 * sal::static_int_cast sal_uInt16 
(nListLevelsToBeClosed) ) )
 {
 do {
 for(size_t j = 0; j  2; ++j)
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-01-16 Thread Stephan Bergmann
 xmloff/source/text/txtparae.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 7ca7690f84cfab87ddd25fa1200b11625d1f7ece
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Jan 17 08:47:22 2012 +0100

Better fix for WaE signed/unsigned comparison.

diff --git a/xmloff/source/text/txtparae.cxx b/xmloff/source/text/txtparae.cxx
index 96316d1..6db50ff 100644
--- a/xmloff/source/text/txtparae.cxx
+++ b/xmloff/source/text/txtparae.cxx
@@ -856,7 +856,7 @@ void XMLTextParagraphExport::exportListChange(
 
 if ( nListLevelsToBeClosed  0 
  pListElements 
- pListElements-size() = ( 2 * sal::static_int_cast sal_uInt16 
(nListLevelsToBeClosed) ) )
+ pListElements-size() = sal::static_int_cast sal_uInt32 ( 2 * 
nListLevelsToBeClosed ) )
 {
 do {
 for(size_t j = 0; j  2; ++j)
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2012-01-06 Thread Cédric Bosdonnat
 xmloff/source/style/PageMasterExportPropMapper.cxx |   10 ++
 1 file changed, 2 insertions(+), 8 deletions(-)

New commits:
commit 0100280a5c5b121fab2aa932092a7a887bbb507c
Author: Cédric Bosdonnat cedric.bosdonnat@free.fr
Date:   Fri Jan 6 10:33:19 2012 +0100

fdo#43862: reintroduce fo:margin-* on top of fo:margin for backward compat

diff --git a/xmloff/source/style/PageMasterExportPropMapper.cxx 
b/xmloff/source/style/PageMasterExportPropMapper.cxx
index ba0f45b..eedb4a4 100644
--- a/xmloff/source/style/PageMasterExportPropMapper.cxx
+++ b/xmloff/source/style/PageMasterExportPropMapper.cxx
@@ -151,14 +151,8 @@ void XMLPropertyStateBuffer::ContextFilter( ::std::vector 
XMLPropertyState  )
 pPMMarginLeft-maValue = nLeft;
 pPMMarginRight-maValue = nRight;
 
-if ((nTop == nBottom)  (nBottom == nLeft)  (nLeft == nRight))
-{
-lcl_RemoveState( pPMMarginTop );
-lcl_RemoveState( pPMMarginBottom );
-lcl_RemoveState( pPMMarginLeft );
-lcl_RemoveState( pPMMarginRight );
-}
-else
+// Don't remove the Margin Left,Right,Top,Bottom for backward 
compatibility
+if ((nTop != nBottom) || (nBottom != nLeft) || (nLeft != nRight))
 lcl_RemoveState( pPMMarginAll );
 }
 else
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2011-12-05 Thread Fridrich Strba
 xmloff/source/core/xmlexp.cxx |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit dc0f9dab100ce02a8c2a275f5f53187bb0df0976
Author: Fridrich Å trba fridrich.st...@bluewin.ch
Date:   Mon Dec 5 11:13:44 2011 +0100

The ODG should have office:drawing after office:body, not office:graphics

diff --git a/xmloff/source/core/xmlexp.cxx b/xmloff/source/core/xmlexp.cxx
index 7ebd3fd..a074d03 100644
--- a/xmloff/source/core/xmlexp.cxx
+++ b/xmloff/source/core/xmlexp.cxx
@@ -1248,6 +1248,8 @@ void SvXMLExport::ImplExportContent()
   GetXMLToken( XML_TRUE ) );
 eClass = XML_TEXT;
 }
+if ( XML_GRAPHICS == eClass )
+eClass = XML_DRAWING;
 // office:body ...
 SetBodyAttributes();
 SvXMLElementExport aElem( *this, meClass != XML_TOKEN_INVALID,
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2011-11-24 Thread Eike Rathke
 xmloff/source/forms/elementimport.cxx |5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 49846da1652661e73acffd1ee8df0419aadf5cdb
Author: Eike Rathke er...@redhat.com
Date:   Thu Nov 24 17:40:44 2011 +0100

fdo#43152 ooo#114415 fixed crash on unnamed form element

An unnamed form element without parent container lead to a crash when 
trying to
obtain a unique name by iterating over (now not) existing container 
elements.

diff --git a/xmloff/source/forms/elementimport.cxx 
b/xmloff/source/forms/elementimport.cxx
index 3fde263..10c4b2f 100644
--- a/xmloff/source/forms/elementimport.cxx
+++ b/xmloff/source/forms/elementimport.cxx
@@ -495,8 +495,11 @@ namespace xmloff
 {
 // no optimization here. If this method gets called, the XML stream 
did not contain a name for the
 // element, which is a heavy error. So in this case we don't care for 
performance
-Sequence ::rtl::OUString  aNames = 
m_xParentContainer-getElementNames();
 static const ::rtl::OUString 
sUnnamedName(RTL_CONSTASCII_USTRINGPARAM(unnamed));
+OSL_ENSURE(m_xParentContainer.is(), 
OElementImport::implGetDefaultName: no parent container!);
+if (!m_xParentContainer.is())
+return sUnnamedName;
+Sequence ::rtl::OUString  aNames = 
m_xParentContainer-getElementNames();
 
 ::rtl::OUString sReturn;
 const ::rtl::OUString* pNames = NULL;
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: xmloff/source

2011-09-29 Thread Tor Lillqvist
 xmloff/source/core/xmlexp.cxx |2 +-
 xmloff/source/core/xmlimp.cxx |2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 42fe8b9f11da41530a6d914397b6301a72db3800
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 29 10:48:57 2011 +0300

WaE: Mark SvXMLExport_Impl and SvXMLImport_Impl as SAL_DLLPRIVATE

Fixes warnings: 'SvXML*_Impl' declared with greater visibility than
the type of its field 'SvXMLExport_Impl::mpRDFaHelper'.

diff --git a/xmloff/source/core/xmlexp.cxx b/xmloff/source/core/xmlexp.cxx
index 2777f61..54b02a4 100644
--- a/xmloff/source/core/xmlexp.cxx
+++ b/xmloff/source/core/xmlexp.cxx
@@ -260,7 +260,7 @@ void SAL_CALL SvXMLExportEventListener::disposing( const 
lang::EventObject )
 
 
//==
 
-class SvXMLExport_Impl
+class SAL_DLLPRIVATE SvXMLExport_Impl
 {
 public:
 SvXMLExport_Impl();
diff --git a/xmloff/source/core/xmlimp.cxx b/xmloff/source/core/xmlimp.cxx
index 1f9f87a..9574bd7 100644
--- a/xmloff/source/core/xmlimp.cxx
+++ b/xmloff/source/core/xmlimp.cxx
@@ -169,7 +169,7 @@ void SAL_CALL SvXMLImportEventListener::disposing( const 
lang::EventObject )
 
 
//==
 
-class SvXMLImport_Impl
+class SAL_DLLPRIVATE SvXMLImport_Impl
 {
 public:
 FontToSubsFontConverter hBatsFontConv;
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits