Make tox.c unambiguously parseable.

Rules:
1. Constants are uppercase names: THE_CONSTANT.
2. SUE[1] types start with an uppercase letter and have at least one
   lowercase letter in it: The_Type, THE_Type.
3. Function types end in "_cb": tox_friend_connection_cb.
4. Variable and function names are all lowercase: the_function.

This makes it easier for humans reading the code to determine what an
identifier means. I'm not convinced by the enum type name change, but I
don't know a better rule. Currently, a lot of enum types are spelled like
constants, which is confusing.

[1] struct/union/enum

1 files changed, 5 insertions, 2 deletions

diff --git a/toxcore/group.h b/toxcore/group.h
index a81dac39..684f7972 100644
--- a/toxcore/group.h
+++ b/toxcore/group.h
@@ -108,6 +108,9 @@ typedef struct {
     void (*group_on_delete)(void *, uint32_t);
 } Group_c;
 
+typedef void g_conference_invite_cb(Messenger *, uint32_t, int, const uint8_t *, size_t, void *);
+typedef void g_conference_message_cb(Messenger *m, uint32_t, uint32_t, int, const uint8_t *, size_t, void *);
+
 typedef struct {
     Messenger *m;
     Friend_Connections *fr_c;
@@ -115,8 +118,8 @@ typedef struct {
     Group_c *chats;
     uint32_t num_chats;
 
-    void (*invite_callback)(Messenger *m, uint32_t, int, const uint8_t *, size_t, void *);
-    void (*message_callback)(Messenger *m, uint32_t, uint32_t, int, const uint8_t *, size_t, void *);
+    g_conference_invite_cb *invite_callback;
+    g_conference_message_cb *message_callback;
     void (*peer_name_callback)(Messenger *m, uint32_t, uint32_t, const uint8_t *, size_t, void *);
     void (*peer_list_changed_callback)(Messenger *m, uint32_t, void *);
     void (*title_callback)(Messenger *m, uint32_t, uint32_t, const uint8_t *, size_t, void *);