No declaring data inside of switch statement

main.c

@@ -19,24 +19,24 @@
 #include <string.h> // errno error message on file open
 #include <inttypes.h>
 
-static struct GameState gs = {0};
+static struct GameState gs = { 0 };
 static int myplayer = -1;
 static bool right_mouse_down = false;
-static bool keydown[SAPP_KEYCODE_MENU] = {0};
+static bool keydown[SAPP_KEYCODE_MENU] = { 0 };
 typedef struct KeyPressed
 {
 	bool pressed;
 	uint64_t frame;
 } KeyPressed;
-static KeyPressed keypressed[SAPP_KEYCODE_MENU] = {0};
-static V2 mouse_pos = {0};
+static KeyPressed keypressed[SAPP_KEYCODE_MENU] = { 0 };
+static V2 mouse_pos = { 0 };
 static bool mouse_pressed = false;
 static uint64_t mouse_pressed_frame = 0;
 static bool mouse_frozen = false;                    // @BeforeShip make this debug only thing
 static float funval = 0.0f;                          // easy to play with value controlled by left mouse button when held down @BeforeShip remove on release builds
-static struct ClientToServer client_to_server = {0}; // buffer of inputs
-static ENetHost *client;
-static ENetPeer *peer;
+static struct ClientToServer client_to_server = { 0 }; // buffer of inputs
+static ENetHost* client;
+static ENetPeer* peer;
 static float zoom_target = 300.0f;
 static float zoom = 300.0f;
 static sg_image image_itemframe;
@@ -49,7 +49,7 @@ static int cur_editing_rotation = 0;
 static struct BoxInfo
 {
 	enum BoxType type;
-    const char *image_path;
+	const char* image_path;
 	sg_image image;
 } boxes[] = { // if added to here will show up in toolbar, is placeable
 	{
@@ -63,7 +63,7 @@ static struct BoxInfo
 	{
 		.type = BoxBattery,
 		.image_path = "loaded/battery.png",
-    }};
+	} };
 const int boxes_len = sizeof(boxes) / sizeof(*boxes);
 
 struct BoxInfo boxinfo(enum BoxType type)
@@ -74,10 +74,10 @@ struct BoxInfo boxinfo(enum BoxType type)
 			return boxes[i];
 	}
 	Log("No box info found for type %d\n", type);
-    return (struct BoxInfo){0};
+	return (struct BoxInfo) { 0 };
 }
 
-static sg_image load_image(const char *path)
+static sg_image load_image(const char* path)
 {
 	sg_image to_return = sg_alloc_image();
 
@@ -86,7 +86,7 @@ static sg_image load_image(const char *path)
 	int comp = 0;
 	const int desired_channels = 4;
 	stbi_set_flip_vertically_on_load(true);
-    stbi_uc *image_data = stbi_load(path, &x, &y, &comp, desired_channels);
+	stbi_uc* image_data = stbi_load(path, &x, &y, &comp, desired_channels);
 	if (!image_data)
 	{
 		fprintf(stderr, "Failed to load image: %s\n", stbi_failure_reason());
@@ -111,10 +111,10 @@ static void init(void)
 {
 	// @BeforeShip make all fprintf into logging to file, warning dialog grids on failure instead of exit(-1), replace the macros in sokol with this as well, like assert
 
-    Entity *entity_data = malloc(sizeof *entity_data * MAX_ENTITIES);
-    initialize(&gs, entity_data, sizeof *entity_data * MAX_ENTITIES);
+	Entity* entity_data = malloc(sizeof * entity_data * MAX_ENTITIES);
+	initialize(&gs, entity_data, sizeof * entity_data * MAX_ENTITIES);
 
-    sg_desc sgdesc = {.context = sapp_sgcontext()};
+	sg_desc sgdesc = { .context = sapp_sgcontext() };
 	sg_setup(&sgdesc);
 	if (!sg_isvalid())
 	{
@@ -122,7 +122,7 @@ static void init(void)
 		exit(-1);
 	}
 
-    sgp_desc sgpdesc = {0};
+	sgp_desc sgpdesc = { 0 };
 	sgp_setup(&sgpdesc);
 	if (!sgp_is_valid())
 	{
@@ -208,7 +208,7 @@ static void draw_texture_rectangle_centered(V2 center, V2 width_height)
 
 static void draw_texture_centered(V2 center, float size)
 {
-    draw_texture_rectangle_centered(center, (V2){size, size});
+	draw_texture_rectangle_centered(center, (V2) { size, size });
 }
 
 static void draw_circle(V2 point, float radius)
@@ -219,19 +219,19 @@ static void draw_circle(V2 point, float radius)
 	{
 		float progress = (float)i / (float)POINTS;
 		float next_progress = (float)(i + 1) / (float)POINTS;
-        lines[i].a = (V2){.x = cos(progress * 2.0f * PI) * radius, .y = sin(progress * 2.0f * PI) * radius};
-        lines[i].b = (V2){.x = cos(next_progress * 2.0f * PI) * radius, .y = sin(next_progress * 2.0f * PI) * radius};
+		lines[i].a = (V2){ .x = cos(progress * 2.0f * PI) * radius, .y = sin(progress * 2.0f * PI) * radius };
+		lines[i].b = (V2){ .x = cos(next_progress * 2.0f * PI) * radius, .y = sin(next_progress * 2.0f * PI) * radius };
 		lines[i].a = V2add(lines[i].a, point);
 		lines[i].b = V2add(lines[i].b, point);
 	}
 	sgp_draw_lines(lines, POINTS);
 }
 
-static Entity *myentity()
+static Entity* myentity()
 {
 	if (myplayer == -1)
 		return NULL;
-    Entity *to_return = get_entity(&gs, gs.players[myplayer].entity);
+	Entity* to_return = get_entity(&gs, gs.players[myplayer].entity);
 	if (to_return != NULL)
 		assert(to_return->is_player);
 	return to_return;
@@ -278,7 +278,7 @@ static void ui(bool draw, float dt, float width, float height)
 		float y = height - itemframe_height * 1.5;
 		for (int i = 0; i < boxes_len; i++)
 		{
-            if (has_point((AABB){
+			if (has_point((AABB) {
 				.x = x,
 					.y = y,
 					.width = itemframe_width,
@@ -346,9 +346,13 @@ static void frame(void)
 				switch (event.type)
 				{
 				case ENET_EVENT_TYPE_CONNECT:
+				{
 					Log("New client from host %x\n", event.peer->address.host);
 					break;
+				}
+
 				case ENET_EVENT_TYPE_RECEIVE:
+				{
 					// @Robust @BeforeShip use some kind of serialization strategy that checks for out of bounds
 					// and other validation instead of just casting to a struct
 					// "Alignment of structure members can be different even among different compilers on the same platform, let alone different platforms."
@@ -361,11 +365,16 @@ static void frame(void)
 					myplayer = msg.your_player;
 					enet_packet_destroy(event.packet);
 					break;
+				}
+
 				case ENET_EVENT_TYPE_DISCONNECT:
+				{
 					fprintf(stderr, "Disconnected from server\n");
 					exit(-1);
 					break;
 				}
+
+				}
 			}
 			else if (enet_status == 0)
 			{
@@ -381,16 +390,16 @@ static void frame(void)
 
 	// gameplay
 	ui(false, dt, width, height); // handle events
-    V2 build_target_pos = {0};
+	V2 build_target_pos = { 0 };
 	float build_target_rotation = 0.0f;
-    static V2 camera_pos = {0}; // keeps camera at same position after player death
+	static V2 camera_pos = { 0 }; // keeps camera at same position after player death
 	V2 world_mouse_pos = mouse_pos;
 	struct BuildPreviewInfo
 	{
 		V2 grid_pos;
 		float grid_rotation;
 		V2 pos;
-    } build_preview = {0};
+	} build_preview = { 0 };
 	bool hand_at_arms_length = false;
 	{
 		// interpolate zoom
@@ -403,14 +412,14 @@ static void frame(void)
 
 				camera_pos = entity_pos(myentity());
 			}
-            world_mouse_pos = V2sub(world_mouse_pos, (V2){.x = width / 2.0f, .y = height / 2.0f});
+			world_mouse_pos = V2sub(world_mouse_pos, (V2) { .x = width / 2.0f, .y = height / 2.0f });
 			world_mouse_pos.x /= zoom;
 			world_mouse_pos.y /= -zoom;
-            world_mouse_pos = V2add(world_mouse_pos, (V2){.x = camera_pos.x, .y = camera_pos.y});
+			world_mouse_pos = V2add(world_mouse_pos, (V2) { .x = camera_pos.x, .y = camera_pos.y });
 		}
 
 		// calculate build preview stuff
-        EntityID grid_to_build_on = (EntityID){0};
+		EntityID grid_to_build_on = (EntityID){ 0 };
 		if (myentity() != NULL)
 		{
 			V2 hand_pos = V2sub(world_mouse_pos, entity_pos(myentity()));
@@ -427,7 +436,7 @@ static void frame(void)
 			hand_pos = V2scale(V2normalize(hand_pos), hand_len);
 			hand_pos = V2add(hand_pos, entity_pos(myentity()));
 
-            Entity *placing_grid = closest_to_point_in_radius(&gs, hand_pos, BUILD_BOX_SNAP_DIST_TO_SHIP);
+			Entity* placing_grid = closest_to_point_in_radius(&gs, hand_pos, BUILD_BOX_SNAP_DIST_TO_SHIP);
 			if (placing_grid == NULL)
 			{
 				build_preview = (struct BuildPreviewInfo){
@@ -443,7 +452,7 @@ static void frame(void)
 				build_preview = (struct BuildPreviewInfo){
 					.grid_pos = entity_pos(placing_grid),
 					.grid_rotation = entity_rotation(placing_grid),
-                    .pos = pos};
+					.pos = pos };
 			}
 		}
 
@@ -451,7 +460,7 @@ static void frame(void)
 		{
 			// @Robust accumulate total time and send input at rate like 20 hz, not every frame
 
-            struct InputFrame cur_input_frame = {0};
+			struct InputFrame cur_input_frame = { 0 };
 			V2 input = (V2){
 				.x = (float)keydown[SAPP_KEYCODE_D] - (float)keydown[SAPP_KEYCODE_A],
 				.y = (float)keydown[SAPP_KEYCODE_W] - (float)keydown[SAPP_KEYCODE_S],
@@ -464,7 +473,7 @@ static void frame(void)
 				cur_input_frame.build_type = cur_editing_boxtype;
 				cur_input_frame.build_rotation = cur_editing_rotation;
 				cur_input_frame.grid_to_build_on = grid_to_build_on;
-                Entity *grid = get_entity(&gs, grid_to_build_on);
+				Entity* grid = get_entity(&gs, grid_to_build_on);
 				if (grid != NULL)
 				{
 					cur_input_frame.build = grid_world_to_local(grid, build_preview.pos);
@@ -496,7 +505,7 @@ static void frame(void)
 			static double last_input_sent_time = 0.0;
 			if (fabs(last_input_sent_time - time) > TIME_BETWEEN_INPUT_PACKETS)
 			{
-                ENetPacket *packet = enet_packet_create((void *)&client_to_server, sizeof(client_to_server), ENET_PACKET_FLAG_UNRELIABLE_FRAGMENT);
+				ENetPacket* packet = enet_packet_create((void*)&client_to_server, sizeof(client_to_server), ENET_PACKET_FLAG_UNRELIABLE_FRAGMENT);
 				enet_peer_send(peer, 0, packet);
 				last_input_sent_time = time;
 			}
@@ -520,7 +529,7 @@ static void frame(void)
 
 		// sokol drawing library draw in world space
 		// world space coordinates are +Y up, -Y down. Like normal cartesian coords
-        transform_scope {
+		transform_scope{
 			sgp_translate(width / 2, height / 2);
 			sgp_scale_at(zoom, -zoom, 0.0f, 0.0f);
 
@@ -574,17 +583,17 @@ static void frame(void)
 
 			for (int i = 0; i < gs.cur_next_entity; i++)
 			{
-                Entity *e = &gs.entities[i];
+				Entity* e = &gs.entities[i];
 				if (!e->exists)
 					continue;
 				if (e->is_grid) // grid drawing
 				{
-                    Entity *g = e;
+					Entity* g = e;
 					BOXES_ITER(&gs, b, g)
 					{
 						sgp_set_color(1.0f, 1.0f, 1.0f, 1.0f);
-// debug draw force vectors for thrusters
-#if 0
+						// debug draw force vectors for thrusters
+						#if 0
 												{
 													if (b->type == BoxThruster)
 													{
@@ -592,7 +601,7 @@ static void frame(void)
 														dbg_line(box_pos(b), V2add(box_pos(b), V2scale(thruster_force(b), -1.0f)));
 													}
 												}
-#endif
+						#endif
 												if (b->box_type == BoxBattery)
 												{
 													float cur_alpha = sgp_get_color().a;
@@ -653,14 +662,14 @@ static void frame(void)
 			// player
 		for (int i = 0; i < MAX_PLAYERS; i++)
 		{
-            struct Player *player = &gs.players[i];
+			struct Player* player = &gs.players[i];
 			if (!player->connected)
 				continue;
-            Entity *p = get_entity(&gs, player->entity);
+			Entity* p = get_entity(&gs, player->entity);
 			if (p == NULL)
 				continue;
-            static float opacities[MAX_PLAYERS] = {1.0f};
-            static V2 positions[MAX_PLAYERS] = {0};
+			static float opacities[MAX_PLAYERS] = { 1.0f };
+			static V2 positions[MAX_PLAYERS] = { 0 };
 			opacities[i] = lerp(opacities[i], p != NULL ? 1.0f : 0.1f, dt * 7.0f);
 			Color col_to_draw = Collerp(WHITE, GOLD, p->goldness);
 			col_to_draw.a = opacities[i];
@@ -689,7 +698,7 @@ static void frame(void)
 	// UI drawn in screen space
 	ui(true, dt, width, height);
 
-    sg_pass_action pass_action = {0};
+	sg_pass_action pass_action = { 0 };
 	sg_begin_default_pass(&pass_action, width, height);
 	sgp_flush();
 	sgp_end();
@@ -706,7 +715,7 @@ void cleanup(void)
 	enet_deinitialize();
 }
 
-void event(const sapp_event *e)
+void event(const sapp_event* e)
 {
 	switch (e->type)
 	{
@@ -775,7 +784,7 @@ void event(const sapp_event *e)
 	case SAPP_EVENTTYPE_MOUSE_MOVE:
 		if (!mouse_frozen)
 		{
-            mouse_pos = (V2){.x = e->mouse_x, .y = e->mouse_y};
+			mouse_pos = (V2){ .x = e->mouse_x, .y = e->mouse_y };
 		}
 		if (right_mouse_down)
 		{
@@ -786,14 +795,14 @@ void event(const sapp_event *e)
 	}
 }
 
-sapp_desc sokol_main(int argc, char *argv[])
+sapp_desc sokol_main(int argc, char* argv[])
 {
 	if (argc > 1)
 	{
 		_beginthread(server, 0, NULL);
 	}
 	(void)argv;
-    return (sapp_desc){
+	return (sapp_desc) {
 		.init_cb = init,
 			.frame_cb = frame,
 			.cleanup_cb = cleanup,

server.c

@@ -6,26 +6,26 @@
 #include <stdlib.h>
 
 // started in a thread from host
-void server(void *data)
+void server(void* data)
 {
 	(void)data;
 
 	stm_setup();
 
-    struct GameState gs = {0};
+	struct GameState gs = { 0 };
 	size_t entities_size = (sizeof(Entity) * MAX_ENTITIES);
-    Entity *entity_data = malloc(entities_size);
+	Entity* entity_data = malloc(entities_size);
 	initialize(&gs, entity_data, entities_size);
 	Log("Allocated %zu bytes for entities\n", entities_size);
 
 	// one box policy
 	if (true)
 	{
-        Entity * grid = new_entity(&gs);
+		Entity* grid = new_entity(&gs);
 		grid_create(&gs, grid);
-        entity_set_pos(grid, (V2){-BOX_SIZE*2, 0.0f});
-        Entity * box = new_entity(&gs);
-        box_create(&gs, box, grid, (V2){0});
+		entity_set_pos(grid, (V2) { -BOX_SIZE * 2, 0.0f });
+		Entity* box = new_entity(&gs);
+		box_create(&gs, box, grid, (V2) { 0 });
 
 	}
 
@@ -36,7 +36,7 @@ void server(void *data)
 	}
 
 	ENetAddress address;
-    ENetHost *server;
+	ENetHost* server;
 	int sethost = enet_address_set_host_ip(&address, LOCAL_SERVER_ADDRESS);
 	if (sethost != 0)
 	{
@@ -60,7 +60,7 @@ void server(void *data)
 	ENetEvent event;
 	uint64_t last_processed_time = stm_now();
 	float total_time = 0.0f;
-    uint64_t player_to_latest_tick_processed[MAX_PLAYERS] = {0};
+	uint64_t player_to_latest_tick_processed[MAX_PLAYERS] = { 0 };
 	while (true)
 	{
 		// @Speed handle enet messages and simulate gamestate in parallel, then sync... must clone gamestate for this
@@ -78,6 +78,7 @@ void server(void *data)
 				switch (event.type)
 				{
 				case ENET_EVENT_TYPE_CONNECT:
+				{
 					Log("A new client connected from %x:%u.\n",
 						event.peer->address.host,
 						event.peer->address.port);
@@ -98,13 +99,16 @@ void server(void *data)
 					}
 					else
 					{
-                        event.peer->data = (void *)player_slot;
-                        gs.players[player_slot] = (struct Player){0};
+						event.peer->data = (void*)player_slot;
+						gs.players[player_slot] = (struct Player){ 0 };
 						gs.players[player_slot].connected = true;
 					}
 
 					break;
+				}
+					
 				case ENET_EVENT_TYPE_RECEIVE:
+				{
 					// Log("A packet of length %zu was received on channel %u.\n",
 					//        event.packet->dataLength,
 					//        event.channelID);
@@ -116,7 +120,7 @@ void server(void *data)
 					}
 					else
 					{
-                        struct ClientToServer received = {0};
+						struct ClientToServer received = { 0 };
 						memcpy(&received, event.packet->data, length);
 						int64_t player_slot = (int64_t)event.peer->data;
 						uint64_t latest_tick = player_to_latest_tick_processed[player_slot];
@@ -156,13 +160,19 @@ void server(void *data)
 					enet_packet_destroy(event.packet);
 
 					break;
+				}
+
 
 				case ENET_EVENT_TYPE_DISCONNECT:
+				{
 					int player_index = (int64_t)event.peer->data;
 					Log("%" PRId64 " disconnected player index %d.\n", (int64_t)event.peer->data, player_index);
 					gs.players[player_index].connected = false;
 					// box_destroy(&gs.players[player_index].box);
 					event.peer->data = NULL;
+					break;
+				}
+
 				}
 			}
 		}
@@ -181,7 +191,7 @@ void server(void *data)
 		if (processed)
 		{
 #define MAX_BYTES_SIZE 2048 * 2
-            static char bytes_buffer[MAX_BYTES_SIZE] = {0};
+			static char bytes_buffer[MAX_BYTES_SIZE] = { 0 };
 			for (int i = 0; i < server->peerCount; i++)
 			{
 				// @Speed don't recreate the packet for every peer, gets expensive copying gamestate over and over again
@@ -196,7 +206,7 @@ void server(void *data)
 				int len = 0;
 				into_bytes(&to_send, bytes_buffer, &len, MAX_BYTES_SIZE);
 
-                ENetPacket *gamestate_packet = enet_packet_create((void *)bytes_buffer, len, ENET_PACKET_FLAG_UNRELIABLE_FRAGMENT);
+				ENetPacket* gamestate_packet = enet_packet_create((void*)bytes_buffer, len, ENET_PACKET_FLAG_UNRELIABLE_FRAGMENT);
 				enet_peer_send(&server->peers[i], 0, gamestate_packet);
 			}
 		}