coppro
Level: Smitemaster
Rank Points: 1308
Registered: 11-24-2005
IP: Logged
|
Re: Tiny bug (0)
While we're at it, you should not be able to place Beethro on master walls only if you've mastered the hold you're editing. (No, seriously. You can, but you can't place master walls under Beethro in any circumstances).
Actually, this should work with any wall. Here's why:
Within CEditRoomWidget::IsSafePlacement, there are multiple fallthroughs. (this is bad and has some other undesirable effects, like the ability to put an open yellow door under a seep). However, what stands out is that the very last thing checked is whether the square is empty. As such, what happens is that it says "Is there tar here?" and if it says yes, immediately returns true. IMHO, that entire function could make do with a rewrite.
Maybe a new function is in order, IsSafeRemoval. I don't think that you should be able to remove the walls out from under seeps, for example.
|