fmang / opustags

Ogg Opus tags editor
BSD 3-Clause "New" or "Revised" License
82 stars 11 forks source link

Settle on #include style #7

Closed rr- closed 8 years ago

rr- commented 8 years ago

tests/actions_tests.cc vs src/main.cc

fmang commented 8 years ago

Here's my preferred order for header inclusion, roughly based on relevance:

This order helps ensure headers include their own dependencies.

I usually separate these groups with a blank line, and sort them using the default alphetical-ish order.

I'll leave this issue open to discussion for a week, then will fix the existing files. It's hardly an issue in my opinion though, but I don't wanna fall into the “style doesn't matter” category.