Guard against the case where mpResults is NULL somehow. Possibly due to a poorly timed cancellation of the job request.
Tidy up the iterator end checking in SpaceStationType.cpp
Limit docking to the applicable docking bay sizes.
Correct some lines endings.
Add method IsLeaving to SpaceStation so we can check if we're in the process of undocking.
Only add applicable options if we're not in the process of undocking.
Clear the "hasAutopilot" flag if we launching, this nicely prevents it showing any autopilot options and avoids double checking it.
Thanks to radius75 for identifying the correct fix!
I did also forget an `&` in there :/
Include the check that we're flying in the initial autopilot test.
Kind of logical for you to only be able to autopilot if you're actually flying.
Removed the IsLaunching method, changed the remaining check to see if we're actually flying.
Also fixed some line endings.
Merge branch 'Fix2263'
Merge remote-tracking branch 'fluffyfreak/ShipDockingSize'
Fix an itsy bitsy boo boo.
The old code generated a point on the sphere for the colour which was based on x and y values which started at 1 instead of 0 which was incorrect. Now I generate it straight from the stored position so colour will always match the position.
Change it back to my initial fix of subtracting 1 from the x and y loops variables.
The previous change would have broken the normal generation.
Also moved the abort checks to somewhere more sensible.
Merge remote-tracking branch 'fluffyfreak/Fix2282'
Merge remote-tracking branch 'fluffyfreak/Fix2295'
merge 'threaded terrain' changelog items