caitp / TwitchSwitcher

OBS-Studio plugin to switch Twitch.tv game name and channel status automatically
Other
35 stars 5 forks source link

Code cleanup #3

Open caitp opened 8 years ago

caitp commented 8 years ago

This stuff is a mess.

Conventions should be established. I like the "TSW_" prefix for macros, and TSW might not be a bad prefix for many internal class names, either.

The "Impl" suffix may be a bit misleading, and should probably be replaced with something.

Includes should be sorted alphabetically, in sections: 1) libc++ headers, 2) libobs/library headers, 3) local includes.

No need for a column limit, but methods should probably not begin with the verb "get". They should, however, always begin with a verb, to clarify intent.