Some are “PascalCase”:
- DrumSynth,
- SpectraVoice,
- WaveShaper,
- MetaModule,
- MultiSynth,
some have spaces with “Inconsistent capitalization”:
- Filter Pro,
- Pitch shifter,
- Vocal filter,
- Vorbis player
That's right. Maybe this is not the best architectural solution. But I don't see any problem here, and I certainly don't see a bug.While it's only cosmetic, if you're going to ever change it
Code: Select all
enum psynth_command
{
PS_CMD_NOP = 0,
...
...
...
//suggestion
PS_CMD_GET_TYPE_AS_INDEX, //module type as index, that way i can ask the module of its type directly without string comparison.
PS_CMD_PUBLISH_CONTROL, //event to publish parameter on parent pnet
PS_CMD_UNPUBLISH_CONTROL, //event to unpublish parameter from parent pnet
PSYNTH_COMMANDS
};