diff options
-rw-r--r-- | include/svx/svdocirc.hxx | 19 | ||||
-rw-r--r-- | sc/qa/extras/anchor.cxx | 2 | ||||
-rw-r--r-- | sc/qa/unit/ucalc.cxx | 2 | ||||
-rw-r--r-- | sc/qa/unit/ucalc_sort.cxx | 2 | ||||
-rw-r--r-- | sc/source/core/tool/detfunc.cxx | 2 | ||||
-rw-r--r-- | sc/source/filter/excel/xiescher.cxx | 4 | ||||
-rw-r--r-- | sd/source/ui/func/fuconarc.cxx | 12 | ||||
-rw-r--r-- | sd/source/ui/func/fuconrec.cxx | 2 | ||||
-rw-r--r-- | svx/inc/sxcikitm.hxx | 8 | ||||
-rw-r--r-- | svx/source/dialog/imapwnd.cxx | 4 | ||||
-rw-r--r-- | svx/source/sdr/properties/circleproperties.cxx | 20 | ||||
-rw-r--r-- | svx/source/svdraw/svdfmtf.cxx | 8 | ||||
-rw-r--r-- | svx/source/svdraw/svdibrow.cxx | 2 | ||||
-rw-r--r-- | svx/source/svdraw/svdobj.cxx | 10 | ||||
-rw-r--r-- | svx/source/svdraw/svdocirc.cxx | 153 | ||||
-rw-r--r-- | svx/source/unodraw/unopage.cxx | 4 | ||||
-rw-r--r-- | sw/source/filter/ww8/ww8graf.cxx | 4 |
17 files changed, 121 insertions, 137 deletions
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); |