[Libreoffice-commits] core.git: oox/source reportdesign/source sal/rtl sc/source
Andrea Gelmini (via logerrit)
logerrit at kemper.freedesktop.org
Mon Feb 8 20:05:52 UTC 2021
oox/source/drawingml/graphicshapecontext.cxx | 1 -
oox/source/drawingml/linepropertiescontext.cxx | 1 -
oox/source/drawingml/shapecontext.cxx | 1 -
oox/source/drawingml/shapepropertiescontext.cxx | 2 --
oox/source/drawingml/textbodycontext.cxx | 2 --
oox/source/vml/vmltextboxcontext.cxx | 2 --
reportdesign/source/filter/xml/xmlfilter.cxx | 7 -------
sal/rtl/ustring.cxx | 1 -
sc/source/core/data/attrib.cxx | 3 ---
sc/source/core/data/column2.cxx | 3 ---
sc/source/core/data/column3.cxx | 1 -
sc/source/core/data/dpcache.cxx | 3 ---
sc/source/core/data/formulacell.cxx | 1 -
sc/source/core/data/global.cxx | 1 -
sc/source/core/data/mtvelements.cxx | 1 -
15 files changed, 30 deletions(-)
New commits:
commit 0b1b167c3f7fa802532373699589104fe4b391c7
Author: Andrea Gelmini <andrea.gelmini at gelma.net>
AuthorDate: Sun Feb 7 20:59:14 2021 +0100
Commit: Julien Nabet <serval2412 at yahoo.fr>
CommitDate: Mon Feb 8 21:05:05 2021 +0100
Remove unneeded breaks
Extending this:
https://gerrit.libreoffice.org/c/core/+/110512
Change-Id: I1c5bfcddeb0f5619dc848bbf02408cf166bebc8e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/110521
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412 at yahoo.fr>
diff --git a/oox/source/drawingml/graphicshapecontext.cxx b/oox/source/drawingml/graphicshapecontext.cxx
index eaf650c051ba..113e5490c51d 100644
--- a/oox/source/drawingml/graphicshapecontext.cxx
+++ b/oox/source/drawingml/graphicshapecontext.cxx
@@ -228,7 +228,6 @@ ContextHandlerRef OleObjectGraphicDataContext::onCreateContext( sal_Int32 nEleme
break;
case PPT_TOKEN( pic ):
return new GraphicShapeContext( *this, mpMasterShapePtr, mpShapePtr );
- break;
}
SAL_WARN("oox", "OleObjectGraphicDataContext::onCreateContext: unhandled element: "
<< getBaseToken(nElement));
diff --git a/oox/source/drawingml/linepropertiescontext.cxx b/oox/source/drawingml/linepropertiescontext.cxx
index dd9af49bf2db..322f98aad609 100644
--- a/oox/source/drawingml/linepropertiescontext.cxx
+++ b/oox/source/drawingml/linepropertiescontext.cxx
@@ -64,7 +64,6 @@ ContextHandlerRef LinePropertiesContext::onCreateContext( sal_Int32 nElement, co
break;
case A_TOKEN( custDash ): // CT_DashStopList
return this;
- break;
case A_TOKEN( ds ):
{
// 'a:ds' has 2 attributes : 'd' and 'sp'
diff --git a/oox/source/drawingml/shapecontext.cxx b/oox/source/drawingml/shapecontext.cxx
index 8d76755855c9..e0cbf9343c1d 100644
--- a/oox/source/drawingml/shapecontext.cxx
+++ b/oox/source/drawingml/shapecontext.cxx
@@ -110,7 +110,6 @@ ContextHandlerRef ShapeContext::onCreateContext( sal_Int32 aElementToken, const
if (!mpShapePtr->getTextBody())
mpShapePtr->setTextBody( std::make_shared<TextBody>() );
return new TextBodyPropertiesContext( *this, rAttribs, mpShapePtr );
- break;
case XML_txbx:
break;
case XML_cNvPicPr:
diff --git a/oox/source/drawingml/shapepropertiescontext.cxx b/oox/source/drawingml/shapepropertiescontext.cxx
index 574e0793c89a..e0fc3274c731 100644
--- a/oox/source/drawingml/shapepropertiescontext.cxx
+++ b/oox/source/drawingml/shapepropertiescontext.cxx
@@ -95,12 +95,10 @@ ContextHandlerRef ShapePropertiesContext::onCreateContext( sal_Int32 aElementTok
// todo not supported by core, only for preservation via grab bags
case A_TOKEN( scene3d ): // CT_Scene3D
return new Scene3DPropertiesContext( *this, mrShape.get3DProperties() );
- break;
// todo not supported by core, only for preservation via grab bags
case A_TOKEN( sp3d ): // CT_Shape3D
return new Shape3DPropertiesContext( *this, rAttribs, mrShape.get3DProperties() );
- break;
}
return FillPropertiesContext::createFillContext( *this, aElementToken, rAttribs, mrShape.getFillProperties() );
diff --git a/oox/source/drawingml/textbodycontext.cxx b/oox/source/drawingml/textbodycontext.cxx
index f4c33f7253e0..49b50309f597 100644
--- a/oox/source/drawingml/textbodycontext.cxx
+++ b/oox/source/drawingml/textbodycontext.cxx
@@ -99,10 +99,8 @@ ContextHandlerRef TextParagraphContext::onCreateContext( sal_Int32 aElementToken
break;
case W_TOKEN( ins ):
return this;
- break;
case OOX_TOKEN(a14, m):
return CreateLazyMathBufferingContext(*this, mrParagraph);
- break;
default:
SAL_WARN("oox", "TextParagraphContext::onCreateContext: unhandled element: " << getBaseToken(aElementToken));
}
diff --git a/oox/source/vml/vmltextboxcontext.cxx b/oox/source/vml/vmltextboxcontext.cxx
index d878c6dcdf3d..2cf5ebfaf40d 100644
--- a/oox/source/vml/vmltextboxcontext.cxx
+++ b/oox/source/vml/vmltextboxcontext.cxx
@@ -249,11 +249,9 @@ ContextHandlerRef TextBoxContext::onCreateContext( sal_Int32 nElement, const Att
return new TextPortionContext( *this, mrTextBox, maParagraph, TextFontModel(), nElement, rAttribs );
else
return this;
- break;
case W_TOKEN(pPr):
case W_TOKEN(sdt):
return this;
- break;
default:
SAL_INFO("oox", "unhandled 0x" << std::hex << getCurrentElement());
break;
diff --git a/reportdesign/source/filter/xml/xmlfilter.cxx b/reportdesign/source/filter/xml/xmlfilter.cxx
index 2fbbb37992d6..8406723095be 100644
--- a/reportdesign/source/filter/xml/xmlfilter.cxx
+++ b/reportdesign/source/filter/xml/xmlfilter.cxx
@@ -561,22 +561,18 @@ public:
case XML_ELEMENT(OFFICE, XML_FONT_FACE_DECLS):
rImport.GetProgressBarHelper()->Increment( PROGRESS_BAR_STEP );
return rImport.CreateFontDeclsContext();
- break;
case XML_ELEMENT(OFFICE, XML_MASTER_STYLES):
{
SvXMLStylesContext* pStyleContext = new RptMLMasterStylesContext_Impl(rImport);
rImport.SetMasterStyles(pStyleContext);
return pStyleContext;
}
- break;
case XML_ELEMENT(OFFICE, XML_STYLES):
rImport.GetProgressBarHelper()->Increment( PROGRESS_BAR_STEP );
return rImport.CreateStylesContext(false);
- break;
case XML_ELEMENT(OFFICE, XML_AUTOMATIC_STYLES):
// don't use the autostyles from the styles-document for the progress
return rImport.CreateStylesContext(true);
- break;
}
return nullptr;
}
@@ -624,15 +620,12 @@ public:
{
case XML_ELEMENT(OFFICE, XML_BODY):
return new RptXMLDocumentBodyContext(rImport);
- break;
case XML_ELEMENT(OFFICE, XML_FONT_FACE_DECLS):
rImport.GetProgressBarHelper()->Increment( PROGRESS_BAR_STEP );
return rImport.CreateFontDeclsContext();
- break;
case XML_ELEMENT(OFFICE, XML_AUTOMATIC_STYLES):
rImport.GetProgressBarHelper()->Increment( PROGRESS_BAR_STEP );
return rImport.CreateStylesContext(true);
- break;
}
return nullptr;
}
diff --git a/sal/rtl/ustring.cxx b/sal/rtl/ustring.cxx
index 68a1a4449c4d..a5efa62d759c 100644
--- a/sal/rtl/ustring.cxx
+++ b/sal/rtl/ustring.cxx
@@ -946,7 +946,6 @@ static int rtl_canGuessUOutputLength( int len, rtl_TextEncoding eTextEncoding )
case RTL_TEXTENCODING_IBM_863:
case RTL_TEXTENCODING_IBM_865:
return len;
- break;
}
return 0;
}
diff --git a/sc/source/core/data/attrib.cxx b/sc/source/core/data/attrib.cxx
index cb39fec0b2f0..1ae01ef1c26f 100644
--- a/sc/source/core/data/attrib.cxx
+++ b/sc/source/core/data/attrib.cxx
@@ -539,7 +539,6 @@ bool ScViewObjectModeItem::GetPresentation
else
rText += ScResId(STR_VOBJ_MODE_HIDE);
return true;
- break;
default: break;
// added to avoid warnings
@@ -620,12 +619,10 @@ bool ScPageScaleToItem::GetPresentation(
case SfxItemPresentation::Nameless:
rText = aValue;
return true;
- break;
case SfxItemPresentation::Complete:
rText = aName + " (" + aValue + ")";
return true;
- break;
default:
OSL_FAIL( "ScPageScaleToItem::GetPresentation - unknown presentation mode" );
diff --git a/sc/source/core/data/column2.cxx b/sc/source/core/data/column2.cxx
index 22d5a16de895..d194ab134dd1 100644
--- a/sc/source/core/data/column2.cxx
+++ b/sc/source/core/data/column2.cxx
@@ -3523,18 +3523,15 @@ public:
case sc::element_type_numeric:
case sc::element_type_string:
return node.size;
- break;
case sc::element_type_formula:
{
// Each formula cell is worth its code length plus 5.
return std::accumulate(sc::formula_block::begin(*node.data), sc::formula_block::end(*node.data), size_t(0),
[](const size_t& rCount, const ScFormulaCell* p) { return rCount + 5 + p->GetCode()->GetCodeLen(); });
}
- break;
case sc::element_type_edittext:
// each edit-text cell is worth 50.
return node.size * 50;
- break;
default:
return 0;
}
diff --git a/sc/source/core/data/column3.cxx b/sc/source/core/data/column3.cxx
index f1a621a985b5..14d1ebdc09c4 100644
--- a/sc/source/core/data/column3.cxx
+++ b/sc/source/core/data/column3.cxx
@@ -837,7 +837,6 @@ public:
deleteNumeric(node, nOffset, nDataSize);
return;
- break;
case sc::element_type_string:
case sc::element_type_edittext:
if (!mbString)
diff --git a/sc/source/core/data/dpcache.cxx b/sc/source/core/data/dpcache.cxx
index d16ceb957c61..a9fe44f3f4e7 100644
--- a/sc/source/core/data/dpcache.cxx
+++ b/sc/source/core/data/dpcache.cxx
@@ -1175,13 +1175,10 @@ sal_uInt32 ScDPCache::GetLocaleIndependentFormat( SvNumberFormatter& rFormatter,
{
case SvNumFormatType::DATE:
return rFormatter.GetFormatIndex( NF_DATE_ISO_YYYYMMDD, LANGUAGE_ENGLISH_US);
- break;
case SvNumFormatType::TIME:
return rFormatter.GetFormatIndex( NF_TIME_HHMMSS, LANGUAGE_ENGLISH_US);
- break;
case SvNumFormatType::DATETIME:
return rFormatter.GetFormatIndex( NF_DATETIME_ISO_YYYYMMDD_HHMMSS, LANGUAGE_ENGLISH_US);
- break;
default:
return rFormatter.GetFormatIndex( NF_NUMBER_STANDARD, LANGUAGE_ENGLISH_US);
}
diff --git a/sc/source/core/data/formulacell.cxx b/sc/source/core/data/formulacell.cxx
index 707acf1d355c..cad18eb8022a 100644
--- a/sc/source/core/data/formulacell.cxx
+++ b/sc/source/core/data/formulacell.cxx
@@ -3576,7 +3576,6 @@ bool ScFormulaCell::UpdateReference(
{
case URM_INSDEL:
return UpdatePosOnShift(rCxt);
- break;
default:
;
}
diff --git a/sc/source/core/data/global.cxx b/sc/source/core/data/global.cxx
index 3a87e3a95577..458e81da5188 100644
--- a/sc/source/core/data/global.cxx
+++ b/sc/source/core/data/global.cxx
@@ -417,7 +417,6 @@ OUString ScGlobal::GetLongErrorString(FormulaError nErr)
break;
default:
return ScResId(STR_ERROR_STR) + OUString::number( static_cast<int>(nErr) );
- break;
}
return ScResId(pErrNumber);
}
diff --git a/sc/source/core/data/mtvelements.cxx b/sc/source/core/data/mtvelements.cxx
index 56c0fbab0d9a..792b4e9d60e6 100644
--- a/sc/source/core/data/mtvelements.cxx
+++ b/sc/source/core/data/mtvelements.cxx
@@ -168,7 +168,6 @@ ScRefCellValue toRefCell( const sc::CellStoreType::const_iterator& itPos, size_t
case sc::element_type_edittext:
// Edit cell
return ScRefCellValue(sc::edittext_block::at(*itPos->data, nOffset));
- break;
case sc::element_type_formula:
// Formula cell
return ScRefCellValue(sc::formula_block::at(*itPos->data, nOffset));
More information about the Libreoffice-commits
mailing list