WIP [NOT FOR MERGING]: code style effort #7
Closed
MicheleC
wants to merge 2 commits from MicheleC:feat/code-formatting
into master
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'MicheleC:feat/code-formatting'
Deleting a branch is permanent. It CANNOT be undone. Continue?
This PR is the first attempt at code styling for libr.
The reasons to start from libr is that it is the first package to be built (besided extra deps) and it is very small.
This PR is meant as a review space, where we can all examine the results, point outs what is not ok and later review after fixes have been applied. It is recommended to look at the files on CLI and checking for spaces/tabs as well.
There are still things to be fixed, but more people looking at the results are likely to spot more problems than a single person. It would be great if you could spend some time to go through the files and highlight (as comments in the PR code) things that are not quite right. I will later work on getting those fixed. Note, you can safely skip the .format_errorsX.txt files and the config file .uncrustify_tde.cfg
Current version has been formatted using upstream uncrustify as of commit dedc17ad with the config file included in this PR.
Note, I am using a forked repository and not the official libr one because I expect this work to take some time and consume some disk space. Once we are happy with the result, we can simply format the official repo and delete this one.
Also, after we are done with libr, we will progressively work on bigger repos, again fixing new issues as they arise while at the same time making sure already formatted repos remain unchanged.
Apologies, created PR in wrong repo. Closing and please ignore!