search

C5T / Current

C++ framework for realtime machine learning.
https://medium.com/dima-korolev/current-for-realtime-machine-learning-4f04aa8ab81a
97 stars 29 forks source link

Moving to `clang-format-10`. #896

Closed dkorolev closed 2 years ago

dkorolev commented 3 years ago

Hi @mzhurovich,

This PR moves Current to clang-format-10. I've also tweaked some .clang-format rules, see https://github.com/dkorolev/Current/commit/a290019130a852ae5da9f55b2f12cd9a68deb9ab for how it reformats the code.

Thinking of wrapping every top-line #include "${relative_path}/port.h" into clang-format-disabling guards, because we used to make it an important rule that port.h is #include-d first (i.e. #define NOMINMAX on Windows). Although we might well live without this change and see if all goes well, right?

Thanks, Dima

dkorolev commented 2 years ago

@mzhurovich, I'll merge this one, as clang-format-3.6 is quite obsolete anyway.