Work in progress

src/application.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _application_hpp
-#define _application_hpp
 
 #include "noncopyable.hpp"
 
@@ -13,5 +11,3 @@ class application : private noncopyable {
   int run();
 };
 }
-
-#endif

src/audiodevice.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _audiodevice_hpp
-#define _audiodevice_hpp
 
 #include "common.hpp"
 
@@ -16,5 +14,3 @@ class audiodevice {
   uint32_t _id;
 };
 }
-
-#endif

src/constants.hpp

@@ -1,6 +1,3 @@
-#ifndef _constants_hpp
-#define _constants_hpp
+#pragma once
 
 static constexpr auto DELAY_MS = 1000 / 60;
-
-#endif

src/delay.hpp

@@ -1,9 +1,5 @@
 #pragma once
-#ifndef _delay_hpp
-#define _delay_hpp
 
 #include "common.hpp"
 
 void sleep(uint32_t ms);
-
-#endif

src/deleters.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _deleters_hpp
-#define _deleters_hpp
 
 #include "common.hpp"
 
@@ -25,5 +23,3 @@ struct SDL_Deleter {
     SDL_RWclose(ptr);
   }
 };
-
-#endif

src/engine.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _engine_hpp
-#define _engine_hpp
 
 #include "common.hpp"
 
@@ -75,5 +73,3 @@ class engine : public input::eventreceiver {
   std::shared_ptr<framework::statemanager> _statemanager;
 };
 }
-
-#endif

src/enginefactory.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _enginefactory_hpp
-#define _enginefactory_hpp
 
 #include "common.hpp"
 
@@ -28,5 +26,3 @@ class enginefactory {
   bool _fullscreen;
 };
 }
-
-#endif

src/entity.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _entity_hpp
-#define _entity_hpp
 
 #include "common.hpp"
 
@@ -68,5 +66,3 @@ class entity : public std::enable_shared_from_this<entity> {
   std::function<void(std::shared_ptr<entity>)> _fn;
 };
 }
-
-#endif

src/entitymanager.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _entitymanager_hpp
-#define _entitymanager_hpp
 
 #include "common.hpp"
 
@@ -27,5 +25,3 @@ class entitymanager {
   std::list<std::shared_ptr<entity>> _entities;
 };
 }
-
-#endif

src/event.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _event_hpp
-#define _event_hpp
 
 #include "common.hpp"
 
@@ -17,5 +15,3 @@ enum class mouseevent : int32_t {};
 
 enum class joysticevent {};
 }
-
-#endif

src/eventmanager.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _eventmanager_hpp_
-#define _eventmanager_hpp_
 
 #include "common.hpp"
 
@@ -22,5 +20,3 @@ class eventmanager : private noncopyable {
   std::list<std::shared_ptr<eventreceiver>> _receivers;
 };
 }
-
-#endif

src/eventreceiver.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _eventreceiver_hpp
-#define _eventreceiver_hpp
 
 #include "common.hpp"
 
@@ -18,5 +16,3 @@ class eventreceiver {
   virtual void on_keyup(const keyevent &event);
 };
 }
-
-#endif

src/filesystem.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _filesystem_hpp
-#define _filesystem_hpp
 
 #include "common.hpp"
 
@@ -14,5 +12,3 @@ class filesystem {
   ~filesystem() noexcept = default;
 };
 }
-
-#endif

src/framerate.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _frames_hpp
-#define _frames_hpp
 
 #include "common.hpp"
 
@@ -22,5 +20,3 @@ class framerate : public loopable {
   uint32_t _start;
 };
 }
-
-#endif

src/helpers.hpp

@@ -1,7 +1,3 @@
 #pragma once
-#ifndef _helpers_hpp
-#define _helpers_hpp
 
 #define UNUSED(x) (void)(x)
-
-#endif

src/io.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _io_hpp
-#define _io_hpp
 
 #include "common.hpp"
 
@@ -13,5 +11,3 @@ class io {
   static const std::vector<uint8_t> read(std::string_view filename);
 };
 }
-
-#endif

src/loopable.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _loopable_hpp
-#define _loopable_hpp
 
 namespace framework {
 class loopable {
@@ -10,5 +8,3 @@ class loopable {
   virtual void loop(uint32_t delta) = 0;
 };
 }
-
-#endif

src/noncopyable.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _noncopyable_hpp_
-#define _noncopyable_hpp_
 
 class noncopyable {
 public:
@@ -12,5 +10,3 @@ class noncopyable {
   noncopyable(noncopyable &&) = default;
   noncopyable &operator=(noncopyable &&) = default;
 };
-
-#endif

src/pixmap.cpp

@@ -30,7 +30,7 @@ pixmap::pixmap(const std::shared_ptr<renderer> renderer, std::string_view filena
 
   std::unique_ptr<SDL_Surface, decltype(&SDL_FreeSurface)> surface{SDL_CreateRGBSurfaceWithFormat(0, decoder->image->width, decoder->image->height, 0, SDL_PIXELFORMAT_ARGB8888), SDL_FreeSurface};
 
-  if (surface == nullptr) {
+  if (!surface) {
     throw std::runtime_error(fmt::format("[SDL_CreateRGBSurfaceWithFormat] error while creating surface with format: {}, error {}", filename, SDL_GetError()));
   }
 
@@ -52,7 +52,7 @@ pixmap::pixmap(const std::shared_ptr<renderer> renderer, std::string_view filena
 
   _texture = texture_ptr(SDL_CreateTextureFromSurface(*renderer, surface.get()), SDL_Deleter());
 
-  if (_texture == nullptr) {
+  if (!_texture) {
     throw std::runtime_error(fmt::format("[SDL_CreateTextureFromSurface] error while creating texture from surface: {}", filename));
   }
 }

src/pixmap.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _pixmap_hpp
-#define _pixmap_hpp
 
 #include "common.hpp"
 
@@ -35,5 +33,3 @@ class pixmap {
   texture_ptr _texture;
 };
 }
-
-#endif

src/pixmappool.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _pixmappool_hpp
-#define _pixmappool_hpp
 
 #include "common.hpp"
 
@@ -22,5 +20,3 @@ class pixmappool {
   std::unordered_map<std::string, std::shared_ptr<pixmap>, std::hash<std::string>> _pool;
 };
 }
-
-#endif

src/point.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _point_hpp
-#define _point_hpp
 
 #include "common.hpp"
 
@@ -25,5 +23,3 @@ class point {
   int32_t _y;
 };
 }
-
-#endif

src/rect.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _rect_hpp
-#define _rect_hpp
 
 #include "common.hpp"
 
@@ -25,5 +23,3 @@ class rect {
   geometry::size _size;
 };
 }
-
-#endif

src/renderer.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _renderer_hpp
-#define _renderer_hpp
 
 #include "common.hpp"
 
@@ -20,5 +18,3 @@ class renderer {
   std::unique_ptr<SDL_Renderer, SDL_Deleter> _renderer;
 };
 }
-
-#endif

src/resourcemanager.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _resourcemanager_hpp
-#define _resourcemanager_hpp
 
 #include "common.hpp"
 
@@ -26,5 +24,3 @@ class resourcemanager {
   std::shared_ptr<audio::soundmanager> _soundmanager;
 };
 }
-
-#endif

src/scriptengine.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _scriptengine_hpp
-#define _scriptengine_hpp
 
 #include "common.hpp"
 
@@ -13,5 +11,3 @@ class scriptengine : private noncopyable {
   void run();
 };
 }
-
-#endif

src/singleton.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _singleton_hpp
-#define _singleton_hpp
 
 #include "common.hpp"
 
@@ -11,7 +9,3 @@ class singleton {
 
   static std::shared_ptr<T> instance();
 };
-
-// #include "singleton.cpp"
-
-#endif

src/size.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _size_hpp
-#define _size_hpp
 
 #include "common.hpp"
 
@@ -26,5 +24,3 @@ class size {
   uint32_t _height;
 };
 }
-
-#endif

src/soundfx.hpp

@@ -1,6 +1,4 @@
 #pragma once
-#ifndef _soundfx_hpp
-#define _soundfx_hpp
 
 #include "common.hpp"
 
@@ -18,5 +16,3 @@ class soundfx {
   std::shared_ptr<audiodevice> _audiodevice;
 };
 }
-
-#endif