AP_Logger: correct the filling order of the struct

[xianglunkai]

correct compilation errors like this:
../../libraries/AP_Logger/AP_Logger_Backend.cpp: In member function ‘bool AP_Logger_Backend::Write_VER()’:
../../libraries/AP_Logger/AP_Logger_Backend.cpp:577:5: sorry, unimplemented: non-trivial designated initializers not supported
};
^

[peterbarker]

Which compiler?

[xianglunkai]

@peterbarker
ubuntu18.04 arm-gcc7.5

[peterbarker]

@peterbarker ubuntu18.04 arm-gcc7.5

We've moved forward in our assumed compiler standards, I'm afraid.

Please upgrade your compiler; apt is easy enough for the Linux compiler, and we provide the release stm32 compiler on firmware.ardupilot.org if needed.

Also note that Ubuntu 18.04 is out of LTS, so more problems may arise into the future... we stopped fixing Python2 problems as soon as it went out of support, for example!

[xianglunkai]

@peterbarker
Txs！
But I think it's best to fix this small issue to prevent testing by other users

[khancyr]

this is correctly supported by recent compiler so we don't need this patch that is a bit less performant than the direct initilization. thanks anyway

[peterbarker]

@xianglunkai turns out we actually do use g++ 7.5.0 - on our build server for non-ChibiOS targets.

I'm planning on merging this once CI passes - because at the moment we're not getting "latest" builds for many of our linux-based boards (e.g. https://firmware.ardupilot.org/Plane/latest/bbbmini/)

Thanks!

[peterbarker]

Merged, thanks @xianglunkai .