search

SFTtech / openage

Free (as in freedom) open source clone of the Age of Empires II engine 🚀
http://openage.dev
Other
12.51k stars 1.11k forks source link

Revert "Merge pull request #1602 from heinezen/clang-format-align" #1605

Closed heinezen closed 7 months ago

heinezen commented 7 months ago

Currently, using BlockIndent can reformat other parts of the code to something less readable, e.g.

void WorldRenderEntity::update(const uint32_t ref_id,
                               const curve::Continuous<coord::phys3> &position,
                               const curve::Segmented<coord::phys_angle_t> &angle,
                               const std::string animation_path,
                               const time::time_t time)

to

void WorldRenderEntity::update(const uint32_t ref_id, const curve::Continuous<coord::phys3> &position, const curve::Segmented<coord::phys_angle_t> &angle, const std::string animation_path, const time::time_t time)

This is not really great for readability, so I would switch back to the old style until we or clang-format has a solution for this.