Remove execute bit. [0]

Not sure why it was set…though I’m kinda impressed by Git preserving it!
Assorted code quality fixes for McvToolbarEditor.

I’ve been sitting on a bunch of these types of changes for a while now,
so don’t be alarmed if there are more of these “quality” commits in the
near future!

[0]