Snap selection to grid: fixed a problem that could cause a crash if geometry got removed by snapping it to the grid

Source/Core/Editing/EditMode.cs

@@ -257,6 +257,10 @@ namespace CodeImp.DoomBuilder.Editing
 		public virtual bool OnMapTestBegin(bool testFromCurrentPosition) { return true; } //called before test map is launched. Returns false if map launch is impossible
 		public virtual void OnMapTestEnd(bool testFromCurrentPosition) { } //called after game engine is closed
 
+		// This should be called by global actions (i.e. that are not part of an editing mode) when they changed map elements,
+		// so that the mode can react to it (for example by rebuilding a blockmap)
+		public virtual void OnMapElementsChanged() { }
+		
 		#endregion
 	}
 }

Source/Core/General/MapManager.cs

@@ -2592,6 +2592,9 @@ namespace CodeImp.DoomBuilder
 				undoredo.WithdrawUndo();
 			}
 
+			// Let the current editing mode know that we changed something
+			General.Editing.Mode.OnMapElementsChanged();
+
 			// Done
 			General.Interface.RedrawDisplay();
 			Cursor.Current = Cursors.Default;
@@ -2651,6 +2654,9 @@ namespace CodeImp.DoomBuilder
 			// Map is changed
 			General.Map.IsChanged = true;
 
+			// Let the current editing mode know that we changed something
+			General.Editing.Mode.OnMapElementsChanged();
+
 			// Done
 			General.Interface.RedrawDisplay();
 			Cursor.Current = Cursors.Default;

Source/Plugins/BuilderModes/ClassicModes/LinedefsMode.cs

@@ -1287,6 +1287,16 @@ namespace CodeImp.DoomBuilder.BuilderModes
 			return base.OnCopyBegin();
 		}
 
+		/// <summary>
+		/// If map elements have changed the blockmap needs to be recreated.
+		/// </summary>
+		public override void OnMapElementsChanged()
+		{
+			base.OnMapElementsChanged();
+
+			CreateBlockmap();
+		}
+
 		//mxd
 		private void RenderComment(Linedef l)
 		{

Source/Plugins/BuilderModes/ClassicModes/SectorsMode.cs

@@ -1545,6 +1545,17 @@ namespace CodeImp.DoomBuilder.BuilderModes
 			base.OnRedoEnd(); //mxd
 		}
 
+
+		/// <summary>
+		/// If map elements have changed the blockmap needs to be recreated.
+		/// </summary>
+		public override void OnMapElementsChanged()
+		{
+			base.OnMapElementsChanged();
+
+			CreateBlockmap();
+		}
+
 		//mxd
 		public override void OnViewSelectionNumbersChanged(bool enabled)
 		{

Source/Plugins/BuilderModes/ClassicModes/VerticesMode.cs

@@ -753,6 +753,16 @@ namespace CodeImp.DoomBuilder.BuilderModes
 			return base.OnCopyBegin();
 		}
 
+		/// <summary>
+		/// If map elements have changed the blockmap needs to be recreated.
+		/// </summary>
+		public override void OnMapElementsChanged()
+		{
+			base.OnMapElementsChanged();
+
+			CreateBlockmap();
+		}
+
 		#endregion
 
 		#region ================== Actions