fluidsynth/CONTRIBUTING.md

45 lines
1.7 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
* submitting pull requests on Github:
2017-10-28 20:15:46 +00:00
https://help.github.com/articles/proposing-changes-to-your-work-with-pull-requests/
2017-10-28 20:48:42 +00:00
* submitting patches to the mailing list.
2017-10-28 20:15:46 +00:00
2017-10-28 20:48:42 +00:00
Patches should be created with `git format-patch`, so in every case you should be familiar with the basics of git.
2017-10-28 20:15:46 +00:00
We'll try comment on our changes within three business days
(and typically, one business day). We may suggest
changes, improvements or alternatives.
Some things that will increase the chance that your pull request or patch is accepted:
2017-10-28 20:48:42 +00:00
* Except for bug fixing, 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".
* Write a meaningful commit messages.
## Style Guide
Find FluidSynth's style guide below. Most of the syntax issues can be automatically applied with `clang-format` and `clang-tidy` using the config files in the repository root.
#### 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