[Libreoffice-commits] core.git: include/svx sc/qa sc/source sd/source svx/inc svx/source sw/source
Noel Grandin (via logerrit)
logerrit at kemper.freedesktop.org
Tue Aug 27 16:43:55 UTC 2019
include/svx/svdocirc.hxx | 19 +--
sc/qa/extras/anchor.cxx | 2
sc/qa/unit/ucalc.cxx | 2
sc/qa/unit/ucalc_sort.cxx | 2
sc/source/core/tool/detfunc.cxx | 2
sc/source/filter/excel/xiescher.cxx | 4
sd/source/ui/func/fuconarc.cxx | 12 -
sd/source/ui/func/fuconrec.cxx | 2
svx/inc/sxcikitm.hxx | 8 -
svx/source/dialog/imapwnd.cxx | 4
svx/source/sdr/properties/circleproperties.cxx | 20 ---
svx/source/svdraw/svdfmtf.cxx | 8 -
svx/source/svdraw/svdibrow.cxx | 2
svx/source/svdraw/svdobj.cxx | 10 -
svx/source/svdraw/svdocirc.cxx | 153 ++++++++++++-------------
svx/source/unodraw/unopage.cxx | 4
sw/source/filter/ww8/ww8graf.cxx | 4
17 files changed, 121 insertions(+), 137 deletions(-)
New commits:
commit bbd500e14fce92b27cfc09e7cffd346e36eb5fb0
Author: Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Tue Aug 27 10:55:13 2019 +0200
Commit: Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Tue Aug 27 18:41:57 2019 +0200
use SdrCircKind enum more widely
and make it a scoped enum
Change-Id: I6325ff86ff8566135f6d01a576c659539dcb3ed9
Reviewed-on: https://gerrit.libreoffice.org/78176
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/include/svx/svdocirc.hxx b/include/svx/svdocirc.hxx
index 5785a725584f..b628811bdba3 100644
--- a/include/svx/svdocirc.hxx
+++ b/include/svx/svdocirc.hxx
@@ -36,6 +36,10 @@ public:
long nEndAngle;
};
+enum class SdrCircKind { Full, Section, Cut, Arc };
+
+extern SVX_DLLPUBLIC SdrCircKind ToSdrCircKind(SdrObjKind);
+
// class SdrCircObj
class SVX_DLLPUBLIC SdrCircObj final : public SdrRectObj
@@ -44,17 +48,14 @@ private:
// to allow sdr::properties::CircleProperties access to ImpSetAttrToCircInfo()
friend class sdr::properties::CircleProperties;
- // only for SdrCircleAttributes
- SdrObjKind GetCircleKind() const { return meCircleKind; }
-
virtual std::unique_ptr<sdr::contact::ViewContact> CreateObjectSpecificViewContact() override;
virtual std::unique_ptr<sdr::properties::BaseProperties> CreateObjectSpecificProperties() override;
- SdrObjKind meCircleKind;
+ SdrCircKind meCircleKind;
long nStartAngle;
long nEndAngle;
- SVX_DLLPRIVATE basegfx::B2DPolygon ImpCalcXPolyCirc(const SdrObjKind eKind, const tools::Rectangle& rRect1, long nStart, long nEnd) const;
+ SVX_DLLPRIVATE basegfx::B2DPolygon ImpCalcXPolyCirc(const SdrCircKind eKind, const tools::Rectangle& rRect1, long nStart, long nEnd) const;
SVX_DLLPRIVATE static void ImpSetCreateParams(SdrDragStat& rStat);
SVX_DLLPRIVATE void ImpSetAttrToCircInfo(); // copy values from pool
SVX_DLLPRIVATE void ImpSetCircInfoToAttr(); // copy values into pool
@@ -72,10 +73,10 @@ private:
public:
SdrCircObj(
SdrModel& rSdrModel,
- SdrObjKind eNewKind); // Circ, CArc, Sect or CCut
+ SdrCircKind eNewKind);
SdrCircObj(
SdrModel& rSdrModel,
- SdrObjKind eNewKind,
+ SdrCircKind eNewKind,
const tools::Rectangle& rRect);
// 0=0.00Deg=3h 9000=90.00Deg=12h 18000=180.00Deg=9h 27000=270.00Deg=6h
@@ -84,11 +85,13 @@ public:
// If nNewStartAngle+36000==nNewEndWink, then the arc has angle of 360 degrees.
SdrCircObj(
SdrModel& rSdrModel,
- SdrObjKind eNewKind,
+ SdrCircKind eNewKind,
const tools::Rectangle& rRect,
long nNewStartAngle,
long nNewEndWink);
+ SdrCircKind GetCircleKind() const { return meCircleKind; }
+
virtual void TakeObjInfo(SdrObjTransformInfoRec& rInfo) const override;
virtual sal_uInt16 GetObjIdentifier() const override;
virtual void TakeUnrotatedSnapRect(tools::Rectangle& rRect) const override;
diff --git a/sc/qa/extras/anchor.cxx b/sc/qa/extras/anchor.cxx
index 8d61466b5a29..8ec960489d65 100644
--- a/sc/qa/extras/anchor.cxx
+++ b/sc/qa/extras/anchor.cxx
@@ -171,7 +171,7 @@ void ScAnchorTest::testTdf76183()
// Add a circle somewhere below first row.
const tools::Rectangle aOrigRect(1000, 1000, 1200, 1200);
- SdrCircObj* pObj = new SdrCircObj(*pDrawLayer, OBJ_CIRC, aOrigRect);
+ SdrCircObj* pObj = new SdrCircObj(*pDrawLayer, SdrCircKind::Full, aOrigRect);
pPage->InsertObject(pObj);
// Anchor to cell
ScDrawLayer::SetCellAnchoredFromPosition(*pObj, rDoc, 0, false);
diff --git a/sc/qa/unit/ucalc.cxx b/sc/qa/unit/ucalc.cxx
index c4a6f84cd3e6..7b48202cedca 100644
--- a/sc/qa/unit/ucalc.cxx
+++ b/sc/qa/unit/ucalc.cxx
@@ -2766,7 +2766,7 @@ void Test::testGraphicsInGroup()
{
// Add a circle.
tools::Rectangle aOrigRect(10,10,210,210); // 200 x 200
- SdrCircObj* pObj = new SdrCircObj(*pDrawLayer, OBJ_CIRC, aOrigRect);
+ SdrCircObj* pObj = new SdrCircObj(*pDrawLayer, SdrCircKind::Full, aOrigRect);
pPage->InsertObject(pObj);
const tools::Rectangle& rNewRect = pObj->GetLogicRect();
CPPUNIT_ASSERT_EQUAL_MESSAGE("Position and size of the circle shouldn't change when inserted into the page.",
diff --git a/sc/qa/unit/ucalc_sort.cxx b/sc/qa/unit/ucalc_sort.cxx
index 8fd4627d86ce..fe31e2a803a7 100644
--- a/sc/qa/unit/ucalc_sort.cxx
+++ b/sc/qa/unit/ucalc_sort.cxx
@@ -1914,7 +1914,7 @@ void Test::testSortImages()
// Insert graphic in cell B2.
const tools::Rectangle aOrigRect(1000, 1000, 1200, 1200);
- SdrCircObj* pObj = new SdrCircObj(*pDrawLayer, OBJ_CIRC, aOrigRect);
+ SdrCircObj* pObj = new SdrCircObj(*pDrawLayer, SdrCircKind::Full, aOrigRect);
SdrPage* pPage = pDrawLayer->GetPage(0);
CPPUNIT_ASSERT(pPage);
pPage->InsertObject(pObj);
diff --git a/sc/source/core/tool/detfunc.cxx b/sc/source/core/tool/detfunc.cxx
index 48a6c432db1e..76a42f661005 100644
--- a/sc/source/core/tool/detfunc.cxx
+++ b/sc/source/core/tool/detfunc.cxx
@@ -638,7 +638,7 @@ void ScDetectiveFunc::DrawCircle( SCCOL nCol, SCROW nRow, ScDetectiveData& rData
SdrCircObj* pCircle = new SdrCircObj(
*pModel,
- OBJ_CIRC,
+ SdrCircKind::Full,
aRect);
SfxItemSet& rAttrSet = rData.GetCircleSet();
diff --git a/sc/source/filter/excel/xiescher.cxx b/sc/source/filter/excel/xiescher.cxx
index 610c6983cc4a..0866ca17a20d 100644
--- a/sc/source/filter/excel/xiescher.cxx
+++ b/sc/source/filter/excel/xiescher.cxx
@@ -1247,7 +1247,7 @@ SdrObjectUniquePtr XclImpOvalObj::DoCreateSdrObj( XclImpDffConverter& rDffConv,
SdrObjectUniquePtr xSdrObj(
new SdrCircObj(
*GetDoc().GetDrawLayer(),
- OBJ_CIRC,
+ SdrCircKind::Full,
rAnchorRect));
ConvertRectStyle( *xSdrObj );
rDffConv.Progress();
@@ -1319,7 +1319,7 @@ SdrObjectUniquePtr XclImpArcObj::DoCreateSdrObj( XclImpDffConverter& rDffConv, c
aNewRect.AdjustTop( -(rAnchorRect.GetHeight()) );
break;
}
- SdrObjKind eObjKind = maFillData.IsFilled() ? OBJ_SECT : OBJ_CARC;
+ SdrCircKind eObjKind = maFillData.IsFilled() ? SdrCircKind::Section : SdrCircKind::Arc;
SdrObjectUniquePtr xSdrObj(
new SdrCircObj(
*GetDoc().GetDrawLayer(),
diff --git a/sd/source/ui/func/fuconarc.cxx b/sd/source/ui/func/fuconarc.cxx
index f10e2315f774..8a968d351051 100644
--- a/sd/source/ui/func/fuconarc.cxx
+++ b/sd/source/ui/func/fuconarc.cxx
@@ -90,12 +90,12 @@ void FuConstructArc::DoExecute( SfxRequest& rReq )
Activate(); // sets aObjKind
SdrCircObj* pNewCircle =
- new SdrCircObj(
- mpView->getSdrModelFromSdrView(),
- static_cast<SdrObjKind>(mpView->GetCurrentObjIdentifier()),
- aNewRectangle,
- static_cast<long>(pPhiStart->GetValue () * 10.0),
- static_cast<long>(pPhiEnd->GetValue () * 10.0));
+ new SdrCircObj(
+ mpView->getSdrModelFromSdrView(),
+ ToSdrCircKind(static_cast<SdrObjKind>(mpView->GetCurrentObjIdentifier())),
+ aNewRectangle,
+ static_cast<long>(pPhiStart->GetValue () * 10.0),
+ static_cast<long>(pPhiEnd->GetValue () * 10.0));
SdrPageView *pPV = mpView->GetSdrPageView();
mpView->InsertObjectAtView(pNewCircle, *pPV, SdrInsertFlags::SETDEFLAYER);
diff --git a/sd/source/ui/func/fuconrec.cxx b/sd/source/ui/func/fuconrec.cxx
index 5d937e118279..05035cd03252 100644
--- a/sd/source/ui/func/fuconrec.cxx
+++ b/sd/source/ui/func/fuconrec.cxx
@@ -141,7 +141,7 @@ void FuConstructRectangle::DoExecute( SfxRequest& rReq )
pCenterY->GetValue () + pAxisY->GetValue () / 2);
SdrCircObj *pNewCircle = new SdrCircObj(
mpView->getSdrModelFromSdrView(),
- OBJ_CIRC,
+ SdrCircKind::Full,
aNewRectangle);
SdrPageView *pPV = mpView->GetSdrPageView();
diff --git a/svx/inc/sxcikitm.hxx b/svx/inc/sxcikitm.hxx
index 03740ab7758c..a92fbc1c8856 100644
--- a/svx/inc/sxcikitm.hxx
+++ b/svx/inc/sxcikitm.hxx
@@ -21,15 +21,11 @@
#include <svx/svddef.hxx>
#include <svl/eitem.hxx>
-
-enum SdrCircKind {SDRCIRC_FULL,
- SDRCIRC_SECT,
- SDRCIRC_CUT,
- SDRCIRC_ARC};
+#include <svx/svdocirc.hxx>
class SdrCircKindItem: public SfxEnumItem<SdrCircKind> {
public:
- SdrCircKindItem(SdrCircKind eKind=SDRCIRC_FULL): SfxEnumItem(SDRATTR_CIRCKIND, eKind) {}
+ SdrCircKindItem(SdrCircKind eKind=SdrCircKind::Full): SfxEnumItem(SDRATTR_CIRCKIND, eKind) {}
virtual SfxPoolItem* Clone(SfxItemPool* pPool=nullptr) const override;
virtual sal_uInt16 GetValueCount() const override; // { return 4; }
diff --git a/svx/source/dialog/imapwnd.cxx b/svx/source/dialog/imapwnd.cxx
index d8a3fd24bd06..3cc57494dc0a 100644
--- a/svx/source/dialog/imapwnd.cxx
+++ b/svx/source/dialog/imapwnd.cxx
@@ -215,7 +215,7 @@ SdrObject* IMapWindow::CreateObj( const IMapObject* pIMapObj )
pSdrObj = static_cast<SdrObject*>(
new SdrCircObj(
*pModel,
- OBJ_CIRC,
+ SdrCircKind::Full,
aCircle,
0,
36000));
@@ -238,7 +238,7 @@ SdrObject* IMapWindow::CreateObj( const IMapObject* pIMapObj )
pSdrObj = static_cast<SdrObject*>(
new SdrCircObj(
*pModel,
- OBJ_CIRC,
+ SdrCircKind::Full,
aDrawRect,
0,
36000));
diff --git a/svx/source/sdr/properties/circleproperties.cxx b/svx/source/sdr/properties/circleproperties.cxx
index 1a89ac8a014d..0ce76a2371f9 100644
--- a/svx/source/sdr/properties/circleproperties.cxx
+++ b/svx/source/sdr/properties/circleproperties.cxx
@@ -94,28 +94,14 @@ namespace sdr
void CircleProperties::ForceDefaultAttributes()
{
SdrCircObj& rObj = static_cast<SdrCircObj&>(GetSdrObject());
- SdrCircKind eKindA = SDRCIRC_FULL;
- SdrObjKind eKind = rObj.GetCircleKind();
+ SdrCircKind eKind = rObj.GetCircleKind();
- if(eKind == OBJ_SECT)
- {
- eKindA = SDRCIRC_SECT;
- }
- else if(eKind == OBJ_CARC)
- {
- eKindA = SDRCIRC_ARC;
- }
- else if(eKind == OBJ_CCUT)
- {
- eKindA = SDRCIRC_CUT;
- }
-
- if(eKindA != SDRCIRC_FULL)
+ if(eKind != SdrCircKind::Full)
{
// force ItemSet
GetObjectItemSet();
- mpItemSet->Put(SdrCircKindItem(eKindA));
+ mpItemSet->Put(SdrCircKindItem(eKind));
if(rObj.GetStartAngle())
{
diff --git a/svx/source/svdraw/svdfmtf.cxx b/svx/source/svdraw/svdfmtf.cxx
index 1534e661d756..e460361efe8d 100644
--- a/svx/source/svdraw/svdfmtf.cxx
+++ b/svx/source/svdraw/svdfmtf.cxx
@@ -739,7 +739,7 @@ void ImpSdrGDIMetaFileImport::DoAction(MetaEllipseAction const & rAct)
{
SdrCircObj* pCirc=new SdrCircObj(
*mpModel,
- OBJ_CIRC,
+ SdrCircKind::Full,
rAct.GetRect());
SetAttributes(pCirc);
InsertObj(pCirc);
@@ -752,7 +752,7 @@ void ImpSdrGDIMetaFileImport::DoAction(MetaArcAction const & rAct)
long nEnd=GetAngle(rAct.GetEndPoint()-aCenter);
SdrCircObj* pCirc = new SdrCircObj(
*mpModel,
- OBJ_CARC,
+ SdrCircKind::Arc,
rAct.GetRect(),nStart,nEnd);
SetAttributes(pCirc);
InsertObj(pCirc);
@@ -765,7 +765,7 @@ void ImpSdrGDIMetaFileImport::DoAction(MetaPieAction const & rAct)
long nEnd=GetAngle(rAct.GetEndPoint()-aCenter);
SdrCircObj* pCirc = new SdrCircObj(
*mpModel,
- OBJ_SECT,
+ SdrCircKind::Section,
rAct.GetRect(),
nStart,
nEnd);
@@ -780,7 +780,7 @@ void ImpSdrGDIMetaFileImport::DoAction(MetaChordAction const & rAct)
long nEnd=GetAngle(rAct.GetEndPoint()-aCenter);
SdrCircObj* pCirc = new SdrCircObj(
*mpModel,
- OBJ_CCUT,
+ SdrCircKind::Cut,
rAct.GetRect(),
nStart,
nEnd);
diff --git a/svx/source/svdraw/svdibrow.cxx b/svx/source/svdraw/svdibrow.cxx
index 300ebe3677ce..431b36f4efc7 100644
--- a/svx/source/svdraw/svdibrow.cxx
+++ b/svx/source/svdraw/svdibrow.cxx
@@ -827,7 +827,7 @@ static bool IsItemIneffective(sal_uInt16 nWhich, const SfxItemSet* pSet, sal_uIn
rIndent=1;
if (ImpGetItem(*pSet,SDRATTR_CIRCKIND,pItem)) {
SdrCircKind eKind=static_cast<const SdrCircKindItem*>(pItem)->GetValue();
- if (eKind==SDRCIRC_FULL) return true;
+ if (eKind==SdrCircKind::Full) return true;
}
} break;
} // switch
diff --git a/svx/source/svdraw/svdobj.cxx b/svx/source/svdraw/svdobj.cxx
index 2ec316e2b810..9d27cc4c3387 100644
--- a/svx/source/svdraw/svdobj.cxx
+++ b/svx/source/svdraw/svdobj.cxx
@@ -3113,19 +3113,15 @@ SdrObject* SdrObjFactory::MakeNewObject(
case OBJ_CARC:
case OBJ_CCUT:
{
+ SdrCircKind eCircKind = ToSdrCircKind(static_cast<SdrObjKind>(nIdentifier));
if(nullptr != pSnapRect)
{
- pObj = new SdrCircObj(
- rSdrModel,
- static_cast<SdrObjKind>(nIdentifier),
- *pSnapRect);
+ pObj = new SdrCircObj(rSdrModel, eCircKind, *pSnapRect);
bSetSnapRect = false;
}
else
{
- pObj = new SdrCircObj(
- rSdrModel,
- static_cast<SdrObjKind>(nIdentifier));
+ pObj = new SdrCircObj(rSdrModel, eCircKind);
}
}
break;
diff --git a/svx/source/svdraw/svdocirc.cxx b/svx/source/svdraw/svdocirc.cxx
index 3e3de148051f..bc726a64b628 100644
--- a/svx/source/svdraw/svdocirc.cxx
+++ b/svx/source/svdraw/svdocirc.cxx
@@ -107,32 +107,45 @@ std::unique_ptr<sdr::contact::ViewContact> SdrCircObj::CreateObjectSpecificViewC
return std::make_unique<sdr::contact::ViewContactOfSdrCircObj>(*this);
}
+SdrCircKind ToSdrCircKind(SdrObjKind eKind)
+{
+ switch (eKind)
+ {
+ case OBJ_CIRC: return SdrCircKind::Full;
+ case OBJ_SECT: return SdrCircKind::Section;
+ case OBJ_CARC: return SdrCircKind::Arc;
+ case OBJ_CCUT: return SdrCircKind::Cut;
+ default: assert(false);
+ }
+ return SdrCircKind::Full;
+}
+
SdrCircObj::SdrCircObj(
SdrModel& rSdrModel,
- SdrObjKind eNewKind)
+ SdrCircKind eNewKind)
: SdrRectObj(rSdrModel)
{
nStartAngle=0;
nEndAngle=36000;
meCircleKind=eNewKind;
- bClosedObj=eNewKind!=OBJ_CARC;
+ bClosedObj=eNewKind!=SdrCircKind::Arc;
}
SdrCircObj::SdrCircObj(
SdrModel& rSdrModel,
- SdrObjKind eNewKind,
+ SdrCircKind eNewKind,
const tools::Rectangle& rRect)
: SdrRectObj(rSdrModel, rRect)
{
nStartAngle=0;
nEndAngle=36000;
meCircleKind=eNewKind;
- bClosedObj=eNewKind!=OBJ_CARC;
+ bClosedObj=eNewKind!=SdrCircKind::Arc;
}
SdrCircObj::SdrCircObj(
SdrModel& rSdrModel,
- SdrObjKind eNewKind,
+ SdrCircKind eNewKind,
const tools::Rectangle& rRect,
long nNewStartWink,
long nNewEndWink)
@@ -143,7 +156,7 @@ SdrCircObj::SdrCircObj(
nEndAngle=NormAngle36000(nNewEndWink);
if (nAngleDif==36000) nEndAngle+=nAngleDif; // full circle
meCircleKind=eNewKind;
- bClosedObj=eNewKind!=OBJ_CARC;
+ bClosedObj=eNewKind!=SdrCircKind::Arc;
}
SdrCircObj::~SdrCircObj()
@@ -161,7 +174,15 @@ void SdrCircObj::TakeObjInfo(SdrObjTransformInfoRec& rInfo) const
sal_uInt16 SdrCircObj::GetObjIdentifier() const
{
- return sal_uInt16(meCircleKind);
+ switch (meCircleKind)
+ {
+ case SdrCircKind::Full: return OBJ_CIRC;
+ case SdrCircKind::Section: return OBJ_SECT;
+ case SdrCircKind::Cut: return OBJ_CCUT;
+ case SdrCircKind::Arc: return OBJ_CARC;
+ default: assert(false);
+ }
+ return OBJ_CIRC;
}
bool SdrCircObj::PaintNeedsXPolyCirc() const
@@ -170,9 +191,9 @@ bool SdrCircObj::PaintNeedsXPolyCirc() const
// ellipse segments.
// If not WIN, then (for now) also for circle/ellipse segments and circle/
// ellipse arcs (for precision)
- bool bNeed=aGeo.nRotationAngle!=0 || aGeo.nShearAngle!=0 || meCircleKind==OBJ_CCUT;
+ bool bNeed=aGeo.nRotationAngle!=0 || aGeo.nShearAngle!=0 || meCircleKind==SdrCircKind::Cut;
// If not WIN, then for everything except full circle (for now!)
- if (meCircleKind!=OBJ_CIRC) bNeed = true;
+ if (meCircleKind!=SdrCircKind::Full) bNeed = true;
const SfxItemSet& rSet = GetObjectItemSet();
if(!bNeed)
@@ -186,7 +207,7 @@ bool SdrCircObj::PaintNeedsXPolyCirc() const
bNeed = rSet.Get(XATTR_LINEWIDTH).GetValue() != 0;
// XPoly is necessary for circle arcs with line ends
- if(!bNeed && meCircleKind == OBJ_CARC)
+ if(!bNeed && meCircleKind == SdrCircKind::Arc)
{
// start of the line is here if StartPolygon, StartWidth!=0
bNeed=rSet.Get(XATTR_LINESTART).GetLineStartValue().count() != 0 &&
@@ -202,24 +223,24 @@ bool SdrCircObj::PaintNeedsXPolyCirc() const
}
// XPoly is necessary if Fill !=None and !=Solid
- if(!bNeed && meCircleKind != OBJ_CARC)
+ if(!bNeed && meCircleKind != SdrCircKind::Arc)
{
drawing::FillStyle eFill=rSet.Get(XATTR_FILLSTYLE).GetValue();
bNeed = eFill != drawing::FillStyle_NONE && eFill != drawing::FillStyle_SOLID;
}
- if(!bNeed && meCircleKind != OBJ_CIRC && nStartAngle == nEndAngle)
+ if(!bNeed && meCircleKind != SdrCircKind::Full && nStartAngle == nEndAngle)
bNeed = true; // otherwise we're drawing a full circle
return bNeed;
}
-basegfx::B2DPolygon SdrCircObj::ImpCalcXPolyCirc(const SdrObjKind eCicrleKind, const tools::Rectangle& rRect1, long nStart, long nEnd) const
+basegfx::B2DPolygon SdrCircObj::ImpCalcXPolyCirc(const SdrCircKind eCicrleKind, const tools::Rectangle& rRect1, long nStart, long nEnd) const
{
const basegfx::B2DRange aRange = vcl::unotools::b2DRectangleFromRectangle(rRect1);
basegfx::B2DPolygon aCircPolygon;
- if(OBJ_CIRC == eCicrleKind)
+ if(SdrCircKind::Full == eCicrleKind)
{
// create full circle. Do not use createPolygonFromEllipse; it's necessary
// to get the start point to the bottom of the circle to keep compatible to
@@ -247,8 +268,8 @@ basegfx::B2DPolygon SdrCircObj::ImpCalcXPolyCirc(const SdrObjKind eCicrleKind, c
fStart, fEnd);
// check closing states
- const bool bCloseSegment(OBJ_CARC != eCicrleKind);
- const bool bCloseUsingCenter(OBJ_SECT == eCicrleKind);
+ const bool bCloseSegment(SdrCircKind::Arc != eCicrleKind);
+ const bool bCloseUsingCenter(SdrCircKind::Section == eCicrleKind);
if(bCloseSegment)
{
@@ -299,18 +320,18 @@ OUString SdrCircObj::TakeObjNameSingul() const
if (maRect.GetWidth() == maRect.GetHeight() && aGeo.nShearAngle==0)
{
switch (meCircleKind) {
- case OBJ_CIRC: pID=STR_ObjNameSingulCIRC; break;
- case OBJ_SECT: pID=STR_ObjNameSingulSECT; break;
- case OBJ_CARC: pID=STR_ObjNameSingulCARC; break;
- case OBJ_CCUT: pID=STR_ObjNameSingulCCUT; break;
+ case SdrCircKind::Full: pID=STR_ObjNameSingulCIRC; break;
+ case SdrCircKind::Section: pID=STR_ObjNameSingulSECT; break;
+ case SdrCircKind::Arc: pID=STR_ObjNameSingulCARC; break;
+ case SdrCircKind::Cut: pID=STR_ObjNameSingulCCUT; break;
default: break;
}
} else {
switch (meCircleKind) {
- case OBJ_CIRC: pID=STR_ObjNameSingulCIRCE; break;
- case OBJ_SECT: pID=STR_ObjNameSingulSECTE; break;
- case OBJ_CARC: pID=STR_ObjNameSingulCARCE; break;
- case OBJ_CCUT: pID=STR_ObjNameSingulCCUTE; break;
+ case SdrCircKind::Full: pID=STR_ObjNameSingulCIRCE; break;
+ case SdrCircKind::Section: pID=STR_ObjNameSingulSECTE; break;
+ case SdrCircKind::Arc: pID=STR_ObjNameSingulCARCE; break;
+ case SdrCircKind::Cut: pID=STR_ObjNameSingulCCUTE; break;
default: break;
}
}
@@ -333,18 +354,18 @@ OUString SdrCircObj::TakeObjNamePlural() const
if (maRect.GetWidth() == maRect.GetHeight() && aGeo.nShearAngle==0)
{
switch (meCircleKind) {
- case OBJ_CIRC: pID=STR_ObjNamePluralCIRC; break;
- case OBJ_SECT: pID=STR_ObjNamePluralSECT; break;
- case OBJ_CARC: pID=STR_ObjNamePluralCARC; break;
- case OBJ_CCUT: pID=STR_ObjNamePluralCCUT; break;
+ case SdrCircKind::Full: pID=STR_ObjNamePluralCIRC; break;
+ case SdrCircKind::Section: pID=STR_ObjNamePluralSECT; break;
+ case SdrCircKind::Arc: pID=STR_ObjNamePluralCARC; break;
+ case SdrCircKind::Cut: pID=STR_ObjNamePluralCCUT; break;
default: break;
}
} else {
switch (meCircleKind) {
- case OBJ_CIRC: pID=STR_ObjNamePluralCIRCE; break;
- case OBJ_SECT: pID=STR_ObjNamePluralSECTE; break;
- case OBJ_CARC: pID=STR_ObjNamePluralCARCE; break;
- case OBJ_CCUT: pID=STR_ObjNamePluralCCUTE; break;
+ case SdrCircKind::Full: pID=STR_ObjNamePluralCIRCE; break;
+ case SdrCircKind::Section: pID=STR_ObjNamePluralSECTE; break;
+ case SdrCircKind::Arc: pID=STR_ObjNamePluralCARCE; break;
+ case SdrCircKind::Cut: pID=STR_ObjNamePluralCCUTE; break;
default: break;
}
}
@@ -397,7 +418,7 @@ public:
sal_uInt32 SdrCircObj::GetHdlCount() const
{
- if(OBJ_CIRC != meCircleKind)
+ if(SdrCircKind::Full != meCircleKind)
{
return 10L;
}
@@ -409,7 +430,7 @@ sal_uInt32 SdrCircObj::GetHdlCount() const
void SdrCircObj::AddToHdlList(SdrHdlList& rHdlList) const
{
- for (sal_uInt32 nHdlNum=(OBJ_CIRC==meCircleKind)?2:0; nHdlNum<=9; ++nHdlNum)
+ for (sal_uInt32 nHdlNum=(SdrCircKind::Full==meCircleKind)?2:0; nHdlNum<=9; ++nHdlNum)
{
Point aPnt;
SdrHdlKind eLocalKind(SdrHdlKind::Move);
@@ -578,7 +599,7 @@ OUString SdrCircObj::getSpecialDragComment(const SdrDragStat& rDrag) const
OUStringBuffer aBuf(aStr);
const sal_uInt32 nPointCount(rDrag.GetPointCount());
- if(OBJ_CIRC != meCircleKind && nPointCount > 2)
+ if(SdrCircKind::Full != meCircleKind && nPointCount > 2)
{
const ImpCircUser* pU = static_cast<const ImpCircUser*>(rDrag.GetUser());
sal_Int32 nAngle;
@@ -725,8 +746,8 @@ bool SdrCircObj::EndCreate(SdrDragStat& rStat, SdrCreateCmd eCmd)
ImpSetCreateParams(rStat);
ImpCircUser* pU=static_cast<ImpCircUser*>(rStat.GetUser());
bool bRet = false;
- if (eCmd==SdrCreateCmd::ForceEnd && rStat.GetPointCount()<4) meCircleKind=OBJ_CIRC;
- if (meCircleKind==OBJ_CIRC) {
+ if (eCmd==SdrCreateCmd::ForceEnd && rStat.GetPointCount()<4) meCircleKind=SdrCircKind::Full;
+ if (meCircleKind==SdrCircKind::Full) {
bRet=rStat.GetPointCount()>=2;
if (bRet) {
maRect = pU->aR;
@@ -743,7 +764,7 @@ bool SdrCircObj::EndCreate(SdrDragStat& rStat, SdrCreateCmd eCmd)
nEndAngle=pU->nEnd;
}
}
- bClosedObj=meCircleKind!=OBJ_CARC;
+ bClosedObj=meCircleKind!=SdrCircKind::Arc;
SetRectsDirty();
SetXPolyDirty();
ImpSetCircInfoToAttr();
@@ -761,7 +782,7 @@ bool SdrCircObj::BckCreate(SdrDragStat& rStat)
{
rStat.SetNoSnap(rStat.GetPointCount()>=3);
rStat.SetOrtho4Possible(rStat.GetPointCount()<3);
- return meCircleKind!=OBJ_CIRC;
+ return meCircleKind!=SdrCircKind::Full;
}
basegfx::B2DPolyPolygon SdrCircObj::TakeCreatePoly(const SdrDragStat& rDrag) const
@@ -771,7 +792,7 @@ basegfx::B2DPolyPolygon SdrCircObj::TakeCreatePoly(const SdrDragStat& rDrag) con
if(rDrag.GetPointCount() < 4)
{
// force to OBJ_CIRC to get full visualisation
- basegfx::B2DPolyPolygon aRetval(ImpCalcXPolyCirc(OBJ_CIRC, pU->aR, pU->nStart, pU->nEnd));
+ basegfx::B2DPolyPolygon aRetval(ImpCalcXPolyCirc(SdrCircKind::Full, pU->aR, pU->nStart, pU->nEnd));
if(3 == rDrag.GetPointCount())
{
@@ -794,10 +815,10 @@ basegfx::B2DPolyPolygon SdrCircObj::TakeCreatePoly(const SdrDragStat& rDrag) con
PointerStyle SdrCircObj::GetCreatePointer() const
{
switch (meCircleKind) {
- case OBJ_CIRC: return PointerStyle::DrawEllipse;
- case OBJ_SECT: return PointerStyle::DrawPie;
- case OBJ_CARC: return PointerStyle::DrawArc;
- case OBJ_CCUT: return PointerStyle::DrawCircleCut;
+ case SdrCircKind::Full: return PointerStyle::DrawEllipse;
+ case SdrCircKind::Section: return PointerStyle::DrawPie;
+ case SdrCircKind::Arc: return PointerStyle::DrawArc;
+ case SdrCircKind::Cut: return PointerStyle::DrawCircleCut;
default: break;
} // switch
return PointerStyle::Cross;
@@ -818,7 +839,7 @@ void SdrCircObj::NbcResize(const Point& rRef, const Fraction& xFact, const Fract
bool bNoShearRota=(aGeo.nRotationAngle==0 && aGeo.nShearAngle==0);
SdrTextObj::NbcResize(rRef,xFact,yFact);
bNoShearRota|=(aGeo.nRotationAngle==0 && aGeo.nShearAngle==0);
- if (meCircleKind!=OBJ_CIRC) {
+ if (meCircleKind!=SdrCircKind::Full) {
bool bXMirr=(xFact.GetNumerator()<0) != (xFact.GetDenominator()<0);
bool bYMirr=(yFact.GetNumerator()<0) != (yFact.GetDenominator()<0);
if (bXMirr || bYMirr) {
@@ -872,7 +893,7 @@ void SdrCircObj::NbcShear(const Point& rRef, long nAngle, double tn, bool bVShea
void SdrCircObj::NbcMirror(const Point& rRef1, const Point& rRef2)
{
- bool bFreeMirr=meCircleKind!=OBJ_CIRC;
+ bool bFreeMirr=meCircleKind!=SdrCircKind::Full;
Point aTmpPt1;
Point aTmpPt2;
if (bFreeMirr) { // some preparations for using an arbitrary axis of reflection
@@ -903,7 +924,7 @@ void SdrCircObj::NbcMirror(const Point& rRef1, const Point& rRef2)
}
}
SdrTextObj::NbcMirror(rRef1,rRef2);
- if (meCircleKind!=OBJ_CIRC) { // adapt starting and finishing angle
+ if (meCircleKind!=SdrCircKind::Full) { // adapt starting and finishing angle
MirrorPoint(aTmpPt1,rRef1,rRef2);
MirrorPoint(aTmpPt2,rRef1,rRef2);
// unrotate:
@@ -965,7 +986,7 @@ static void Union(tools::Rectangle& rR, const Point& rP)
void SdrCircObj::TakeUnrotatedSnapRect(tools::Rectangle& rRect) const
{
rRect = maRect;
- if (meCircleKind!=OBJ_CIRC) {
+ if (meCircleKind!=SdrCircKind::Full) {
const Point aPntStart(GetAnglePnt(maRect,nStartAngle));
const Point aPntEnd(GetAnglePnt(maRect,nEndAngle));
long a=nStartAngle;
@@ -988,7 +1009,7 @@ void SdrCircObj::TakeUnrotatedSnapRect(tools::Rectangle& rRect) const
if ((a<=9000 && e>=9000) || (a>e && (a<=9000 || e>=9000))) {
Union(rRect,maRect.TopCenter());
}
- if (meCircleKind==OBJ_SECT) {
+ if (meCircleKind==SdrCircKind::Section) {
Union(rRect,maRect.Center());
}
if (aGeo.nRotationAngle!=0) {
@@ -1026,7 +1047,7 @@ void SdrCircObj::RecalcSnapRect()
void SdrCircObj::NbcSetSnapRect(const tools::Rectangle& rRect)
{
- if (aGeo.nRotationAngle!=0 || aGeo.nShearAngle!=0 || meCircleKind!=OBJ_CIRC) {
+ if (aGeo.nRotationAngle!=0 || aGeo.nShearAngle!=0 || meCircleKind!=SdrCircKind::Full) {
tools::Rectangle aSR0(GetSnapRect());
long nWdt0=aSR0.Right()-aSR0.Left();
long nHgt0=aSR0.Bottom()-aSR0.Top();
@@ -1045,7 +1066,7 @@ void SdrCircObj::NbcSetSnapRect(const tools::Rectangle& rRect)
sal_uInt32 SdrCircObj::GetSnapPointCount() const
{
- if (meCircleKind==OBJ_CIRC) {
+ if (meCircleKind==SdrCircKind::Full) {
return 1;
} else {
return 3;
@@ -1072,17 +1093,7 @@ void SdrCircObj::Notify(SfxBroadcaster& rBC, const SfxHint& rHint)
void SdrCircObj::ImpSetAttrToCircInfo()
{
const SfxItemSet& rSet = GetObjectItemSet();
- SdrCircKind eNewKindA = rSet.Get(SDRATTR_CIRCKIND).GetValue();
- SdrObjKind eNewKind = meCircleKind;
-
- if(eNewKindA == SDRCIRC_FULL)
- eNewKind = OBJ_CIRC;
- else if(eNewKindA == SDRCIRC_SECT)
- eNewKind = OBJ_SECT;
- else if(eNewKindA == SDRCIRC_ARC)
- eNewKind = OBJ_CARC;
- else if(eNewKindA == SDRCIRC_CUT)
- eNewKind = OBJ_CCUT;
+ SdrCircKind eNewKind = rSet.Get(SDRATTR_CIRCKIND).GetValue();
sal_Int32 nNewStart = rSet.Get(SDRATTR_CIRCSTARTANGLE).GetValue();
sal_Int32 nNewEnd = rSet.Get(SDRATTR_CIRCENDANGLE).GetValue();
@@ -1096,7 +1107,7 @@ void SdrCircObj::ImpSetAttrToCircInfo()
nStartAngle = nNewStart;
nEndAngle = nNewEnd;
- if(bKindChg || (meCircleKind != OBJ_CIRC && bAngleChg))
+ if(bKindChg || (meCircleKind != SdrCircKind::Full && bAngleChg))
{
SetXPolyDirty();
SetRectsDirty();
@@ -1106,27 +1117,19 @@ void SdrCircObj::ImpSetAttrToCircInfo()
void SdrCircObj::ImpSetCircInfoToAttr()
{
- SdrCircKind eNewKindA = SDRCIRC_FULL;
const SfxItemSet& rSet = GetObjectItemSet();
- if(meCircleKind == OBJ_SECT)
- eNewKindA = SDRCIRC_SECT;
- else if(meCircleKind == OBJ_CARC)
- eNewKindA = SDRCIRC_ARC;
- else if(meCircleKind == OBJ_CCUT)
- eNewKindA = SDRCIRC_CUT;
-
SdrCircKind eOldKindA = rSet.Get(SDRATTR_CIRCKIND).GetValue();
sal_Int32 nOldStartAngle = rSet.Get(SDRATTR_CIRCSTARTANGLE).GetValue();
sal_Int32 nOldEndAngle = rSet.Get(SDRATTR_CIRCENDANGLE).GetValue();
- if(eNewKindA != eOldKindA || nStartAngle != nOldStartAngle || nEndAngle != nOldEndAngle)
+ if(meCircleKind != eOldKindA || nStartAngle != nOldStartAngle || nEndAngle != nOldEndAngle)
{
// since SetItem() implicitly calls ImpSetAttrToCircInfo()
// setting the item directly is necessary here.
- if(eNewKindA != eOldKindA)
+ if(meCircleKind != eOldKindA)
{
- GetProperties().SetObjectItemDirect(SdrCircKindItem(eNewKindA));
+ GetProperties().SetObjectItemDirect(SdrCircKindItem(meCircleKind));
}
if(nStartAngle != nOldStartAngle)
@@ -1146,7 +1149,7 @@ void SdrCircObj::ImpSetCircInfoToAttr()
SdrObject* SdrCircObj::DoConvertToPolyObj(bool bBezier, bool bAddText) const
{
- const bool bFill(meCircleKind != OBJ_CARC);
+ const bool bFill(meCircleKind != SdrCircKind::Arc);
const basegfx::B2DPolygon aCircPolygon(ImpCalcXPolyCirc(meCircleKind, maRect, nStartAngle, nEndAngle));
SdrObject* pRet = ImpConvertMakeObj(basegfx::B2DPolyPolygon(aCircPolygon), bFill, bBezier);
diff --git a/svx/source/unodraw/unopage.cxx b/svx/source/unodraw/unopage.cxx
index 44bb5c142d51..fb068fc36d8c 100644
--- a/svx/source/unodraw/unopage.cxx
+++ b/svx/source/unodraw/unopage.cxx
@@ -785,7 +785,7 @@ SvxShape* SvxDrawPage::CreateShapeByTypeAndInventor( sal_uInt16 nType, SdrInvent
pRet = new SvxTableShape( pObj );
break;
default: // unknown 2D-object on page
- OSL_FAIL("Not implemented Starone-Shape created! [CL]");
+ assert(false && "Not implemented Starone-Shape created");
pRet = new SvxShapeText( pObj );
break;
}
@@ -793,7 +793,7 @@ SvxShape* SvxDrawPage::CreateShapeByTypeAndInventor( sal_uInt16 nType, SdrInvent
}
default: // unknown inventor
{
- OSL_FAIL("AW: Unknown Inventor in SvxDrawPage::CreateShape()");
+ assert(false && "Unknown Inventor in SvxDrawPage::CreateShape()");
break;
}
}
diff --git a/sw/source/filter/ww8/ww8graf.cxx b/sw/source/filter/ww8/ww8graf.cxx
index 2f303bdac0e1..d3c4a102bf40 100644
--- a/sw/source/filter/ww8/ww8graf.cxx
+++ b/sw/source/filter/ww8/ww8graf.cxx
@@ -397,7 +397,7 @@ SdrObject* SwWW8ImplReader::ReadElipse(WW8_DPHEAD const * pHd, SfxAllItemSet &rS
SdrObject* pObj = new SdrCircObj(
*m_pDrawModel,
- OBJ_CIRC,
+ SdrCircKind::Full,
tools::Rectangle(aP0, aP1));
SetStdAttr( rSet, aElipse.aLnt, aElipse.aShd );
@@ -432,7 +432,7 @@ SdrObject* SwWW8ImplReader::ReadArc(WW8_DPHEAD const * pHd, SfxAllItemSet &rSet)
SdrObject* pObj = new SdrCircObj(
*m_pDrawModel,
- OBJ_SECT,
+ SdrCircKind::Section,
tools::Rectangle(aP0, aP1),
nW * 9000,
( ( nW + 1 ) & 3 ) * 9000);
More information about the Libreoffice-commits
mailing list