diff --git a/src/Mod/TechDraw/App/DrawProjGroup.cpp b/src/Mod/TechDraw/App/DrawProjGroup.cpp index ebb729ad2..b17de8d0a 100644 --- a/src/Mod/TechDraw/App/DrawProjGroup.cpp +++ b/src/Mod/TechDraw/App/DrawProjGroup.cpp @@ -453,7 +453,8 @@ void DrawProjGroup::arrangeViewPointers(DrawProjGroupItem *viewPtrs[10]) const if (oView) { const char *viewTypeCStr = oView->Type.getValueAsString(); if (strcmp(viewTypeCStr, "Front") == 0) { - viewPtrs[thirdAngle ? 4 : 4] = oView; + //viewPtrs[thirdAngle ? 4 : 4] = oView; + viewPtrs[4] = oView; } else if (strcmp(viewTypeCStr, "Left") == 0) { viewPtrs[thirdAngle ? 3 : 5] = oView; } else if (strcmp(viewTypeCStr, "Right") == 0) { diff --git a/src/Mod/TechDraw/App/DrawProjectSplit.h b/src/Mod/TechDraw/App/DrawProjectSplit.h index ec04f27e0..753c9b028 100644 --- a/src/Mod/TechDraw/App/DrawProjectSplit.h +++ b/src/Mod/TechDraw/App/DrawProjectSplit.h @@ -55,7 +55,10 @@ struct splitPoint { class edgeSortItem { public: - edgeSortItem() {} + edgeSortItem() { + startAngle = endAngle = 0.0; + idx = 0; + } ~edgeSortItem() {} Base::Vector3d start; diff --git a/src/Mod/TechDraw/App/DrawViewSpreadsheet.cpp b/src/Mod/TechDraw/App/DrawViewSpreadsheet.cpp index 52ffb73e5..92fe6c9b5 100644 --- a/src/Mod/TechDraw/App/DrawViewSpreadsheet.cpp +++ b/src/Mod/TechDraw/App/DrawViewSpreadsheet.cpp @@ -246,7 +246,7 @@ std::string DrawViewSpreadsheet::getSheetImage(void) celltext = field.str(); } // get colors, style, alignment and span - int alignment; + int alignment = 0; std::string bcolor = "none"; std::string fcolor = c.asCSSString(); std::string textstyle = ""; diff --git a/src/Mod/TechDraw/App/Geometry.h b/src/Mod/TechDraw/App/Geometry.h index 239366ce5..f638e0dd2 100644 --- a/src/Mod/TechDraw/App/Geometry.h +++ b/src/Mod/TechDraw/App/Geometry.h @@ -164,7 +164,7 @@ class TechDrawExport BezierSegment: public BaseGeom { public: BezierSegment(const TopoDS_Edge &e); - BezierSegment() {} + BezierSegment() { poles = degree = 0; } ~BezierSegment() = default; int poles; diff --git a/src/Mod/TechDraw/Gui/QGIView.cpp b/src/Mod/TechDraw/Gui/QGIView.cpp index e5915fe11..e06122a7e 100644 --- a/src/Mod/TechDraw/Gui/QGIView.cpp +++ b/src/Mod/TechDraw/Gui/QGIView.cpp @@ -66,6 +66,7 @@ const float labelCaptionFudge = 0.2f; // temp fiddle for devel QGIView::QGIView() :QGraphicsItemGroup(), + viewObj(nullptr), locked(false), borderVisible(true), m_innerView(false) @@ -304,7 +305,9 @@ void QGIView::setViewFeature(TechDraw::DrawView *obj) void QGIView::toggleCache(bool state) { // temp for devl. chaching was hiding problems WF - setCacheMode((state)? NoCache : NoCache); + //setCacheMode((state)? NoCache : NoCache); + Q_UNUSED(state); + setCacheMode(NoCache); } void QGIView::toggleBorder(bool state) diff --git a/src/Mod/TechDraw/Gui/QGIViewPart.cpp b/src/Mod/TechDraw/Gui/QGIViewPart.cpp index cd17e5903..5049d4dfe 100644 --- a/src/Mod/TechDraw/Gui/QGIViewPart.cpp +++ b/src/Mod/TechDraw/Gui/QGIViewPart.cpp @@ -755,7 +755,8 @@ void QGIViewPart::toggleCache(bool state) QList items = childItems(); for(QList::iterator it = items.begin(); it != items.end(); it++) { //(*it)->setCacheMode((state)? DeviceCoordinateCache : NoCache); //TODO: fiddle cache settings if req'd for performance - (*it)->setCacheMode((state)? NoCache : NoCache); + Q_UNUSED(state); + (*it)->setCacheMode(NoCache); (*it)->update(); } } diff --git a/src/Mod/TechDraw/Gui/TaskProjGroup.cpp b/src/Mod/TechDraw/Gui/TaskProjGroup.cpp index 8d7cdf6cd..a0191d887 100644 --- a/src/Mod/TechDraw/Gui/TaskProjGroup.cpp +++ b/src/Mod/TechDraw/Gui/TaskProjGroup.cpp @@ -418,7 +418,7 @@ Base::Vector3d TaskProjGroup::get3DViewDir() Base::Vector3d viewDir(0.0,-1.0,0.0); //default to front std::list mdis = Gui::Application::Instance->activeDocument()->getMDIViews(); Gui::View3DInventor *view; - Gui::View3DInventorViewer *viewer; + Gui::View3DInventorViewer *viewer = nullptr; for (auto& m: mdis) { //find the 3D viewer view = dynamic_cast(m); if (view) { @@ -491,8 +491,10 @@ bool TaskProjGroup::reject() ///////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////// //TODO: Do we really need to hang on to the TaskDlgProjGroup in this class? IR -TaskDlgProjGroup::TaskDlgProjGroup(TechDraw::DrawProjGroup* featView, bool mode) : TaskDialog(), - multiView(featView) +TaskDlgProjGroup::TaskDlgProjGroup(TechDraw::DrawProjGroup* featView, bool mode) + : TaskDialog() + , viewProvider(nullptr) + , multiView(featView) { //viewProvider = dynamic_cast(featView); widget = new TaskProjGroup(featView,mode);