AndyLoft / AB3D2_LevelED

Level editors for Alien Breed 3D 2 by Team17
2 stars 1 forks source link

moving/removing control points causes a crash #1

Open AndyLoft opened 1 year ago

AndyLoft commented 1 year ago

Control points can't be moved or removed (you just get a crash for your trouble) if they have any links (physical or visual). It's often next to impossible to try and remove the links manually because turning on visual links in a level tends to create an impenetrable purple web. Ideally, you should be able to select a control point and press something to show only the visual links it has and something else to remove them all. It would also be handy if there was a global option to just remove all visual links in the whole level in the same way you can automatically calculate the visual ones.

AndyLoft commented 1 year ago

looking over the code it would appear that the procedure to select the control point closest to the mouse cursor is never called. I will redirect the 'g' key to the correct routine and see what happens.

AndyLoft commented 1 year ago

keyboard 'g' redirected to _V_CPTMOVE when OP=35 (move control point) is selected, it was calling _V_CONNECTCONT. this seems to be working with the limited testing/experience I have with setting up control points.

AndyLoft commented 1 year ago

deleting control points works sometimes and crashes on others...

AndyLoft commented 1 year ago

check needed to see if control point is selected when trying to move them to avoid a crash. fix for this crash uploaded to EAB https://eab.abime.net/showpost.php?p=1633744&postcount=268

AndyLoft commented 1 year ago

control points seem to be deletable until the add alien/object is selected, after which, deleting a control point results in a crash and being dumped back to workbench with an 'illegal function call'

I am thinking of removing the 'delete' control point option till it can be better understood and fixed. it shouldn't cause any usability issues as they can still be reused by unlinking and moving to where needed.