Topics

1

Programming / Strafing backwards and diagonal strafing bug

« on: February 27, 2016, 09:34:49 pm »

Hello again,

I'm right away posting another contribution.
I was looking at the strafing feature (another nice addition ;-) and what it could take to allow backwards movement.
The code is nearly ready so I gave it a try just to find out some strange outcome.

It led me to find a bug in current nightly when strafing diagonally.
The movement preview shows correctly that it will cost 7 TUs (and should cost 3 Energy).
However it actually costs 5 TUs and 5 Energy because the engine has both "strafe" and "run" flags set.
This is incorrect IMHO so I tried to fix this.
The fix also fixed the incorrect behaviour mentioned above when backwards strafing was enabled.
Specifically, the movement around building corner (X +- 1, Y +- 1) was affected, which has the same basic properties as diagonal movement.

I'm attaching both file patches, bug fix and strafe enhancement.
I have done just basic testing, please if you try it, let me know how it works.

Thanks, Marek

2

Programming / Craft interception using meeting point

« on: February 27, 2016, 08:56:38 pm »

Hi everyone,

This is my first post, I'm new here, although I'm far from new to XCom games.
I have spent quite some time in the past playing this game and am very excited about this project.
Finally I have given it more thorough attention and started to look at the code itself.

I have found this very nice feature when interceptors chase the UFOs flying to meeting point, which I always thought was missing and should have been included.
At the same time I have realized that when interceptor has no chance of catching the UFO because of insufficient speed, it reverts to its previous behaviour, which I find unreasonable with this setting enabled.

So I have tried to patch things up and here is the outcome.
It is quite a simple patch I would say that also tries to find the best direction for the interceptor to get as close as possible even if it cannot actually catch the UFO.

I'm not experienced with git but tried my best.
I can post it in the bug tracker if preferred.

What do you think, does this behaviour change make sense ?

Thanks again for your efforts,
Marek