Fix compiler warnings in newer builds

[LadyCailin]

This is a 'light touch' change, in that I didn't change most of the older syntax other than the places that will cause problems in future builds. For the most part, this manifests itself in using the new syntax for procs (proc _name() {} instead of proc(_name, ...)) quoting event names in binds, as well as removing a few other uses of bare strings here and there. These eliminate all the compiler warnings in current builds of CH, but should be backwards compatible with all builds in the past 5 years or whatever. There are a few compile errors which I didn't fix, because presumably you are still on an older build in which these work, and fixing them for modern CH will break it for older CH. However, there are only a handful of these, so upgrading in the future should be much easier now for you.

[totemo]

Hi LadyCailin. Thanks for the PR. Unfortunately we're using the luckperms branch for P and C at the moment and we'll be merging that once we get around to updating the lobby.

[LadyCailin]

Doh! Deaygo told me this after the fact 🙄 I’ll see about doing the other branch then.

…

On 5 Sep 2019, at 01:39, totemo ***@***.***> wrote: Hi LadyCailin. Thanks for the PR. Unfortunately we're using the luckperms branch for P and C at the moment and we'll be merging that once we get around to updating the lobby. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.