fluidsynth/CONTRIBUTING.md

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

Contributing can be done by

• submitting pull requests on Github: https://help.github.com/articles/proposing-changes-to-your-work-with-pull-requests/
• submitting patches to the mailing list.

Patches should be created with git format-patch, so in every case you should be familiar with the basics of git.

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:

• Except for bug fixing, give a reasoning / motivation for any changes or proposals you make.
• 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
• Prefer int to unsigned int
• 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