reomve private module fragments

modules/app/application.cppm

@@ -43,7 +43,8 @@ private:
 
 } // namespace lt::app
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::app {
 
 void Application::game_loop()

modules/input/system.cppm

@@ -51,7 +51,8 @@ private:
 } // namespace lt::input
 
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::input {
 
 template<class... Ts>

modules/renderer/factory.cppm

@@ -60,7 +60,8 @@ export namespace lt::renderer {
 
 } // namespace lt::renderer
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer {
 
 [[nodiscard]] auto get_instance(Api target_api) -> IInstance *

modules/renderer/system.cppm

@@ -115,7 +115,8 @@ private:
 
 } // namespace lt::renderer
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer {
 
 System::System(CreateInfo info)

modules/renderer/vk/api_wrapper.cppm

@@ -2801,7 +2801,8 @@ void Device::name(T &object, const char *name)
 } // namespace lt::renderer::vk
 
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer::vk {
 
 namespace api {

modules/renderer/vk/buffer.cppm

@@ -56,7 +56,8 @@ private:
 
 } // namespace lt::renderer::vkb
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer::vkb {
 
 Buffer::Buffer(IDevice *device, IGpu *gpu, const CreateInfo &info)

modules/renderer/vk/debugger.cppm

@@ -35,7 +35,8 @@ private:
 
 } // namespace lt::renderer::vkb
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer::vkb {
 
 [[nodiscard]] auto to_native_severity(IDebugger::MessageSeverity severity) -> vk::Flags

modules/renderer/vk/device.cppm

@@ -63,7 +63,8 @@ private:
 } // namespace lt::renderer::vkb
 
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer::vkb {
 
 Device::Device(IGpu *gpu, ISurface *surface)

modules/renderer/vk/gpu.cppm

@@ -29,7 +29,8 @@ private:
 } // namespace lt::renderer::vkb
 
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer::vkb {
 
 Gpu::Gpu(IInstance *instance)

modules/renderer/vk/instance.cppm

@@ -42,7 +42,8 @@ private:
 
 } // namespace lt::renderer::vkb
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer::vkb {
 
 Instance::Instance()

modules/renderer/vk/pass.cppm

@@ -52,7 +52,8 @@ private:
 
 } // namespace lt::renderer::vkb
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer::vkb {
 
 using enum vk::DescriptorSetLayout::Binding::FlagBits;

modules/renderer/vk/renderer.cppm

@@ -102,7 +102,8 @@ private:
 
 } // namespace lt::renderer::vkb
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer::vkb {
 
 Renderer::Renderer(IGpu *gpu, IDevice *device, ISwapchain *swapchain, u32 max_frames_in_flight)

modules/renderer/vk/surface.cppm

@@ -32,7 +32,8 @@ private:
 
 } // namespace lt::renderer::vkb
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer::vkb {
 
 Surface::Surface(IInstance *instance, const ecs::Entity &surface_entity)

modules/renderer/vk/swapchain.cppm

@@ -73,7 +73,8 @@ private:
 
 } // namespace lt::renderer::vkb
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::renderer::vkb {
 
 Swapchain::Swapchain(ISurface *surface, IGpu *gpu, IDevice *device)

modules/sandbox/sandbox.cpp

@@ -106,6 +106,13 @@ try
 			    lt::log::info("Closing due to: Escape key pressed");
 			    should_close = true;
 		    }
+
+		    if (event.get_key() == lt::Key::q)
+		    {
+			    lt::log::info("Closing due to: Q key pressed");
+			    should_close = true;
+		    }
+
 		    lt::log::info("Key pressed: {}", event.get_key());
 		},
 		[&](const lt::surface::KeyReleasedEvent &event) {
@@ -116,8 +123,22 @@ try
 		}
 	};
 
+	auto timer = lt::time::Timer {};
 	while (!should_close)
 	{
+		if (timer.elapsed_time() > std::chrono::milliseconds { 250 })
+		{
+			const auto [width, height] = window.get_resolution();
+			window.push_request(
+			    lt::surface::ModifyResolutionRequest {
+			        .resolution = { width + 50u, height + 50u },
+			    }
+			);
+
+			lt::log::trace("mrrp");
+			timer.reset();
+		}
+
 		surface_system.tick({});
 		renderer_system.tick({});
 

modules/surface/system.cppm

@@ -202,7 +202,8 @@ private:
 
 } // namespace lt::surface
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::surface {
 
 

modules/test/registry.cppm

@@ -112,7 +112,8 @@ private:
 //////////////////////////////////////
 // -------* IMPLEMENTATION *------- //
 /////////////////////////////////////
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::test {
 
 /* static */ void Registry::register_suite(SuiteFunction suite)

modules/test/test.cppm

@@ -134,7 +134,8 @@ auto operator""_suite(const char *name, size_t size) -> TestSuite
 //////////////////////////////////////
 // -------* IMPLEMENTATION *------- //
 /////////////////////////////////////
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::test {
 
 TestCase::TestCase(std::string name): m_name(name)

modules/time/timer.cppm

@@ -26,7 +26,8 @@ private:
 
 } // namespace lt::time
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::time {
 
 Timer::Timer(Timepoint start): m_start(start)

modules/tracer/tracer.cppm

@@ -78,7 +78,8 @@ private:
 #define lt_trace_begin_session(outputPath) ::lt::Instrumentor::begin_session(outputPath)
 #define lt_trace_end_session()             ::lt::Instrumentor::end_session()
 
-module :private;
+/** @todo(Light): unimplemented in gcc -- is it even right to use a private fragment? */
+// module :private;
 namespace lt::tracer {
 
 void Tracer::begin_session_impl(const std::string &outputPath)