Fix for Issue#8725: refactoring _currentTrackBegin and _previousTrackPiece in Ride.cpp to use CoordsXYZ struct

[ndngo]

In the Ride.cpp file,

uint16_t _currentTrackBeginX;
uint16_t _currentTrackBeginY;
uint16_t _currentTrackBeginZ;

were refactored to use the CoordsXYZ struct

CoordsXYZ _previousTrackPiece;

struct CoordsXYZ
{
    int32_t x = 0;
    int32_t y = 0;
    int32_t z = 0;

    CoordsXYZ() = default;
    constexpr CoordsXYZ(int32_t _x, int32_t _y, int32_t _z)
        : x(_x)
        , y(_y)
        , z(_z)
    {
    }
};

such that

x = _currentTrackBeginX;
y = _currentTrackBeginY;
z = _currentTrackBeginZ;

becomes

x = _currentTrackBegin.x;
y = _currentTrackBegin.y;
z = _currentTrackBegin.z;

The same refactoring logic was used to refactor _previousTrackPiece.

[IntelOrca]

You will also need to apply changes to other source files which use _currentTrackBegin and _previousTrackPiece

[duncanspumpkin]

Unsure what this change has been made for. Please revert.

[duncanspumpkin]

Looks good. Almost there just need to make a formatting adjustment to keep the clang-format ci happy.

If your unsure how to find more information on that look below where it says continuous-integration/travis-ci/pr — The Travis CI build failed and click details. Then when on Travis select the clang format option and it will tell you which line needs adjusting.