You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
We are all aware of how hercules source code doesn't follow a consistent code style format, because it is old and lots of people already contributed to it. But now we are already in 2022 and we have plenty of tools to make formatting easier, so my suggestion is that we add clang-format to hercules source, or at very least apply it once.
Is your feature request related to a problem? Please describe.
We are all aware of how hercules source code doesn't follow a consistent code style format, because it is old and lots of people already contributed to it. But now we are already in 2022 and we have plenty of tools to make formatting easier, so my suggestion is that we add clang-format to hercules source, or at very least apply it once.
You can get an idea of how the changes will look like here in this commit:
5a39285#diff-883d0abbee797807e33ccba4fe4d012fe04c286b2c231270e212116b07276ac9
Describe the solution you'd like
clang-format
Suggested configuration
(most similar of what our code already looks like)
Of course it needs to be fine tuned, for example:
but yeah, code looks fine after all these changes
The text was updated successfully, but these errors were encountered: