fluidsynth/CONTRIBUTING.md

41 lines
1.8 KiB
Markdown
Raw Normal View History

2017-10-28 20:15:46 +00:00
# Contributing
Thanks for considering to contribute to FluidSynth. Before implementing
any huge new feature, consider bringing up your ideas on our mailing list:
https://lists.nongnu.org/mailman/listinfo/fluid-dev
2017-10-28 20:48:42 +00:00
Contributing can be done by
2017-11-04 08:05:17 +00:00
* [submitting pull requests on Github](
https://help.github.com/articles/proposing-changes-to-your-work-with-pull-requests/) or
2017-10-28 20:48:42 +00:00
* submitting patches to the mailing list.
2017-10-28 20:15:46 +00:00
2018-03-08 13:01:40 +00:00
Patches should be created with `git format-patch`, so in every case you should be familiar with the basics of git. Make sure you develop against the master branch, i.e. **not** against any FluidSynth release.
2017-10-28 20:15:46 +00:00
Some things that will increase the chance that your pull request or patch is accepted:
2017-10-29 09:41:11 +00:00
* Give a reasoning / motivation for any changes or proposals you make.
2017-10-28 20:15:46 +00:00
* Follow our style guide.
* Keep your commits "atomic".
2017-11-04 08:05:17 +00:00
* Write meaningful commit messages.
## Style Guide
Find FluidSynth's style guide below. Syntax related issues, like missing braces, will be automatically taken care of when compiling fluidsynth by calling `make` (provided that cmake has found `clang-tidy >= 5.0` on your system). An explicit reformatting can be achieved using `make format` (provided that cmake has found `clang-format` on your system).
#### General
* Every function should have a short comment explaining it's purpose
* Every public API function **must** be documented with purpose, params and return value
2017-10-28 20:58:53 +00:00
* Prefer signed integer types to unsigned ones
* Use spaces rather than tabs
* Avoid macros
#### Naming Conventions
* Words separated by underscores
* Macros always UPPER_CASE
* Function and Variable names always lower_case, (e.g. `fluid_componentname_purpose()`)
#### Bracing
* Every block after an if, else, while or for should be enclosed in braces
* **Allman-Style** braces everywhere