Un-nest namespaces, because that's a C++17 feature

For some reason gcc doesn't tell us this

src/net-arp.cpp

@@ -8,7 +8,9 @@
 #include "types.h"
 #include <uspi.h>
 
-namespace Net::Arp
+namespace Net
+{
+	namespace Arp
 	{
 		Packet::Packet() {}
 
@@ -96,7 +98,8 @@ namespace Net::Arp
 			size += ethernetHeader.Serialize(buffer + size, sizeof(buffer) - size);
 			size += arpPacket.Serialize(buffer + size, sizeof(buffer) - size);
 
-		const auto expectedSize = ethernetHeader.SerializedLength() + arpPacket.SerializedLength();
+			const auto expectedSize =
+				ethernetHeader.SerializedLength() + arpPacket.SerializedLength();
 			assert(size == expectedSize);
 			assert(size <= sizeof(buffer));
 
@@ -142,7 +145,8 @@ namespace Net::Arp
 				return;
 			}
 
-		if (arpPacket.hardwareType != 1 || arpPacket.protocolType != Ethernet::EtherType::Ipv4 ||
+			if (arpPacket.hardwareType != 1 ||
+				arpPacket.protocolType != Ethernet::EtherType::Ipv4 ||
 				arpPacket.targetIp != Utils::Ipv4Address)
 			{
 				// Might want to disable because of spamminess
@@ -167,4 +171,5 @@ namespace Net::Arp
 		}
 
 		std::unordered_map<uint32_t, Utils::MacAddress> ArpTable;
-} // namespace Net::Arp
+	} // namespace Arp
+} // namespace Net

src/net-arp.h

@@ -4,7 +4,9 @@
 #include "net-ethernet.h"
 #include "net-utils.h"
 
-namespace Net::Arp
+namespace Net
+{
+	namespace Arp
 	{
 		enum Operation
 		{
@@ -64,4 +66,5 @@ namespace Net::Arp
 		void SendAnnouncement(const Utils::MacAddress mac, const uint32_t ip);
 
 		extern std::unordered_map<uint32_t, Utils::MacAddress> ArpTable;
-} // namespace Net::Arp
+	} // namespace Arp
+} // namespace Net

src/net-dhcp.cpp

@@ -11,7 +11,9 @@
 #include "types.h"
 #include <uspi.h>
 
-namespace Net::Dhcp
+namespace Net
+{
+	namespace Dhcp
 	{
 		Header::Header() {}
 
@@ -98,9 +100,11 @@ namespace Net::Dhcp
 			out.transactionId = buffer[4] << 24 | buffer[5] << 16 | buffer[6] << 8 | buffer[7];
 			out.secondsElapsed = buffer[8] << 8 | buffer[9];
 			out.flags = buffer[10] << 8 | buffer[11];
-		out.clientIpAddress = buffer[12] << 24 | buffer[13] << 16 | buffer[14] << 8 | buffer[15];
+			out.clientIpAddress =
+				buffer[12] << 24 | buffer[13] << 16 | buffer[14] << 8 | buffer[15];
 			out.yourIpAddress = buffer[16] << 24 | buffer[17] << 16 | buffer[18] << 8 | buffer[19];
-		out.serverIpAddress = buffer[20] << 24 | buffer[21] << 16 | buffer[22] << 8 | buffer[23];
+			out.serverIpAddress =
+				buffer[20] << 24 | buffer[21] << 16 | buffer[22] << 8 | buffer[23];
 			out.relayIpAddress = buffer[24] << 24 | buffer[25] << 16 | buffer[26] << 8 | buffer[27];
 
 			std::memcpy(
@@ -185,7 +189,8 @@ namespace Net::Dhcp
 
 			size_t ipv4Length = udpLength + Ipv4::Header::SerializedLength();
 			const Ipv4::Header ipv4Header(Ipv4::Protocol::Udp, 0, 0xFFFFFFFF, ipv4Length);
-		const Ethernet::Header ethernetHeader(Utils::GetMacAddress(), Ethernet::EtherType::Ipv4);
+			const Ethernet::Header ethernetHeader(
+				Utils::GetMacAddress(), Ethernet::EtherType::Ipv4);
 
 			uint8_t buffer[USPI_FRAME_BUFFER_SIZE];
 			size_t size = 0;
@@ -209,11 +214,12 @@ namespace Net::Dhcp
 			sendDiscover();
 
 			// Wait three seconds for responses
-		const auto callbackVoid = static_cast<void*>(&callback);
+			// const auto callbackVoid = static_cast<void*>(&callback);
-		StartKernelTimer(3 * HZ, discoverTimerHandler, callbackVoid, nullptr);
+			// StartKernelTimer(3 * HZ, discoverTimerHandler, callbackVoid, nullptr);
 		}
 
-	static void handleOfferPacket(const Ethernet::Header ethernetHeader, const Header dhcpHeader)
+		static void
+		handleOfferPacket(const Ethernet::Header ethernetHeader, const Header dhcpHeader)
 		{
 			offeredIpAddresses.push_back(dhcpHeader.yourIpAddress);
 			serverIpAddresses.push_back(dhcpHeader.serverIpAddress);
@@ -233,7 +239,8 @@ namespace Net::Dhcp
 			serverSelected = false;
 		}
 
-	void HandlePacket(const Ethernet::Header& ethernetHeader, const uint8_t* buffer, size_t size)
+		void
+		HandlePacket(const Ethernet::Header& ethernetHeader, const uint8_t* buffer, size_t size)
 		{
 			Header header;
 			const auto dhcpSize = Header::Deserialize(header, buffer, size);
@@ -264,4 +271,5 @@ namespace Net::Dhcp
 				handleAckPacket(ethernetHeader, header);
 			}
 		}
-} // namespace Net::Dhcp
+	} // namespace Dhcp
+} // namespace Net

src/net-dhcp.h

@@ -4,7 +4,9 @@
 #include "net-ethernet.h"
 #include "net.h"
 
-namespace Net::Dhcp
+namespace Net
+{
+	namespace Dhcp
 	{
 		enum class Opcode : uint8_t
 		{
@@ -67,11 +69,12 @@ namespace Net::Dhcp
 
 			constexpr static size_t SerializedLength()
 			{
-			return sizeof(Opcode) + sizeof(hardwareAddressType) + sizeof(hardwareAddressLength) +
+				return sizeof(Opcode) + sizeof(hardwareAddressType) +
-				sizeof(hops) + sizeof(transactionId) + sizeof(secondsElapsed) + sizeof(flags) +
+					sizeof(hardwareAddressLength) + sizeof(hops) + sizeof(transactionId) +
-				sizeof(clientIpAddress) + sizeof(yourIpAddress) + sizeof(serverIpAddress) +
+					sizeof(secondsElapsed) + sizeof(flags) + sizeof(clientIpAddress) +
-				sizeof(relayIpAddress) + sizeof(clientHardwareAddress) + sizeof(serverHostname) +
+					sizeof(yourIpAddress) + sizeof(serverIpAddress) + sizeof(relayIpAddress) +
-				sizeof(bootFile) + sizeof(magicValue);
+					sizeof(clientHardwareAddress) + sizeof(serverHostname) + sizeof(bootFile) +
+					sizeof(magicValue);
 			}
 
 			size_t Serialize(uint8_t* buffer, const size_t size) const;
@@ -79,5 +82,7 @@ namespace Net::Dhcp
 		};
 
 		void ObtainIp(std::function<void()>& callback);
-	void HandlePacket(const Ethernet::Header& ethernetHeader, const uint8_t* buffer, size_t size);
+		void
-} // namespace Net::Dhcp
+		HandlePacket(const Ethernet::Header& ethernetHeader, const uint8_t* buffer, size_t size);
+	} // namespace Dhcp
+} // namespace Net

src/net-ethernet.cpp

@@ -2,7 +2,9 @@
 
 #include "net-ethernet.h"
 
-namespace Net::Ethernet
+namespace Net
+{
+	namespace Ethernet
 	{
 		Header::Header() {}
 
@@ -54,4 +56,5 @@ namespace Net::Ethernet
 			out.type = static_cast<EtherType>(buffer[12] << 8 | buffer[13]);
 			return 14;
 		}
-} // namespace Net::Ethernet
+	} // namespace Ethernet
+} // namespace Net

src/net-ethernet.h

@@ -3,7 +3,9 @@
 
 #include "net-utils.h"
 
-namespace Net::Ethernet
+namespace Net
+{
+	namespace Ethernet
 	{
 		using Utils::MacAddress;
 
@@ -32,4 +34,5 @@ namespace Net::Ethernet
 			size_t Serialize(uint8_t* buffer, const size_t size) const;
 			static size_t Deserialize(Header& out, const uint8_t* buffer, const size_t size);
 		};
-} // namespace Net::Ethernet
+	} // namespace Ethernet
+} // namespace Net

src/net-icmp.cpp

@@ -9,7 +9,9 @@
 #include "types.h"
 #include <uspi.h>
 
-namespace Net::Icmp
+namespace Net
+{
+	namespace Icmp
 	{
 		//
 		// Header
@@ -18,7 +20,10 @@ namespace Net::Icmp
 		Header::Header(Type type, uint8_t code) : type(type), code(code), checksum(0) {}
 
 		size_t Header::Serialize(
-		uint8_t* buffer, const size_t bufferSize, const uint8_t* data, const size_t dataSize) const
+			uint8_t* buffer,
+			const size_t bufferSize,
+			const uint8_t* data,
+			const size_t dataSize) const
 		{
 			if (bufferSize < SerializedLength())
 			{
@@ -78,7 +83,8 @@ namespace Net::Icmp
 			return i;
 		}
 
-	size_t EchoHeader::Deserialize(EchoHeader& out, const uint8_t* buffer, const size_t bufferSize)
+		size_t
+		EchoHeader::Deserialize(EchoHeader& out, const uint8_t* buffer, const size_t bufferSize)
 		{
 			if (bufferSize < SerializedLength())
 			{
@@ -146,8 +152,8 @@ namespace Net::Icmp
 			size_t respSize = 0;
 			respSize += respEthernetHeader.Serialize(
 				respBuffer.data() + respSize, respBuffer.size() - respSize);
-		respSize +=
+			respSize += respIpv4Header.Serialize(
-			respIpv4Header.Serialize(respBuffer.data() + respSize, respBuffer.size() - respSize);
+				respBuffer.data() + respSize, respBuffer.size() - respSize);
 			respSize += respIcmpHeader.Serialize(
 				respBuffer.data() + respSize, respBuffer.size() - respSize, reqBuffer, payloadSize);
 
@@ -187,4 +193,5 @@ namespace Net::Icmp
 					bufferSize - headerSize);
 			}
 		}
-} // namespace Net::Icmp
+	} // namespace Icmp
+} // namespace Net

src/net-ipv4.cpp

@@ -13,7 +13,9 @@
 #include <cstring>
 #include <uspi.h>
 
-namespace Net::Ipv4
+namespace Net
+{
+	namespace Ipv4
 	{
 		Header::Header() {}
 
@@ -137,7 +139,8 @@ namespace Net::Ipv4
 			if (header.version != 4)
 			{
 				DEBUG_LOG(
-				"Dropped IPv4 packet (invalid header version %u, expected 4)\r\n", header.version);
+					"Dropped IPv4 packet (invalid header version %u, expected 4)\r\n",
+					header.version);
 				return;
 			}
 			if (header.ihl != 5)
@@ -171,7 +174,9 @@ namespace Net::Ipv4
 			else if (header.protocol == Ipv4::Protocol::Udp)
 			{
 				DEBUG_LOG("Ethernet -> IPv4 -> UDP\r\n");
-			Udp::HandlePacket(ethernetHeader, header, buffer + headerSize, bufferSize - headerSize);
+				Udp::HandlePacket(
+					ethernetHeader, header, buffer + headerSize, bufferSize - headerSize);
 			}
 		}
-} // namespace Net::Ipv4
+	} // namespace Ipv4
+} // namespace Net

src/net-ipv4.h

@@ -4,7 +4,9 @@
 
 #include "net-ethernet.h"
 
-namespace Net::Ipv4
+namespace Net
+{
+	namespace Ipv4
 	{
 		enum class Protocol : uint8_t
 		{
@@ -30,7 +32,8 @@ namespace Net::Ipv4
 			uint32_t destinationIp;
 
 			Header();
-		Header(Protocol protocol, uint32_t sourceIp, uint32_t destinationIp, uint16_t totalLength);
+			Header(
+				Protocol protocol, uint32_t sourceIp, uint32_t destinationIp, uint16_t totalLength);
 
 			static constexpr size_t SerializedLength()
 			{
@@ -44,4 +47,5 @@ namespace Net::Ipv4
 
 		void HandlePacket(
 			const Ethernet::Header& ethernetHeader, const uint8_t* buffer, const size_t bufferSize);
-} // namespace Net::Ipv4
+	} // namespace Ipv4
+} // namespace Net

src/net-tftp.cpp

@@ -14,7 +14,9 @@
 #include "types.h"
 #include <uspi.h>
 
-namespace Net::Tftp
+namespace Net
+{
+	namespace Tftp
 	{
 		// TODO Allow multiple files open
 		static FIL outFile;
@@ -234,7 +236,8 @@ namespace Net::Tftp
 
 		size_t WriteReadRequestPacket::Deserialize(const uint8_t* buffer, const size_t bufferSize)
 		{
-		// Can't use SerializedLength here, as it's variable. Check each field separately instead.
+			// Can't use SerializedLength here, as it's variable. Check each field separately
+			// instead.
 			size_t i = 0;
 
 			if (sizeof(Opcode) >= bufferSize - i)
@@ -370,4 +373,5 @@ namespace Net::Tftp
 			data = std::vector<uint8_t>(buffer + 4, buffer + bufferSize);
 			return bufferSize;
 		}
-} // namespace Net::Tftp
+	} // namespace Tftp
+} // namespace Net

src/net-udp.cpp

@@ -4,7 +4,9 @@
 
 #include "debug.h"
 
-namespace Net::Udp
+namespace Net
+{
+	namespace Udp
 	{
 		Header::Header() {}
 
@@ -95,4 +97,5 @@ namespace Net::Udp
 					bufferSize - udpHeader.SerializedLength());
 			}
 		}
-} // namespace Net::Udp
+	} // namespace Udp
+} // namespace Net

src/net-udp.h

@@ -6,7 +6,9 @@
 #include "net-ethernet.h"
 #include "net-ipv4.h"
 
-namespace Net::Udp
+namespace Net
+{
+	namespace Udp
 	{
 		enum class Port : uint16_t
 		{
@@ -27,7 +29,8 @@ namespace Net::Udp
 
 			static constexpr size_t SerializedLength()
 			{
-			return sizeof(sourcePort) + sizeof(destinationPort) + sizeof(length) + sizeof(checksum);
+				return sizeof(sourcePort) + sizeof(destinationPort) + sizeof(length) +
+					sizeof(checksum);
 			}
 
 			size_t Serialize(uint8_t* buffer, const size_t size) const;
@@ -39,4 +42,5 @@ namespace Net::Udp
 			const Ipv4::Header ipv4Header,
 			const uint8_t* buffer,
 			const size_t size);
-} // namespace Net::Udp
+	} // namespace Udp
+} // namespace Net

src/net-utils.cpp

@@ -3,7 +3,9 @@
 #include "types.h"
 #include <uspi.h>
 
-namespace Net::Utils
+namespace Net
+{
+	namespace Utils
 	{
 		const MacAddress MacBroadcast{0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF};
 		uint32_t Ipv4Address = 0xC0A80164;
@@ -96,4 +98,5 @@ namespace Net::Utils
 
 			return macAddress;
 		}
-} // namespace Net::Utils
+	} // namespace Utils
+} // namespace Net

src/net-utils.h

@@ -2,7 +2,9 @@
 #include <cstddef>
 #include <cstdint>
 
-namespace Net::Utils
+namespace Net
+{
+	namespace Utils
 	{
 		typedef std::array<uint8_t, 6> MacAddress;
 		extern const MacAddress MacBroadcast;
@@ -11,4 +13,5 @@ namespace Net::Utils
 		uint32_t Crc32(const uint8_t* buffer, size_t size);
 		uint16_t InternetChecksum(const void* data, size_t size);
 		MacAddress GetMacAddress();
-}; // namespace Net::Utils
+	} // namespace Utils
+} // namespace Net

src/net.cpp

@@ -58,7 +58,6 @@ namespace Net
 		}
 	}
 
-
 	void Update()
 	{
 		if (postInitializeTime && read32(ARM_SYSTIMER_CLO) > postInitializeTime)