Fix sector deletion in sector mode

usernameak / GZDoomSLADE

2 stars 1 forks source link

Fix sector deletion in sector mode #2

Closed Talon1024 closed 5 years ago

Talon1024 commented 6 years ago

Since removeSide now removes sectors, deleting a sector will result in two sectors being deleted. This change delegates the sector deletion to removeSide.

NOTE: If you intend to merge SLADE's master or stable branch, I've already made a pull request like this to SLADE, and it has already been merged.

usernameak commented 5 years ago

I am going to merge SLADE's master soon. It'll be merged in then.

OrdinaryMagician commented 5 years ago

Oh thank goodness, upstream patches have been piling up for my personal builds.