Discussion:
[Flightgear-commitlogs] [FlightGear] branch next updated: Animation bindings: fix bug with specified index on bindings
(too old to reply)
James Turner
2017-07-23 14:20:36 UTC
Permalink
new f507ca1 Animation bindings: fix bug with specified index on bindings
f507ca1 is described below
SF URL: http://sourceforge.net/p/flightgear/flightgear/ci/f507ca1452a698f4eb472af1f13d1c61be6ebfd3/ <http://sourceforge.net/p/flightgear/flightgear/ci/f507ca1452a698f4eb472af1f13d1c61be6ebfd3/>
Commit: f507ca1452a698f4eb472af1f13d1c61be6ebfd3
Author: Richard Harrison
Committer: Richard Harrison
AuthorDate: Sun Jul 23 13:41:56 2017 +0200
Animation bindings: fix bug with specified index on bindings
This is a fault that I introduced with the bindings rework - by adding the index of 0 to the get prop it overrides the index in the parameter.
This the kind of area where it should now be possible to add unit-tests using FGtestLib, if you are interested in doing that.

I’m currently adding tests for my keybinding / action work, and from my experience already, on a codebase like FlightGear where starting the full system takes a non-trival amount of time, having a test you can run instantly from the command line saves a huge amount of development time.

Kind regards,
James

Loading...