From 3d97f7c60536bf14f314cc703883f74345aacd21 Mon Sep 17 00:00:00 2001
From: heavydemon21 <nielsstunnebrink1@gmail.com>
Date: Mon, 11 Nov 2024 20:49:49 +0100
Subject: rendering particle

---
 src/crepe/facade/SDLContext.h | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

(limited to 'src/crepe/facade/SDLContext.h')

diff --git a/src/crepe/facade/SDLContext.h b/src/crepe/facade/SDLContext.h
index e358c21..7329b74 100644
--- a/src/crepe/facade/SDLContext.h
+++ b/src/crepe/facade/SDLContext.h
@@ -10,6 +10,7 @@
 #include "../api/Sprite.h"
 #include "../api/Transform.h"
 #include "api/Camera.h"
+#include "api/Vector2.h"
 
 // FIXME: this needs to be removed
 const int SCREEN_WIDTH = 640;
@@ -97,14 +98,14 @@ private:
 	 * \param texture Reference to the Texture object.
 	 * \return Width of the texture as an integer.
 	 */
-	int get_width(const Texture &) const;
+	int get_width(const Texture &) ;
 
 	/**
 	 * \brief Gets the height of a texture.
 	 * \param texture Reference to the Texture object.
 	 * \return Height of the texture as an integer.
 	 */
-	int get_height(const Texture &) const;
+	int get_height(const Texture &) ;
 
 private:
 	//! Will use draw,clear_screen, present_screen, camera.
@@ -119,6 +120,8 @@ private:
 	void draw(const Sprite & sprite, const Transform & transform,
 			  const Camera & camera);
 
+	void draw_particle(const Vector2 & pos, const Camera & camera);
+
 	//! Clears the screen, preparing for a new frame.
 	void clear_screen();
 
-- 
cgit v1.2.3


From e25aa67c4263b3a940ae9e92f2fcd31a4486f407 Mon Sep 17 00:00:00 2001
From: heavydemon21 <nielsstunnebrink1@gmail.com>
Date: Thu, 14 Nov 2024 20:18:19 +0100
Subject: adjusted based on feedback of PR#31

---
 src/crepe/facade/SDLContext.cpp   | 33 +++++++++++----------------------
 src/crepe/facade/SDLContext.h     |  9 ++-------
 src/crepe/system/RenderSystem.cpp |  6 +++---
 src/crepe/system/RenderSystem.h   |  6 ++++++
 4 files changed, 22 insertions(+), 32 deletions(-)

(limited to 'src/crepe/facade/SDLContext.h')

diff --git a/src/crepe/facade/SDLContext.cpp b/src/crepe/facade/SDLContext.cpp
index cacf238..febbe4b 100644
--- a/src/crepe/facade/SDLContext.cpp
+++ b/src/crepe/facade/SDLContext.cpp
@@ -7,10 +7,9 @@
 #include <cmath>
 #include <cstddef>
 #include <functional>
-#include <iostream>
 #include <memory>
+#include <stdexcept>
 #include <string>
-#include <utility>
 
 #include "../api/Sprite.h"
 #include "../api/Texture.h"
@@ -32,31 +31,23 @@ SDLContext::SDLContext() {
 	// FIXME: read window defaults from config manager
 
 	if (SDL_Init(SDL_INIT_VIDEO) < 0) {
-		// FIXME: throw exception
-		std::cerr << "SDL could not initialize! SDL_Error: " << SDL_GetError()
-				  << std::endl;
-		return;
+		throw std::runtime_error("SDL could not initialize!");
 	}
+
 	SDL_Window * tmp_window = SDL_CreateWindow(
 		"Crepe Game Engine", SDL_WINDOWPOS_CENTERED, SDL_WINDOWPOS_CENTERED,
 		this->viewport.w, this->viewport.h, 0);
 	if (!tmp_window) {
-		// FIXME: throw exception
-		std::cerr << "Window could not be created! SDL_Error: "
-				  << SDL_GetError() << std::endl;
-		return;
+		throw std::runtime_error("Window could not be created!");
 	}
+
 	this->game_window
 		= {tmp_window, [](SDL_Window * window) { SDL_DestroyWindow(window); }};
 
 	SDL_Renderer * tmp_renderer = SDL_CreateRenderer(
 		this->game_window.get(), -1, SDL_RENDERER_ACCELERATED);
 	if (!tmp_renderer) {
-		// FIXME: throw exception
-		std::cerr << "Renderer could not be created! SDL_Error: "
-				  << SDL_GetError() << std::endl;
-		SDL_DestroyWindow(this->game_window.get());
-		return;
+		throw std::runtime_error("Renderer could not be created!");
 	}
 
 	this->game_renderer = {tmp_renderer, [](SDL_Renderer * renderer) {
@@ -65,9 +56,7 @@ SDLContext::SDLContext() {
 
 	int img_flags = IMG_INIT_PNG;
 	if (!(IMG_Init(img_flags) & img_flags)) {
-		// FIXME: throw exception
-		std::cout << "SDL_image could not initialize! SDL_image Error: "
-				  << IMG_GetError() << std::endl;
+		throw std::runtime_error("SDL_image could not initialize!");
 	}
 }
 
@@ -145,8 +134,8 @@ void SDLContext::draw(const Sprite & sprite, const Transform & transform,
 void SDLContext::camera(const Camera & cam) {
 	this->viewport.w = static_cast<int>(cam.aspect_width);
 	this->viewport.h = static_cast<int>(cam.aspect_height);
-	this->viewport.x = static_cast<int>(cam.x) - (SCREEN_WIDTH / 2);
-	this->viewport.y = static_cast<int>(cam.y) - (SCREEN_HEIGHT / 2);
+	this->viewport.x = static_cast<int>(cam.x) - (this->viewport.w / 2);
+	this->viewport.y = static_cast<int>(cam.y) - (this->viewport.h / 2);
 
 	SDL_SetRenderDrawColor(this->game_renderer.get(), cam.bg_color.r,
 						   cam.bg_color.g, cam.bg_color.b, cam.bg_color.a);
@@ -181,12 +170,12 @@ SDLContext::texture_from_path(const std::string & path) {
 
 	return img_texture;
 }
-int SDLContext::get_width(const Texture & ctx) {
+int SDLContext::get_width(const Texture & ctx) const {
 	int w;
 	SDL_QueryTexture(ctx.texture.get(), NULL, NULL, &w, NULL);
 	return w;
 }
-int SDLContext::get_height(const Texture & ctx) {
+int SDLContext::get_height(const Texture & ctx) const {
 	int h;
 	SDL_QueryTexture(ctx.texture.get(), NULL, NULL, NULL, &h);
 	return h;
diff --git a/src/crepe/facade/SDLContext.h b/src/crepe/facade/SDLContext.h
index c4392bd..0614036 100644
--- a/src/crepe/facade/SDLContext.h
+++ b/src/crepe/facade/SDLContext.h
@@ -10,11 +10,6 @@
 #include "../api/Sprite.h"
 #include "../api/Transform.h"
 #include "api/Camera.h"
-#include "api/Vector2.h"
-
-// FIXME: this needs to be removed
-const int SCREEN_WIDTH = 640;
-const int SCREEN_HEIGHT = 480;
 
 namespace crepe {
 
@@ -109,14 +104,14 @@ private:
 	 * \param texture Reference to the Texture object.
 	 * \return Width of the texture as an integer.
 	 */
-	int get_width(const Texture &) ;
+	int get_width(const Texture &) const;
 
 	/**
 	 * \brief Gets the height of a texture.
 	 * \param texture Reference to the Texture object.
 	 * \return Height of the texture as an integer.
 	 */
-	int get_height(const Texture &) ;
+	int get_height(const Texture &) const;
 
 private:
 	//! Will use draw,clear_screen, present_screen, camera.
diff --git a/src/crepe/system/RenderSystem.cpp b/src/crepe/system/RenderSystem.cpp
index 17a2337..6ecd604 100644
--- a/src/crepe/system/RenderSystem.cpp
+++ b/src/crepe/system/RenderSystem.cpp
@@ -85,13 +85,13 @@ void RenderSystem::render() {
 	auto sprites = mgr.get_components_by_type<Sprite>();
 	for (const Sprite & sprite : sprites) {
 		if (!sprite.active) continue;
-		auto transform = mgr.get_components_by_id<Transform>(sprite.game_object_id);
+		auto transform = mgr.get_components_by_id<Transform>(sprite.game_object_id).front().get();
 
-		bool rendered_particles = this->render_particle(sprite, transform[0].get());
+		bool rendered_particles = this->render_particle(sprite, transform);
 
 		if (rendered_particles) continue;
 
-		this->render_normal(sprite, transform[0].get());
+		this->render_normal(sprite, transform);
 	}
 }
 
diff --git a/src/crepe/system/RenderSystem.h b/src/crepe/system/RenderSystem.h
index 6a87eec..3d6286f 100644
--- a/src/crepe/system/RenderSystem.h
+++ b/src/crepe/system/RenderSystem.h
@@ -31,6 +31,12 @@ public:
 	 */
 	void update() override;
 
+
+	RenderSystem(const RenderSystem &) = delete;
+	RenderSystem(RenderSystem &&) = delete;
+	RenderSystem & operator=(const RenderSystem &) = delete;
+	RenderSystem & operator=(RenderSystem &&) = delete;
+
 private:
 	// Private constructor to enforce singleton pattern.
 	RenderSystem();
-- 
cgit v1.2.3


From 66bbea079bf1ae84c355349d337db468c18eac32 Mon Sep 17 00:00:00 2001
From: heavydemon21 <nielsstunnebrink1@gmail.com>
Date: Mon, 18 Nov 2024 21:07:26 +0100
Subject: feedback from PR#38 implemented

---
 src/crepe/facade/SDLContext.h     |  5 -----
 src/crepe/system/RenderSystem.cpp |  5 ++---
 src/crepe/system/RenderSystem.h   | 10 ++++++----
 3 files changed, 8 insertions(+), 12 deletions(-)

(limited to 'src/crepe/facade/SDLContext.h')

diff --git a/src/crepe/facade/SDLContext.h b/src/crepe/facade/SDLContext.h
index 007092b..d437bc4 100644
--- a/src/crepe/facade/SDLContext.h
+++ b/src/crepe/facade/SDLContext.h
@@ -21,8 +21,6 @@ namespace crepe {
 // typedef is unusable when crepe is packaged. Wouter will fix this later.
 typedef SDL_Keycode CREPE_KEYCODES;
 
-class Texture;
-class LoopManager;
 
 /**
  * \class SDLContext
@@ -91,9 +89,6 @@ private:
 	//! Will use the funtions: texture_from_path, get_width,get_height.
 	friend class Texture;
 
-	//! Will use the funtions: texture_from_path, get_width,get_height.
-	friend class Animator;
-
 	/**
 	 * \brief Loads a texture from a file path.
 	 * \param path Path to the image file.
diff --git a/src/crepe/system/RenderSystem.cpp b/src/crepe/system/RenderSystem.cpp
index bf5dade..11c20f5 100644
--- a/src/crepe/system/RenderSystem.cpp
+++ b/src/crepe/system/RenderSystem.cpp
@@ -22,7 +22,7 @@ void RenderSystem::update_camera() {
 
 	for (Camera & cam : cameras) {
 		SDLContext::get_instance().camera(cam);
-		this->curr_cam = &cam;
+		this->curr_cam_ref = &cam;
 	}
 }
 
@@ -35,7 +35,6 @@ bool sorting_comparison(const Sprite & a, const Sprite & b) {
 
 std::vector<std::reference_wrapper<Sprite>>
 RenderSystem::sort(std::vector<std::reference_wrapper<Sprite>> & objs) {
-	if (objs.empty()) return {};
 
 	std::vector<std::reference_wrapper<Sprite>> sorted_objs(objs);
 	std::sort(sorted_objs.begin(), sorted_objs.end(), sorting_comparison);
@@ -52,7 +51,7 @@ void RenderSystem::render_sprites() {
 	SDLContext & render = SDLContext::get_instance();
 	for (const Sprite & sprite : sorted_sprites) {
 		auto transforms = mgr.get_components_by_id<Transform>(sprite.game_object_id);
-		render.draw(sprite, transforms[0], *curr_cam);
+		render.draw(sprite, transforms[0], *this->curr_cam_ref);
 	}
 }
 
diff --git a/src/crepe/system/RenderSystem.h b/src/crepe/system/RenderSystem.h
index 7661cbd..1176b06 100644
--- a/src/crepe/system/RenderSystem.h
+++ b/src/crepe/system/RenderSystem.h
@@ -1,12 +1,14 @@
 #pragma once
 
-#include "api/Camera.h"
 
-#include "System.h"
-#include "api/Sprite.h"
 #include <functional>
 #include <vector>
 
+#include "api/Sprite.h"
+#include "api/Camera.h"
+
+#include "System.h"
+
 namespace crepe {
 
 /**
@@ -54,7 +56,7 @@ private:
 
 private:
 	//! Pointer to the current active camera for rendering
-	Camera * curr_cam = nullptr;
+	Camera * curr_cam_ref = nullptr;
 	// TODO: needs a better solution
 };
 
-- 
cgit v1.2.3


From 52e84ca0a81210cd1dc61e096c228586649c9bfc Mon Sep 17 00:00:00 2001
From: heavydemon21 <nielsstunnebrink1@gmail.com>
Date: Wed, 20 Nov 2024 11:35:25 +0100
Subject: updated adjsuted rendering based on feedback

---
 src/crepe/facade/SDLContext.cpp    | 51 +++++++++++++++++++++++++++++---------
 src/crepe/facade/SDLContext.h      | 26 +++++++++++++++++--
 src/crepe/system/RenderSystem.cpp  |  8 +++---
 src/crepe/system/RenderSystem.h    |  3 ++-
 src/example/rendering_particle.cpp | 11 ++++----
 5 files changed, 74 insertions(+), 25 deletions(-)

(limited to 'src/crepe/facade/SDLContext.h')

diff --git a/src/crepe/facade/SDLContext.cpp b/src/crepe/facade/SDLContext.cpp
index 40189f6..8131df2 100644
--- a/src/crepe/facade/SDLContext.cpp
+++ b/src/crepe/facade/SDLContext.cpp
@@ -1,5 +1,6 @@
 #include <SDL2/SDL.h>
 #include <SDL2/SDL_image.h>
+#include <SDL2/SDL_keycode.h>
 #include <SDL2/SDL_rect.h>
 #include <SDL2/SDL_render.h>
 #include <SDL2/SDL_surface.h>
@@ -15,6 +16,8 @@
 #include "../api/Texture.h"
 #include "../api/Transform.h"
 #include "../util/Log.h"
+#include "api/Camera.h"
+#include "api/Vector2.h"
 
 #include "SDLContext.h"
 
@@ -93,30 +96,54 @@ void SDLContext::handle_events(bool & running) {
 void SDLContext::clear_screen() { SDL_RenderClear(this->game_renderer.get()); }
 void SDLContext::present_screen() { SDL_RenderPresent(this->game_renderer.get()); }
 
-void SDLContext::draw(const Sprite & sprite, const Transform & transform, const Camera & cam) {
-
-	SDL_RendererFlip render_flip
-		= (SDL_RendererFlip) ((SDL_FLIP_HORIZONTAL * sprite.flip.flip_x)
-							  | (SDL_FLIP_VERTICAL * sprite.flip.flip_y));
-
-	double adjusted_x = (transform.position.x - cam.x) * cam.zoom;
-	double adjusted_y = (transform.position.y - cam.y) * cam.zoom;
-	double adjusted_w = sprite.sprite_rect.w * transform.scale * cam.zoom;
-	double adjusted_h = sprite.sprite_rect.h * transform.scale * cam.zoom;
 
-	SDL_Rect srcrect = {
+SDL_Rect SDLContext::get_src_rect(const Sprite & sprite) {
+	return SDL_Rect{
 		.x = sprite.sprite_rect.x,
 		.y = sprite.sprite_rect.y,
 		.w = sprite.sprite_rect.w,
 		.h = sprite.sprite_rect.h,
 	};
+}
+SDL_Rect SDLContext::get_dst_rect(const Sprite & sprite, const Vector2 & pos,
+								  const double & scale, const Camera & cam) {
+
+	double adjusted_x = (pos.x - cam.x) * cam.zoom;
+	double adjusted_y = (pos.y - cam.y) * cam.zoom;
+	double adjusted_w = sprite.sprite_rect.w * scale * cam.zoom;
+	double adjusted_h = sprite.sprite_rect.h * scale * cam.zoom;
 
-	SDL_Rect dstrect = {
+	return SDL_Rect{
 		.x = static_cast<int>(adjusted_x),
 		.y = static_cast<int>(adjusted_y),
 		.w = static_cast<int>(adjusted_w),
 		.h = static_cast<int>(adjusted_h),
 	};
+}
+
+void SDLContext::draw_particle(const Sprite & sprite, const Vector2 & pos,
+							   const double & angle, const double & scale,
+							   const Camera & camera) {
+
+	SDL_RendererFlip render_flip
+		= (SDL_RendererFlip) ((SDL_FLIP_HORIZONTAL * sprite.flip.flip_x)
+						| (SDL_FLIP_VERTICAL * sprite.flip.flip_y));
+
+	SDL_Rect srcrect = this->get_src_rect(sprite);
+	SDL_Rect dstrect = this->get_dst_rect(sprite, pos, scale, camera);
+
+	SDL_RenderCopyEx(this->game_renderer.get(), sprite.sprite_image->texture.get(), &srcrect,
+				  &dstrect, angle, NULL, render_flip);
+}
+
+void SDLContext::draw(const Sprite & sprite, const Transform & transform, const Camera & cam) {
+
+	SDL_RendererFlip render_flip
+		= (SDL_RendererFlip) ((SDL_FLIP_HORIZONTAL * sprite.flip.flip_x)
+							  | (SDL_FLIP_VERTICAL * sprite.flip.flip_y));
+
+	SDL_Rect srcrect = this->get_src_rect(sprite);
+	SDL_Rect dstrect = this->get_dst_rect(sprite, transform.position, transform.scale, cam);
 
 	SDL_RenderCopyEx(this->game_renderer.get(), sprite.sprite_image->texture.get(), &srcrect,
 					 &dstrect, transform.rotation, NULL, render_flip);
diff --git a/src/crepe/facade/SDLContext.h b/src/crepe/facade/SDLContext.h
index 78ac56b..fb09015 100644
--- a/src/crepe/facade/SDLContext.h
+++ b/src/crepe/facade/SDLContext.h
@@ -1,8 +1,10 @@
 #pragma once
 
 #include <SDL2/SDL_keycode.h>
+#include <SDL2/SDL_rect.h>
 #include <SDL2/SDL_render.h>
 #include <SDL2/SDL_video.h>
+#include <cmath>
 #include <functional>
 #include <memory>
 #include <string>
@@ -10,6 +12,7 @@
 #include "../api/Sprite.h"
 #include "../api/Transform.h"
 #include "api/Camera.h"
+#include "api/Vector2.h"
 
 namespace crepe {
 
@@ -122,8 +125,7 @@ private:
 	 * \param camera Reference to the Camera for view adjustments.
 	 */
 	void draw(const Sprite & sprite, const Transform & transform, const Camera & camera);
-
-	void draw_particle(const Vector2 & pos, const Camera & camera);
+	void draw_particle(const Sprite & sprite, const Vector2 & pos, const double & angle, const double & scale, const Camera & camera);
 
 	//! Clears the screen, preparing for a new frame.
 	void clear_screen();
@@ -137,6 +139,26 @@ private:
 	 */
 	void camera(const Camera & camera);
 
+private:
+	/**
+	 * \brief calculates the sqaure size of the image
+	 *
+	 * \param sprite Reference to the sprite to calculate the rectangle
+	 * \return sdl rectangle to draw a src image
+	 */
+	SDL_Rect get_src_rect(const Sprite & sprite);
+	/**
+	 * \brief calculates the sqaure size of the image for an destination
+	 *
+	 * \param sprite Reference to the sprite to calculate the rectangle
+	 * \param pos the pos in pixel positions
+	 * \param scale the multiplier to increase of decrease for the specified sprite 
+	 * \param cam Reference to the current camera in the scene to calculate the position based
+	 * on the camera 
+	 * \return sdl rectangle to draw a dst image to draw on the screen
+	 */
+	SDL_Rect get_dst_rect(const Sprite & sprite, const Vector2 & pos, const double & scale, const Camera & cam);
+
 private:
 	//! sdl Window
 	std::unique_ptr<SDL_Window, std::function<void(SDL_Window *)>> game_window;
diff --git a/src/crepe/system/RenderSystem.cpp b/src/crepe/system/RenderSystem.cpp
index 39e0e3f..1ae5ca7 100644
--- a/src/crepe/system/RenderSystem.cpp
+++ b/src/crepe/system/RenderSystem.cpp
@@ -27,7 +27,7 @@ void RenderSystem::update_camera() {
 }
 
 bool RenderSystem::render_particle(const Sprite & sprite,
-								   Transform tm) {
+								   const double & scale) {
 
 	ComponentManager & mgr = this->component_manager;
 	SDLContext & render = SDLContext::get_instance();
@@ -44,9 +44,7 @@ bool RenderSystem::render_particle(const Sprite & sprite,
 
 		for (const Particle & p : em.data.particles) {
 			if (!p.active) continue;
-			tm.position = p.position;
-			tm.rotation = p.angle;
-			render.draw(em.data.sprite, tm, *curr_cam);
+			render.draw_particle(sprite, p.position, p.angle, scale, *this->curr_cam);
 		}
 	}
 	return rendering_particles;
@@ -68,7 +66,7 @@ void RenderSystem::render() {
 		if (!sprite.active) continue;
 		auto transform = mgr.get_components_by_id<Transform>(sprite.game_object_id).front().get();
 
-		bool rendered_particles = this->render_particle(sprite, transform);
+		bool rendered_particles = this->render_particle(sprite, transform.scale);
 
 		if (rendered_particles) continue;
 
diff --git a/src/crepe/system/RenderSystem.h b/src/crepe/system/RenderSystem.h
index c18b80b..6643084 100644
--- a/src/crepe/system/RenderSystem.h
+++ b/src/crepe/system/RenderSystem.h
@@ -5,6 +5,7 @@
 #include "api/Transform.h"
 
 #include "System.h"
+#include <cmath>
 
 namespace crepe {
 
@@ -51,7 +52,7 @@ private:
 	 * \param tm the Transform component for scale
 	 * \return true if particles have been rendered
 	 */
-	bool render_particle(const Sprite &, Transform tm);
+	bool render_particle(const Sprite &, const double & scale);
 
 	/**
 	 * \brief renders a sprite with a Transform component on the screen 
diff --git a/src/example/rendering_particle.cpp b/src/example/rendering_particle.cpp
index 33013ef..71b50ba 100644
--- a/src/example/rendering_particle.cpp
+++ b/src/example/rendering_particle.cpp
@@ -10,7 +10,6 @@
 #include <crepe/api/Texture.h>
 #include <crepe/api/Transform.h>
 #include <crepe/system/RenderSystem.h>
-#include <crepe/util/log.h>
 #include <crepe/api/Color.h>
 #include <crepe/api/Sprite.h>
 #include <crepe/api/Vector2.h>
@@ -21,7 +20,11 @@ using namespace crepe;
 using namespace std;
 
 int main(int argc, char * argv[]) {
-	GameObject game_object(0, "", "", Vector2{100, 100}, 0, 0.1);
+	ComponentManager mgr;	
+	GameObject game_object = mgr.new_object("", "", Vector2{100, 100}, 0, 0.1);
+	RenderSystem sys{mgr};
+	ParticleSystem psys{mgr};
+
 	Color color(255, 255, 255, 255);
 	Sprite test_sprite = game_object.add_component<Sprite>(
 		make_shared<Texture>("../asset/texture/img.png"), color,
@@ -47,11 +50,9 @@ int main(int argc, char * argv[]) {
 	});
 	game_object.add_component<Camera>(Color::get_white());
 
-	auto & sys = crepe::RenderSystem::get_instance();
-	auto sys_part = crepe::ParticleSystem();
 	auto start = std::chrono::steady_clock::now();
 	while (std::chrono::steady_clock::now() - start < std::chrono::seconds(5)) {
-		sys_part.update();
+		psys.update();
 		sys.update();
 		SDL_Delay(10 );
 	}
-- 
cgit v1.2.3


From 5121e95e5ba98ca5cadfb63b7603731dfb41a322 Mon Sep 17 00:00:00 2001
From: heavydemon21 <nielsstunnebrink1@gmail.com>
Date: Wed, 20 Nov 2024 11:53:17 +0100
Subject: implemented feedback

---
 makefile                          |  2 +-
 mwe/events/include/event.h        |  2 +-
 src/crepe/api/Sprite.h            |  3 ++-
 src/crepe/facade/SDLContext.h     |  1 -
 src/crepe/system/RenderSystem.cpp | 15 +++++++--------
 src/crepe/system/RenderSystem.h   | 20 +++++++++++---------
 6 files changed, 22 insertions(+), 21 deletions(-)

(limited to 'src/crepe/facade/SDLContext.h')

diff --git a/makefile b/makefile
index dd7c587..32e68c0 100644
--- a/makefile
+++ b/makefile
@@ -6,5 +6,5 @@ doxygen: Doxyfile FORCE
 FMT += $(shell git ls-files '*.c' '*.cpp' '*.h' '*.hpp')
 format: FORCE
 	clang-format -i $(FMT)
-	$(MAKE) -C src $@
+	#$(MAKE) -C src $@
 
diff --git a/mwe/events/include/event.h b/mwe/events/include/event.h
index ee1bf52..e1b220b 100644
--- a/mwe/events/include/event.h
+++ b/mwe/events/include/event.h
@@ -148,7 +148,7 @@ private:
 };
 class ShutDownEvent : public Event {
 public:
-	ShutDownEvent() : Event("ShutDownEvent") {};
+	ShutDownEvent() : Event("ShutDownEvent"){};
 
 	REGISTER_EVENT_TYPE(ShutDownEvent)
 
diff --git a/src/crepe/api/Sprite.h b/src/crepe/api/Sprite.h
index 0192793..74a55d4 100644
--- a/src/crepe/api/Sprite.h
+++ b/src/crepe/api/Sprite.h
@@ -2,8 +2,9 @@
 
 #include <memory>
 
+#include "../Component.h"
+
 #include "Color.h"
-#include "Component.h"
 #include "Texture.h"
 
 namespace crepe {
diff --git a/src/crepe/facade/SDLContext.h b/src/crepe/facade/SDLContext.h
index d437bc4..652a83e 100644
--- a/src/crepe/facade/SDLContext.h
+++ b/src/crepe/facade/SDLContext.h
@@ -21,7 +21,6 @@ namespace crepe {
 // typedef is unusable when crepe is packaged. Wouter will fix this later.
 typedef SDL_Keycode CREPE_KEYCODES;
 
-
 /**
  * \class SDLContext
  * \brief Facade for the SDL library
diff --git a/src/crepe/system/RenderSystem.cpp b/src/crepe/system/RenderSystem.cpp
index 82257c4..96c5f27 100644
--- a/src/crepe/system/RenderSystem.cpp
+++ b/src/crepe/system/RenderSystem.cpp
@@ -12,10 +12,11 @@
 #include "RenderSystem.h"
 
 using namespace crepe;
+using namespace std;
 
-void RenderSystem::clear_screen() const { SDLContext::get_instance().clear_screen(); }
+void RenderSystem::clear_screen() { this->context.clear_screen(); }
 
-void RenderSystem::present_screen() const { SDLContext::get_instance().present_screen(); }
+void RenderSystem::present_screen() { this->context.present_screen(); }
 void RenderSystem::update_camera() {
 	ComponentManager & mgr = this->component_manager;
 
@@ -24,7 +25,7 @@ void RenderSystem::update_camera() {
 	if (cameras.size() == 0) throw std::runtime_error("No cameras in current scene");
 
 	for (Camera & cam : cameras) {
-		SDLContext::get_instance().camera(cam);
+		this->context.camera(cam);
 		this->curr_cam_ref = &cam;
 	}
 }
@@ -47,14 +48,12 @@ RenderSystem::sort(std::vector<std::reference_wrapper<Sprite>> & objs) {
 
 void RenderSystem::render_sprites() {
 	ComponentManager & mgr = this->component_manager;
+	vector<reference_wrapper<Sprite>> sprites = mgr.get_components_by_type<Sprite>();
+	vector<reference_wrapper<Sprite>> sorted_sprites = this->sort(sprites);
 
-	auto sprites = mgr.get_components_by_type<Sprite>();
-	auto sorted_sprites = this->sort(sprites);
-
-	SDLContext & render = SDLContext::get_instance();
 	for (const Sprite & sprite : sorted_sprites) {
 		auto transforms = mgr.get_components_by_id<Transform>(sprite.game_object_id);
-		render.draw(sprite, transforms[0], *this->curr_cam_ref);
+		this->context.draw(sprite, transforms[0], *this->curr_cam_ref);
 	}
 }
 
diff --git a/src/crepe/system/RenderSystem.h b/src/crepe/system/RenderSystem.h
index 1176b06..8914b96 100644
--- a/src/crepe/system/RenderSystem.h
+++ b/src/crepe/system/RenderSystem.h
@@ -1,23 +1,23 @@
 #pragma once
 
-
 #include <functional>
 #include <vector>
 
-#include "api/Sprite.h"
-#include "api/Camera.h"
+#include "facade/SDLContext.h"
 
 #include "System.h"
 
 namespace crepe {
 
+class Camera;
+class Sprite;
+
 /**
  * \class RenderSystem
  * \brief Manages rendering operations for all game objects.
  *
- * RenderSystem is responsible for rendering sprites, clearing and presenting the screen, and
- * managing the active camera. It functions as a singleton, providing centralized rendering
- * services for the application.
+ * RenderSystem is responsible for rendering, clearing and presenting the screen, and
+ * managing the active camera. 
  */
 class RenderSystem : public System {
 public:
@@ -30,16 +30,16 @@ public:
 
 private:
 	//! Clears the screen in preparation for rendering.
-	void clear_screen() const;
+	void clear_screen();
 
 	//! Presents the rendered frame to the display.
-	void present_screen() const;
+	void present_screen();
 
 	//! Updates the active camera used for rendering.
 	void update_camera();
 
 	//! Renders all active sprites to the screen.
-	void render_sprites() ;
+	void render_sprites();
 
 	std::vector<std::reference_wrapper<Sprite>>
 	sort(std::vector<std::reference_wrapper<Sprite>> & objs);
@@ -58,6 +58,8 @@ private:
 	//! Pointer to the current active camera for rendering
 	Camera * curr_cam_ref = nullptr;
 	// TODO: needs a better solution
+
+	SDLContext & context = SDLContext::get_instance();
 };
 
 } // namespace crepe
-- 
cgit v1.2.3


From 158022890cb9abd308a6a445588cc2fca8d84e67 Mon Sep 17 00:00:00 2001
From: heavydemon21 <nielsstunnebrink1@gmail.com>
Date: Wed, 20 Nov 2024 13:25:44 +0100
Subject: implementing feedback

---
 src/crepe/facade/SDLContext.cpp | 3 ++-
 src/crepe/facade/SDLContext.h   | 5 +++--
 2 files changed, 5 insertions(+), 3 deletions(-)

(limited to 'src/crepe/facade/SDLContext.h')

diff --git a/src/crepe/facade/SDLContext.cpp b/src/crepe/facade/SDLContext.cpp
index 8131df2..eacb10a 100644
--- a/src/crepe/facade/SDLContext.cpp
+++ b/src/crepe/facade/SDLContext.cpp
@@ -149,7 +149,7 @@ void SDLContext::draw(const Sprite & sprite, const Transform & transform, const
 					 &dstrect, transform.rotation, NULL, render_flip);
 }
 
-void SDLContext::camera(const Camera & cam) {
+void SDLContext::set_camera(const Camera & cam) {
 	this->viewport.w = static_cast<int>(cam.aspect_width);
 	this->viewport.h = static_cast<int>(cam.aspect_height);
 	this->viewport.x = static_cast<int>(cam.x) - (this->viewport.w / 2);
@@ -167,6 +167,7 @@ SDLContext::texture_from_path(const std::string & path) {
 	SDL_Surface * tmp = IMG_Load(path.c_str());
 	if (tmp == nullptr) {
 		tmp = IMG_Load("../asset/texture/ERROR.png");
+		if (tmp == nullptr) throw runtime_error("cannot load image");
 	}
 
 	std::unique_ptr<SDL_Surface, std::function<void(SDL_Surface *)>> img_surface;
diff --git a/src/crepe/facade/SDLContext.h b/src/crepe/facade/SDLContext.h
index fb09015..287ad5d 100644
--- a/src/crepe/facade/SDLContext.h
+++ b/src/crepe/facade/SDLContext.h
@@ -125,6 +125,7 @@ private:
 	 * \param camera Reference to the Camera for view adjustments.
 	 */
 	void draw(const Sprite & sprite, const Transform & transform, const Camera & camera);
+
 	void draw_particle(const Sprite & sprite, const Vector2 & pos, const double & angle, const double & scale, const Camera & camera);
 
 	//! Clears the screen, preparing for a new frame.
@@ -134,10 +135,10 @@ private:
 	void present_screen();
 
 	/**
-	 * \brief Sets the current camera for rendering.
+	 * \brief sets the background of the camera (will be adjusted in future PR)
 	 * \param camera Reference to the Camera object.
 	 */
-	void camera(const Camera & camera);
+	void set_camera(const Camera & camera);
 
 private:
 	/**
-- 
cgit v1.2.3


From 3344e0df01435be903e38ccbd3b9cee608d574e7 Mon Sep 17 00:00:00 2001
From: heavydemon21 <nielsstunnebrink1@gmail.com>
Date: Wed, 20 Nov 2024 14:06:54 +0100
Subject: implemented feedback

---
 src/crepe/facade/SDLContext.cpp    | 18 +++++++++---------
 src/crepe/facade/SDLContext.h      | 10 ++++++----
 src/crepe/system/RenderSystem.cpp  | 18 +++++++++---------
 src/crepe/system/RenderSystem.h    | 14 ++++----------
 src/example/rendering_particle.cpp | 14 ++++++--------
 5 files changed, 34 insertions(+), 40 deletions(-)

(limited to 'src/crepe/facade/SDLContext.h')

diff --git a/src/crepe/facade/SDLContext.cpp b/src/crepe/facade/SDLContext.cpp
index eacb10a..daf0050 100644
--- a/src/crepe/facade/SDLContext.cpp
+++ b/src/crepe/facade/SDLContext.cpp
@@ -12,12 +12,12 @@
 #include <stdexcept>
 #include <string>
 
+#include "../api/Camera.h"
 #include "../api/Sprite.h"
 #include "../api/Texture.h"
 #include "../api/Transform.h"
+#include "../api/Vector2.h"
 #include "../util/Log.h"
-#include "api/Camera.h"
-#include "api/Vector2.h"
 
 #include "SDLContext.h"
 
@@ -96,8 +96,7 @@ void SDLContext::handle_events(bool & running) {
 void SDLContext::clear_screen() { SDL_RenderClear(this->game_renderer.get()); }
 void SDLContext::present_screen() { SDL_RenderPresent(this->game_renderer.get()); }
 
-
-SDL_Rect SDLContext::get_src_rect(const Sprite & sprite) {
+SDL_Rect SDLContext::get_src_rect(const Sprite & sprite) const {
 	return SDL_Rect{
 		.x = sprite.sprite_rect.x,
 		.y = sprite.sprite_rect.y,
@@ -106,7 +105,7 @@ SDL_Rect SDLContext::get_src_rect(const Sprite & sprite) {
 	};
 }
 SDL_Rect SDLContext::get_dst_rect(const Sprite & sprite, const Vector2 & pos,
-								  const double & scale, const Camera & cam) {
+								  const double & scale, const Camera & cam) const {
 
 	double adjusted_x = (pos.x - cam.x) * cam.zoom;
 	double adjusted_y = (pos.y - cam.y) * cam.zoom;
@@ -123,20 +122,21 @@ SDL_Rect SDLContext::get_dst_rect(const Sprite & sprite, const Vector2 & pos,
 
 void SDLContext::draw_particle(const Sprite & sprite, const Vector2 & pos,
 							   const double & angle, const double & scale,
-							   const Camera & camera) {
+							   const Camera & camera) const {
 
 	SDL_RendererFlip render_flip
 		= (SDL_RendererFlip) ((SDL_FLIP_HORIZONTAL * sprite.flip.flip_x)
-						| (SDL_FLIP_VERTICAL * sprite.flip.flip_y));
+							  | (SDL_FLIP_VERTICAL * sprite.flip.flip_y));
 
 	SDL_Rect srcrect = this->get_src_rect(sprite);
 	SDL_Rect dstrect = this->get_dst_rect(sprite, pos, scale, camera);
 
 	SDL_RenderCopyEx(this->game_renderer.get(), sprite.sprite_image->texture.get(), &srcrect,
-				  &dstrect, angle, NULL, render_flip);
+					 &dstrect, angle, NULL, render_flip);
 }
 
-void SDLContext::draw(const Sprite & sprite, const Transform & transform, const Camera & cam) {
+void SDLContext::draw(const Sprite & sprite, const Transform & transform,
+					  const Camera & cam) const {
 
 	SDL_RendererFlip render_flip
 		= (SDL_RendererFlip) ((SDL_FLIP_HORIZONTAL * sprite.flip.flip_x)
diff --git a/src/crepe/facade/SDLContext.h b/src/crepe/facade/SDLContext.h
index 287ad5d..718c40f 100644
--- a/src/crepe/facade/SDLContext.h
+++ b/src/crepe/facade/SDLContext.h
@@ -124,9 +124,10 @@ private:
 	 * \param transform Reference to the Transform for positioning.
 	 * \param camera Reference to the Camera for view adjustments.
 	 */
-	void draw(const Sprite & sprite, const Transform & transform, const Camera & camera);
+	void draw(const Sprite & sprite, const Transform & transform, const Camera & camera) const;
 
-	void draw_particle(const Sprite & sprite, const Vector2 & pos, const double & angle, const double & scale, const Camera & camera);
+	void draw_particle(const Sprite & sprite, const Vector2 & pos, const double & angle,
+					   const double & scale, const Camera & camera) const;
 
 	//! Clears the screen, preparing for a new frame.
 	void clear_screen();
@@ -147,7 +148,7 @@ private:
 	 * \param sprite Reference to the sprite to calculate the rectangle
 	 * \return sdl rectangle to draw a src image
 	 */
-	SDL_Rect get_src_rect(const Sprite & sprite);
+	SDL_Rect get_src_rect(const Sprite & sprite) const;
 	/**
 	 * \brief calculates the sqaure size of the image for an destination
 	 *
@@ -158,7 +159,8 @@ private:
 	 * on the camera 
 	 * \return sdl rectangle to draw a dst image to draw on the screen
 	 */
-	SDL_Rect get_dst_rect(const Sprite & sprite, const Vector2 & pos, const double & scale, const Camera & cam);
+	SDL_Rect get_dst_rect(const Sprite & sprite, const Vector2 & pos, const double & scale,
+						  const Camera & cam) const;
 
 private:
 	//! sdl Window
diff --git a/src/crepe/system/RenderSystem.cpp b/src/crepe/system/RenderSystem.cpp
index 1ae5ca7..28bcf56 100644
--- a/src/crepe/system/RenderSystem.cpp
+++ b/src/crepe/system/RenderSystem.cpp
@@ -2,11 +2,11 @@
 #include <vector>
 
 #include "../ComponentManager.h"
+#include "../api/ParticleEmitter.h"
 #include "../api/Sprite.h"
 #include "../api/Transform.h"
-#include "../facade/SDLContext.h"
-#include "../api/ParticleEmitter.h"
 #include "../api/Vector2.h"
+#include "../facade/SDLContext.h"
 
 #include "RenderSystem.h"
 
@@ -21,13 +21,12 @@ void RenderSystem::update_camera() {
 	auto cameras = mgr.get_components_by_type<Camera>();
 
 	for (Camera & cam : cameras) {
-		SDLContext::get_instance().camera(cam);
+		SDLContext::get_instance().set_camera(cam);
 		this->curr_cam = &cam;
 	}
 }
 
-bool RenderSystem::render_particle(const Sprite & sprite,
-								   const double & scale) {
+bool RenderSystem::render_particle(const Sprite & sprite, const double & scale) const {
 
 	ComponentManager & mgr = this->component_manager;
 	SDLContext & render = SDLContext::get_instance();
@@ -49,22 +48,23 @@ bool RenderSystem::render_particle(const Sprite & sprite,
 	}
 	return rendering_particles;
 }
-void RenderSystem::render_normal(const Sprite & sprite, const Transform & tm) {
+void RenderSystem::render_normal(const Sprite & sprite, const Transform & tm) const {
 
 	ComponentManager & mgr = this->component_manager;
 	SDLContext & render = SDLContext::get_instance();
-	
+
 	render.draw(sprite, tm, *curr_cam);
 }
 
-void RenderSystem::render() {
+void RenderSystem::render() const {
 
 	ComponentManager & mgr = this->component_manager;
 
 	auto sprites = mgr.get_components_by_type<Sprite>();
 	for (const Sprite & sprite : sprites) {
 		if (!sprite.active) continue;
-		auto transform = mgr.get_components_by_id<Transform>(sprite.game_object_id).front().get();
+		const Transform & transform
+			= mgr.get_components_by_id<Transform>(sprite.game_object_id).front().get();
 
 		bool rendered_particles = this->render_particle(sprite, transform.scale);
 
diff --git a/src/crepe/system/RenderSystem.h b/src/crepe/system/RenderSystem.h
index 6643084..8841f72 100644
--- a/src/crepe/system/RenderSystem.h
+++ b/src/crepe/system/RenderSystem.h
@@ -25,13 +25,7 @@ public:
 	 * This method is called to perform all rendering operations for the current game frame.
 	 */
 	void update() override;
-
-
-	RenderSystem(const RenderSystem &) = delete;
-	RenderSystem(RenderSystem &&) = delete;
-	RenderSystem & operator=(const RenderSystem &) = delete;
-	RenderSystem & operator=(RenderSystem &&) = delete;
-
+	
 private:
 	//! Clears the screen in preparation for rendering.
 	void clear_screen() const;
@@ -43,7 +37,7 @@ private:
 	void update_camera();
 
 	//! Renders the whole screen
-	void render();
+	void render() const;
 
 	/**
 	 * \brief Renders all the particles on the screen from a given sprite.
@@ -52,7 +46,7 @@ private:
 	 * \param tm the Transform component for scale
 	 * \return true if particles have been rendered
 	 */
-	bool render_particle(const Sprite &, const double & scale);
+	bool render_particle(const Sprite & sprite, const double & scale) const;
 
 	/**
 	 * \brief renders a sprite with a Transform component on the screen 
@@ -60,7 +54,7 @@ private:
 	 * \param sprite  the sprite component that holds all the data
 	 * \param tm the Transform component that holds the position,rotation and scale 
 	 */
-	void render_normal(const Sprite &, const Transform & tm);
+	void render_normal(const Sprite & sprite, const Transform & tm) const;
 
 	/**
 	 * \todo Include color handling for sprites.
diff --git a/src/example/rendering_particle.cpp b/src/example/rendering_particle.cpp
index 71b50ba..3f71750 100644
--- a/src/example/rendering_particle.cpp
+++ b/src/example/rendering_particle.cpp
@@ -4,15 +4,15 @@
 #include <crepe/ComponentManager.h>
 
 #include <crepe/Component.h>
+#include <crepe/api/Color.h>
 #include <crepe/api/GameObject.h>
 #include <crepe/api/ParticleEmitter.h>
 #include <crepe/api/Rigidbody.h>
+#include <crepe/api/Sprite.h>
 #include <crepe/api/Texture.h>
 #include <crepe/api/Transform.h>
-#include <crepe/system/RenderSystem.h>
-#include <crepe/api/Color.h>
-#include <crepe/api/Sprite.h>
 #include <crepe/api/Vector2.h>
+#include <crepe/system/RenderSystem.h>
 
 #include <chrono>
 
@@ -20,15 +20,14 @@ using namespace crepe;
 using namespace std;
 
 int main(int argc, char * argv[]) {
-	ComponentManager mgr;	
+	ComponentManager mgr;
 	GameObject game_object = mgr.new_object("", "", Vector2{100, 100}, 0, 0.1);
 	RenderSystem sys{mgr};
 	ParticleSystem psys{mgr};
 
 	Color color(255, 255, 255, 255);
 	Sprite test_sprite = game_object.add_component<Sprite>(
-		make_shared<Texture>("../asset/texture/img.png"), color,
-		FlipSettings{false, false});
+		make_shared<Texture>("../asset/texture/img.png"), color, FlipSettings{false, false});
 	game_object.add_component<ParticleEmitter>(ParticleEmitter::Data{
 		.position = {0, 0},
 		.max_particles = 10,
@@ -54,9 +53,8 @@ int main(int argc, char * argv[]) {
 	while (std::chrono::steady_clock::now() - start < std::chrono::seconds(5)) {
 		psys.update();
 		sys.update();
-		SDL_Delay(10 );
+		SDL_Delay(10);
 	}
 
 	return 0;
 }
-
-- 
cgit v1.2.3


From 700892e9a2801902da31835c0069ae27da54cf4c Mon Sep 17 00:00:00 2001
From: heavydemon21 <nielsstunnebrink1@gmail.com>
Date: Wed, 20 Nov 2024 19:21:46 +0100
Subject: implemented feedback

---
 src/crepe/facade/SDLContext.h     |  4 ++--
 src/crepe/system/RenderSystem.cpp | 11 ++++++-----
 src/crepe/system/RenderSystem.h   | 10 +++++-----
 3 files changed, 13 insertions(+), 12 deletions(-)

(limited to 'src/crepe/facade/SDLContext.h')

diff --git a/src/crepe/facade/SDLContext.h b/src/crepe/facade/SDLContext.h
index 9a514cb..3b17c0b 100644
--- a/src/crepe/facade/SDLContext.h
+++ b/src/crepe/facade/SDLContext.h
@@ -118,10 +118,10 @@ private:
 	 * \param transform Reference to the Transform for positioning.
 	 * \param camera Reference to the Camera for view adjustments.
 	 */
-	void draw(const Sprite & sprite, const Transform & transform, const Camera & camera) const;
+	void draw(const Sprite & sprite, const Transform & transform, const Camera & camera) ;
 
 	void draw_particle(const Sprite & sprite, const Vector2 & pos, const double & angle,
-					   const double & scale, const Camera & camera) const;
+					   const double & scale, const Camera & camera) ;
 
 	//! Clears the screen, preparing for a new frame.
 	void clear_screen();
diff --git a/src/crepe/system/RenderSystem.cpp b/src/crepe/system/RenderSystem.cpp
index 05b7337..e379771 100644
--- a/src/crepe/system/RenderSystem.cpp
+++ b/src/crepe/system/RenderSystem.cpp
@@ -18,9 +18,9 @@
 using namespace crepe;
 using namespace std;
 
-void RenderSystem::clear_screen() const { this->context.clear_screen(); }
+void RenderSystem::clear_screen() { this->context.clear_screen(); }
 
-void RenderSystem::present_screen() const { this->context.present_screen(); }
+void RenderSystem::present_screen() { this->context.present_screen(); }
 void RenderSystem::update_camera() {
 	ComponentManager & mgr = this->component_manager;
 
@@ -29,6 +29,7 @@ void RenderSystem::update_camera() {
 	if (cameras.size() == 0) throw std::runtime_error("No cameras in current scene");
 
 	for (Camera & cam : cameras) {
+		if (!cam.active) continue;
 		this->context.set_camera(cam);
 		this->curr_cam_ref = &cam;
 	}
@@ -57,7 +58,7 @@ void RenderSystem::update() {
 	this->present_screen();
 }
 
-bool RenderSystem::render_particle(const Sprite & sprite, const double & scale) const {
+bool RenderSystem::render_particle(const Sprite & sprite, const double & scale) {
 
 	ComponentManager & mgr = this->component_manager;
 
@@ -79,11 +80,11 @@ bool RenderSystem::render_particle(const Sprite & sprite, const double & scale)
 	}
 	return rendering_particles;
 }
-void RenderSystem::render_normal(const Sprite & sprite, const Transform & tm) const {
+void RenderSystem::render_normal(const Sprite & sprite, const Transform & tm) {
 	this->context.draw(sprite, tm, *this->curr_cam_ref);
 }
 
-void RenderSystem::render() const {
+void RenderSystem::render() {
 
 	ComponentManager & mgr = this->component_manager;
 	vector<reference_wrapper<Sprite>> sprites = mgr.get_components_by_type<Sprite>();
diff --git a/src/crepe/system/RenderSystem.h b/src/crepe/system/RenderSystem.h
index 1a11d5b..5a2af17 100644
--- a/src/crepe/system/RenderSystem.h
+++ b/src/crepe/system/RenderSystem.h
@@ -31,16 +31,16 @@ public:
 
 private:
 	//! Clears the screen in preparation for rendering.
-	void clear_screen() const;
+	void clear_screen() ;
 
 	//! Presents the rendered frame to the display.
-	void present_screen() const;
+	void present_screen() ;
 
 	//! Updates the active camera used for rendering.
 	void update_camera();
 
 	//! Renders the whole screen
-	void render() const;
+	void render();
 
 	/**
 	 * \brief Renders all the particles on the screen from a given sprite.
@@ -49,7 +49,7 @@ private:
 	 * \param tm the Transform component for scale
 	 * \return true if particles have been rendered
 	 */
-	bool render_particle(const Sprite & sprite, const double & scale) const;
+	bool render_particle(const Sprite & sprite, const double & scale);
 
 	/**
 	 * \brief renders a sprite with a Transform component on the screen 
@@ -57,7 +57,7 @@ private:
 	 * \param sprite  the sprite component that holds all the data
 	 * \param tm the Transform component that holds the position,rotation and scale 
 	 */
-	void render_normal(const Sprite & sprite, const Transform & tm) const;
+	void render_normal(const Sprite & sprite, const Transform & tm);
 
 	/**
 	 * \brief sort a vector sprite objects with
-- 
cgit v1.2.3


From 30492d9eee7ee47a42e2471dd0716044f59df705 Mon Sep 17 00:00:00 2001
From: heavydemon21 <nielsstunnebrink1@gmail.com>
Date: Wed, 20 Nov 2024 19:30:34 +0100
Subject: make format

---
 src/crepe/Component.h           | 8 ++++----
 src/crepe/facade/SDLContext.cpp | 3 +--
 src/crepe/facade/SDLContext.h   | 4 ++--
 src/crepe/system/RenderSystem.h | 4 ++--
 4 files changed, 9 insertions(+), 10 deletions(-)

(limited to 'src/crepe/facade/SDLContext.h')

diff --git a/src/crepe/Component.h b/src/crepe/Component.h
index 6d405c7..dc17721 100644
--- a/src/crepe/Component.h
+++ b/src/crepe/Component.h
@@ -27,10 +27,10 @@ protected:
 	//! Only the ComponentManager can create components
 	friend class ComponentManager;
 
-  Component(const Component &) = delete;
-  Component(Component &&) = delete;
-  virtual Component & operator=(const Component &) = delete;
-  virtual Component & operator=(Component &&) = delete;
+	Component(const Component &) = delete;
+	Component(Component &&) = delete;
+	virtual Component & operator=(const Component &) = delete;
+	virtual Component & operator=(Component &&) = delete;
 
 public:
 	virtual ~Component() = default;
diff --git a/src/crepe/facade/SDLContext.cpp b/src/crepe/facade/SDLContext.cpp
index f78ec8f..26a1bdc 100644
--- a/src/crepe/facade/SDLContext.cpp
+++ b/src/crepe/facade/SDLContext.cpp
@@ -135,8 +135,7 @@ void SDLContext::draw_particle(const Sprite & sprite, const Vector2 & pos,
 					 &dstrect, angle, NULL, render_flip);
 }
 
-void SDLContext::draw(const Sprite & sprite, const Transform & transform,
-					  const Camera & cam) {
+void SDLContext::draw(const Sprite & sprite, const Transform & transform, const Camera & cam) {
 
 	SDL_RendererFlip render_flip
 		= (SDL_RendererFlip) ((SDL_FLIP_HORIZONTAL * sprite.flip.flip_x)
diff --git a/src/crepe/facade/SDLContext.h b/src/crepe/facade/SDLContext.h
index 3b17c0b..841ffc9 100644
--- a/src/crepe/facade/SDLContext.h
+++ b/src/crepe/facade/SDLContext.h
@@ -118,10 +118,10 @@ private:
 	 * \param transform Reference to the Transform for positioning.
 	 * \param camera Reference to the Camera for view adjustments.
 	 */
-	void draw(const Sprite & sprite, const Transform & transform, const Camera & camera) ;
+	void draw(const Sprite & sprite, const Transform & transform, const Camera & camera);
 
 	void draw_particle(const Sprite & sprite, const Vector2 & pos, const double & angle,
-					   const double & scale, const Camera & camera) ;
+					   const double & scale, const Camera & camera);
 
 	//! Clears the screen, preparing for a new frame.
 	void clear_screen();
diff --git a/src/crepe/system/RenderSystem.h b/src/crepe/system/RenderSystem.h
index 5a2af17..d25a6e3 100644
--- a/src/crepe/system/RenderSystem.h
+++ b/src/crepe/system/RenderSystem.h
@@ -31,10 +31,10 @@ public:
 
 private:
 	//! Clears the screen in preparation for rendering.
-	void clear_screen() ;
+	void clear_screen();
 
 	//! Presents the rendered frame to the display.
-	void present_screen() ;
+	void present_screen();
 
 	//! Updates the active camera used for rendering.
 	void update_camera();
-- 
cgit v1.2.3