remove useless variable mSnappedToVertex

lbartoletti · wonder-sk · commit 1ac18f919e0d · 2019-03-12T17:56:49.000+01:00
diff --git a/src/gui/qgsadvanceddigitizingdockwidget.cpp b/src/gui/qgsadvanceddigitizingdockwidget.cpp
@@ -555,8 +555,6 @@ bool QgsAdvancedDigitizingDockWidget::applyConstraints( QgsMapMouseEvent *e )
 
   mSnapMatch = context.snappingUtils->snapToMap( point );
 
-  mSnappedToVertex = mSnapMatch.hasVertex();
-
   // update the point list
   updateCurrentPoint( point );
 
@@ -1028,7 +1026,6 @@ void QgsAdvancedDigitizingDockWidget::clearPoints()
 {
   mCadPointList.clear();
   mSnappedSegment.clear();
-  mSnappedToVertex = false;
 
   updateCapacity();
 }
diff --git a/src/gui/qgsadvanceddigitizingdockwidget.h b/src/gui/qgsadvanceddigitizingdockwidget.h
@@ -317,7 +317,7 @@ class GUI_EXPORT QgsAdvancedDigitizingDockWidget : public QgsDockWidget, private
     /**
      * Is it snapped to a vertex
      */
-    inline bool snappedToVertex() const { return mSnappedToVertex; }
+    inline bool snappedToVertex() const { return ( mSnapMatch.isValid() && mSnapMatch.hasVertex() ); }
 
     /**
      * Snapped to a segment
@@ -485,7 +485,6 @@ class GUI_EXPORT QgsAdvancedDigitizingDockWidget : public QgsDockWidget, private
     // point list and current snap point / segment
     QList<QgsPointXY> mCadPointList;
     QList<QgsPointXY> mSnappedSegment;
-    bool mSnappedToVertex = false;
 
     bool mSessionActive = false;
 

Original file line number	Diff line number	Diff line change
`@@ -555,8 +555,6 @@ bool QgsAdvancedDigitizingDockWidget::applyConstraints( QgsMapMouseEvent *e )`
`555`	`555`
`556`	`556`	`mSnapMatch = context.snappingUtils->snapToMap( point );`
`557`	`557`
`558`		`- mSnappedToVertex = mSnapMatch.hasVertex();`
`559`		`-`
`560`	`558`	`// update the point list`
`561`	`559`	`updateCurrentPoint( point );`
`562`	`560`
`@@ -1028,7 +1026,6 @@ void QgsAdvancedDigitizingDockWidget::clearPoints()`
`1028`	`1026`	`{`
`1029`	`1027`	`mCadPointList.clear();`
`1030`	`1028`	`mSnappedSegment.clear();`
`1031`		`- mSnappedToVertex = false;`
`1032`	`1029`
`1033`	`1030`	`updateCapacity();`
`1034`	`1031`	`}`