actions: fix windows & universal artifacts

# Conflicts:
#	CMakeLists.txt
#	src/tmx2gba.cpp

.github/workflows/cmake.yml

@@ -0,0 +1,47 @@
+name: CMake
+
+on:
+  push:
+  pull_request:
+    branches: [ "master" ]
+
+env:
+  ARTIFACT_NAME: tmx2gba
+  BUILD_TYPE: RelWithDebInfo
+
+jobs:
+  build:
+    strategy:
+      matrix:
+        config:
+        - { name: "MacOS 13.0 Universal", os: macos-13, artifact: macos-universal, arch: arm64;x86_64 }
+        - { name: "Windows MSVC x86", os: windows-latest, artifact: windows-x86, arch: x86 }
+        - { name: "Windows MSVC x64", os: windows-latest, artifact: windows-x64 }
+        - { name: "Windows MSVC ARM64", os: windows-latest, artifact: windows-arm64, arch: amd64_arm64 }
+        - { name: "Ubuntu", artifact: "linux", os: ubuntu-latest }
+    runs-on: ${{matrix.config.os}}
+
+    steps:
+    - uses: actions/checkout@v4
+    # Since ninja isn't used it will take less time if this step is skipped
+    - uses: lukka/get-cmake@latest
+      if: ${{!startsWith(matrix.config.os, 'windows')}}
+    - uses: TheMrMilchmann/setup-msvc-dev@v3
+      if: ${{startsWith(matrix.config.os, 'windows')}}
+      with:
+        arch: ${{matrix.config.arch && matrix.config.arch || 'x64'}}
+
+    - name: Configure CMake
+      run: >-
+        cmake -B build
+        -G "${{startsWith(matrix.config.os, 'windows') && 'NMake Makefiles' || 'Ninja'}}"
+        ${{(startsWith(matrix.config.os, 'macos') && matrix.config.arch) && format('-DCMAKE_OSX_ARCHITECTURES="{0}"', matrix.config.arch) || ''}}
+        ${{matrix.config.extra}} -DCMAKE_BUILD_TYPE=${{env.BUILD_TYPE}}
+
+    - name: Build
+      run: cmake --build build --config ${{env.BUILD_TYPE}}
+
+    - uses: actions/upload-artifact@v4
+      with:
+        name: ${{env.ARTIFACT_NAME}}-${{matrix.config.artifact}}
+        path: build/src/${{env.ARTIFACT_NAME}}${{startsWith(matrix.config.os, 'windows') && '.exe' || ''}}

.gitignore

@@ -1,23 +1,13 @@
 # Compiled Object files
-*.slo
-*.lo
 *.o
 *.obj
 
-# Precompiled Headers
-*.gch
-*.pch
-
 # Compiled Dynamic libraries
 *.so
 *.dylib
 *.dll
 
-# Fortran module files
-*.mod
-
 # Compiled Static libraries
-*.lai
 *.la
 *.a
 *.lib
@@ -27,7 +17,7 @@
 *.out
 *.app
 
-# VS rubbish.
+# VS rubbish
 Debug/
 Release/
 *.opensdf
@@ -35,7 +25,19 @@ Release/
 *.v12.suo
 *.vcxproj.user
 
-# Some files I used for testing.
+# CLion Rubbish
+.idea/
+
+# CMake Rubbish
+build/
+cmake-build-*/
+xcode/
+
+# OS Rubbish
+.DS_Store
+Thumbs.db
+
+# Some files I used for testing
 *.tmx
 *.raw
 plains.png

CMakeLists.txt

@@ -1,45 +1,25 @@
-# tmx2gba - CMakeLists.txt
+cmake_minimum_required(VERSION "3.15" FATAL_ERROR)
-# Copyright (C) 2018 Nicholas Curtis (a dinosaur)
+project(tmx2gba VERSION "0.3")
-#
-# This software is provided 'as-is', without any express or implied
-# warranty.  In no event will the authors be held liable for any damages
-# arising from the use of this software.
-#
-# Permission is granted to anyone to use this software for any purpose,
-# including commercial applications, and to alter it and redistribute it
-# freely, subject to the following restrictions:
-#
-# 1. The origin of this software must not be misrepresented; you must not
-#    claim that you wrote the original software. If you use this software
-#    in a product, an acknowledgment in the product documentation would be
-#    appreciated but is not required.
-# 2. Altered source versions must be plainly marked as such, and must not be
-#    misrepresented as being the original software.
-# 3. This notice may not be removed or altered from any source distribution.
 
-cmake_minimum_required(VERSION 3.10)
+# Options
-project(tmx2gba)
+option(TMX2GBA_DKP_INSTALL "Install into DEVKITPRO prefix" OFF)
+option(ASAN "Enable address sanitiser" OFF)
 
-set(CMAKE_CXX_STANDARD 11)
+if (ASAN)
-
+	add_compile_options(-fsanitize=address -fno-omit-frame-pointer)
-include_directories(inc)
+	add_link_options(-fsanitize=address -shared-libasan)
-set(SOURCES
-	src/tmx2gba.cpp
-	src/tmxlayer.cpp	src/tmxlayer.h
-	src/tmxobject.cpp	src/tmxobject.h
-	src/tmxreader.cpp	src/tmxreader.h
-	src/tmxtileset.cpp	src/tmxtileset.h
-
-	src/base64.cpp		inc/base64.h
-	inc/miniz.h
-
-	inc/rapidxml/rapidxml.hpp
-	inc/rapidxml/rapidxml_iterators.hpp
-	inc/rapidxml/rapidxml_print.hpp
-	inc/rapidxml/rapidxml_utils.hpp)
-
-if (MSVC)
-	set(SOURCES ${SOURCES} src/XGetopt.cpp inc/XGetopt.h)
 endif()
 
-add_executable(tmx2gba ${SOURCES})
+# Libraries
+add_subdirectory(ext/base64)
+add_subdirectory(ext/miniz)
+add_subdirectory(ext/rapidxml)
+
+# Main tmx2gba sources
+add_subdirectory(src)
+
+if (MSVC)
+	# Default to tmx2gba as startup project when generating Solutions
+	set_property(DIRECTORY ${CMAKE_SOURCE_DIR}
+		PROPERTY VS_STARTUP_PROJECT tmx2gba)
+endif()

COPYING.txt

@@ -0,0 +1,17 @@
+Copyright (C) 2015-2024 a dinosaur
+
+This software is provided 'as-is', without any express or implied
+warranty.  In no event will the authors be held liable for any damages
+arising from the use of this software.
+
+Permission is granted to anyone to use this software for any purpose,
+including commercial applications, and to alter it and redistribute it
+freely, subject to the following restrictions:
+
+1. The origin of this software must not be misrepresented; you must not
+   claim that you wrote the original software. If you use this software
+   in a product, an acknowledgment in the product documentation would be
+   appreciated but is not required.
+2. Altered source versions must be plainly marked as such, and must not be
+   misrepresented as being the original software.
+3. This notice may not be removed or altered from any source distribution.

README.md

@@ -1,6 +1,6 @@
 # tmx2gba #
-tmx2gba is a simple command line utility that converts [Tiled](http://www.mapeditor.org/) .tmx maps to Game Boy Advance compatible charmaps.
+tmx2gba is a simple command line utility that converts [Tiled](http://www.mapeditor.org/) .tmx maps to Game Boy Advance formatted charmaps.
-Originally developed for my own personal use, I've thrown it up on glorious Github/Gitlab/whatever in case this is of use to anyone else.
+Originally developed for my own personal use, I've thrown it up in case this is of use to anyone else.
 
 If you find a bug, please open an issue.
 
@@ -17,50 +17,63 @@ Enjoy!
 tmx2gba [-h] [-r offset] [-lyc name] [-p 0-15] <-i inpath> <-o outpath>
 ```
 
-Command     | Required | Notes
+| Command      | Required | Notes                                                                 |
-------------|----------|----------------------------------------------------------------------
+|--------------|----------|-----------------------------------------------------------------------|
--h          | N/A      | Display help & command info.
+| -h           | N/A      | Display help & command info.                                          |
--l (name)   | No       | Name of layer to use (default first layer in TMX).
+| -v           | No       | Display version & quit.                                               |
--y (name)   | No       | Layer for palette mappings.
+| -l (name)    | No       | Name of layer to use (default first layer in TMX).                    |
--c (name)   | No       | Output a separate 8bit collision map of the specified layer.
+| -y (name)    | No       | Layer for palette mappings.                                           |
--r (offset) | No       | Offset tile indices (default 0).
+| -c (name)    | No       | Output a separate 8bit collision map of the specified layer.          |
--p (0-15)   | No       | Select which palette to use for 4-bit tilesets.
+| -r (offset)  | No       | Offset tile indices (default 0).                                      |
--m (name;id)| No       | Map an object name to an ID, will enable object exports.
+| -p (0-15)    | No       | Select which palette to use for 4-bit tilesets.                       |
--i (path)   | *Yes*    | Path to input TMX file.
+| -m (name;id) | No       | Map an object name to an ID, will enable object exports.              |
--o (path)   | *Yes*    | Path to output files.
+| -i (path)    | *Yes*    | Path to input TMX file.                                               |
--f <file>   | No       | Command line instructions list for easy integration with buildscripts
+| -o (path)    | *Yes*    | Path to output files.                                                 |
+| -f <file>    | No       | Command line instructions list for easy integration with buildscripts |
 
 ### How do I build it? ###
 
 Dependencies for building are CMake 3.x and a C++11 compliant compiler,
 all other dependencies are in-tree so you should be able to build with:
 ```bash
-mkdir build && cd build
+cmake -B build -DCMAKE_BUILD_TYPE=RelWithDebInfo
-cmake ..
+make -C build -j$(nproc --all)
-make
 ```
 
-Optionally, to make it convenient for my dkp projects:
+Optionally, you may install it to use it system wide:
 ```bash
-sudo cp tmx2gba $DEVKITPRO/tools/bin/tmx2gba
+sudo cmake --install build
+```
+Which will copy the tmx2gba executable to /usr/local/bin/tmx2gba by default,
+if you prefer to use /usr for some reason you may specify a prefix like so:
+```bash
+sudo cmake --install build --prefix /usr
+```
+If you're a devkitPro user and would prefer to keep all your development tools compartmentalised
+you may optionally install to the tools directory with the `TMX2GBA_DKP_INSTALL` option (OFF by default).
+The build scripts will respect your `DEVKITPRO` environment variable but if not set will install to
+`/opt/devkitpro/tools/bin/tmx2gba` directly, the `--prefix` argument has no effect in this mode.
+```bash
+cmake -B build -DCMAKE_BUILD_TYPE=Release -DTMX2GBA_DKP_INSTALL:BOOL=ON
+cmake --build build
+sudo cmake --install build
 ```
 
 ### Todo list ###
 * Add support for multi-SBB prepared charmaps.
-* Test CMakeLists for Windows compatibility.
 * Check if this works for NDS as well.
 * Compression support.
-* Prehaps use GNU style getopt_long?
+* Support for less common TMX formats.
-* Refactor & Fix bugs. *(duh)*
 
 ### License ###
-tmx2gba is licensed under the zlib license.
+[tmx2gba](https://github.com/ScrelliCopter/tmx2gba) is licensed under the zlib license.
-RapidXML is licensed under the Boost & MIT licenses.
+[RapidXML](http://rapidxml.sourceforge.net/) is licensed under the Boost & MIT licenses.
-René Nyffenegger's base64.cpp is licensed under the zlib license.
+[René Nyffenegger's base64.cpp](https://github.com/ReneNyffenegger/cpp-base64) is licensed under the zlib license.
-XGetopt & miniz are both public domain software.
+[miniz](https://github.com/richgel999/miniz) is public domain software.
+[ultragetopt](https://github.com/kevinoid/ultragetopt) is licensed under the MIT license.
 
 ```
-  Copyright (C) 2015-2018 Nicholas Curtis (a dinosaur)
+  Copyright (C) 2015-2023 a dinosaur
 
   This software is provided 'as-is', without any express or implied
   warranty.  In no event will the authors be held liable for any damages

ext/base64/CMakeLists.txt

@@ -0,0 +1,5 @@
+add_library(base64
+	base64.cpp base64.h)
+add_library(External::base64 ALIAS base64)
+target_include_directories(base64
+	PUBLIC ${CMAKE_CURRENT_SOURCE_DIR})

ext/base64/base64.cpp

@@ -0,0 +1,282 @@
+/*
+   base64.cpp and base64.h
+
+   base64 encoding and decoding with C++.
+   More information at
+     https://renenyffenegger.ch/notes/development/Base64/Encoding-and-decoding-base-64-with-cpp
+
+   Version: 2.rc.08 (release candidate)
+
+   Copyright (C) 2004-2017, 2020, 2021 René Nyffenegger
+
+   This source code is provided 'as-is', without any express or implied
+   warranty. In no event will the author be held liable for any damages
+   arising from the use of this software.
+
+   Permission is granted to anyone to use this software for any purpose,
+   including commercial applications, and to alter it and redistribute it
+   freely, subject to the following restrictions:
+
+   1. The origin of this source code must not be misrepresented; you must not
+      claim that you wrote the original source code. If you use this source code
+      in a product, an acknowledgment in the product documentation would be
+      appreciated but is not required.
+
+   2. Altered source versions must be plainly marked as such, and must not be
+      misrepresented as being the original source code.
+
+   3. This notice may not be removed or altered from any source distribution.
+
+   René Nyffenegger rene.nyffenegger@adp-gmbh.ch
+
+*/
+
+#include "base64.h"
+
+#include <algorithm>
+#include <stdexcept>
+
+ //
+ // Depending on the url parameter in base64_chars, one of
+ // two sets of base64 characters needs to be chosen.
+ // They differ in their last two characters.
+ //
+static const char* base64_chars[2] = {
+             "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
+             "abcdefghijklmnopqrstuvwxyz"
+             "0123456789"
+             "+/",
+
+             "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
+             "abcdefghijklmnopqrstuvwxyz"
+             "0123456789"
+             "-_"};
+
+static unsigned int pos_of_char(const unsigned char chr) {
+ //
+ // Return the position of chr within base64_encode()
+ //
+
+    if      (chr >= 'A' && chr <= 'Z') return chr - 'A';
+    else if (chr >= 'a' && chr <= 'z') return chr - 'a' + ('Z' - 'A')               + 1;
+    else if (chr >= '0' && chr <= '9') return chr - '0' + ('Z' - 'A') + ('z' - 'a') + 2;
+    else if (chr == '+' || chr == '-') return 62; // Be liberal with input and accept both url ('-') and non-url ('+') base 64 characters (
+    else if (chr == '/' || chr == '_') return 63; // Ditto for '/' and '_'
+    else
+ //
+ // 2020-10-23: Throw std::exception rather than const char*
+ //(Pablo Martin-Gomez, https://github.com/Bouska)
+ //
+    throw std::runtime_error("Input is not valid base64-encoded data.");
+}
+
+static std::string insert_linebreaks(std::string str, size_t distance) {
+ //
+ // Provided by https://github.com/JomaCorpFX, adapted by me.
+ //
+    if (!str.length()) {
+        return "";
+    }
+
+    size_t pos = distance;
+
+    while (pos < str.size()) {
+        str.insert(pos, "\n");
+        pos += distance + 1;
+    }
+
+    return str;
+}
+
+template <typename String, unsigned int line_length>
+static std::string encode_with_line_breaks(String s) {
+  return insert_linebreaks(base64_encode(s, false), line_length);
+}
+
+template <typename String>
+static std::string encode_pem(String s) {
+  return encode_with_line_breaks<String, 64>(s);
+}
+
+template <typename String>
+static std::string encode_mime(String s) {
+  return encode_with_line_breaks<String, 76>(s);
+}
+
+template <typename String>
+static std::string encode(String s, bool url) {
+  return base64_encode(reinterpret_cast<const unsigned char*>(s.data()), s.length(), url);
+}
+
+std::string base64_encode(unsigned char const* bytes_to_encode, size_t in_len, bool url) {
+
+    size_t len_encoded = (in_len +2) / 3 * 4;
+
+    unsigned char trailing_char = url ? '.' : '=';
+
+ //
+ // Choose set of base64 characters. They differ
+ // for the last two positions, depending on the url
+ // parameter.
+ // A bool (as is the parameter url) is guaranteed
+ // to evaluate to either 0 or 1 in C++ therefore,
+ // the correct character set is chosen by subscripting
+ // base64_chars with url.
+ //
+    const char* base64_chars_ = base64_chars[url];
+
+    std::string ret;
+    ret.reserve(len_encoded);
+
+    unsigned int pos = 0;
+
+    while (pos < in_len) {
+        ret.push_back(base64_chars_[(bytes_to_encode[pos + 0] & 0xfc) >> 2]);
+
+        if (pos+1 < in_len) {
+           ret.push_back(base64_chars_[((bytes_to_encode[pos + 0] & 0x03) << 4) + ((bytes_to_encode[pos + 1] & 0xf0) >> 4)]);
+
+           if (pos+2 < in_len) {
+              ret.push_back(base64_chars_[((bytes_to_encode[pos + 1] & 0x0f) << 2) + ((bytes_to_encode[pos + 2] & 0xc0) >> 6)]);
+              ret.push_back(base64_chars_[  bytes_to_encode[pos + 2] & 0x3f]);
+           }
+           else {
+              ret.push_back(base64_chars_[(bytes_to_encode[pos + 1] & 0x0f) << 2]);
+              ret.push_back(trailing_char);
+           }
+        }
+        else {
+
+            ret.push_back(base64_chars_[(bytes_to_encode[pos + 0] & 0x03) << 4]);
+            ret.push_back(trailing_char);
+            ret.push_back(trailing_char);
+        }
+
+        pos += 3;
+    }
+
+
+    return ret;
+}
+
+template <typename String>
+static std::string decode(String encoded_string, bool remove_linebreaks) {
+ //
+ // decode(…) is templated so that it can be used with String = const std::string&
+ // or std::string_view (requires at least C++17)
+ //
+
+    if (encoded_string.empty()) return std::string();
+
+    if (remove_linebreaks) {
+
+       std::string copy(encoded_string);
+
+       copy.erase(std::remove(copy.begin(), copy.end(), '\n'), copy.end());
+
+       return base64_decode(copy, false);
+    }
+
+    size_t length_of_string = encoded_string.length();
+    size_t pos = 0;
+
+ //
+ // The approximate length (bytes) of the decoded string might be one or
+ // two bytes smaller, depending on the amount of trailing equal signs
+ // in the encoded string. This approximation is needed to reserve
+ // enough space in the string to be returned.
+ //
+    size_t approx_length_of_decoded_string = length_of_string / 4 * 3;
+    std::string ret;
+    ret.reserve(approx_length_of_decoded_string);
+
+    while (pos < length_of_string) {
+    //
+    // Iterate over encoded input string in chunks. The size of all
+    // chunks except the last one is 4 bytes.
+    //
+    // The last chunk might be padded with equal signs or dots
+    // in order to make it 4 bytes in size as well, but this
+    // is not required as per RFC 2045.
+    //
+    // All chunks except the last one produce three output bytes.
+    //
+    // The last chunk produces at least one and up to three bytes.
+    //
+
+       size_t pos_of_char_1 = pos_of_char(encoded_string[pos+1] );
+
+    //
+    // Emit the first output byte that is produced in each chunk:
+    //
+       ret.push_back(static_cast<std::string::value_type>( ( (pos_of_char(encoded_string[pos+0]) ) << 2 ) + ( (pos_of_char_1 & 0x30 ) >> 4)));
+
+       if ( ( pos + 2 < length_of_string  )       &&  // Check for data that is not padded with equal signs (which is allowed by RFC 2045)
+              encoded_string[pos+2] != '='        &&
+              encoded_string[pos+2] != '.'            // accept URL-safe base 64 strings, too, so check for '.' also.
+          )
+       {
+       //
+       // Emit a chunk's second byte (which might not be produced in the last chunk).
+       //
+          unsigned int pos_of_char_2 = pos_of_char(encoded_string[pos+2] );
+          ret.push_back(static_cast<std::string::value_type>( (( pos_of_char_1 & 0x0f) << 4) + (( pos_of_char_2 & 0x3c) >> 2)));
+
+          if ( ( pos + 3 < length_of_string )     &&
+                 encoded_string[pos+3] != '='     &&
+                 encoded_string[pos+3] != '.'
+             )
+          {
+          //
+          // Emit a chunk's third byte (which might not be produced in the last chunk).
+          //
+             ret.push_back(static_cast<std::string::value_type>( ( (pos_of_char_2 & 0x03 ) << 6 ) + pos_of_char(encoded_string[pos+3])   ));
+          }
+       }
+
+       pos += 4;
+    }
+
+    return ret;
+}
+
+std::string base64_decode(std::string const& s, bool remove_linebreaks) {
+   return decode(s, remove_linebreaks);
+}
+
+std::string base64_encode(std::string const& s, bool url) {
+   return encode(s, url);
+}
+
+std::string base64_encode_pem (std::string const& s) {
+   return encode_pem(s);
+}
+
+std::string base64_encode_mime(std::string const& s) {
+   return encode_mime(s);
+}
+
+#if __cplusplus >= 201703L
+//
+// Interface with std::string_view rather than const std::string&
+// Requires C++17
+// Provided by Yannic Bonenberger (https://github.com/Yannic)
+//
+
+std::string base64_encode(std::string_view s, bool url) {
+   return encode(s, url);
+}
+
+std::string base64_encode_pem(std::string_view s) {
+   return encode_pem(s);
+}
+
+std::string base64_encode_mime(std::string_view s) {
+   return encode_mime(s);
+}
+
+std::string base64_decode(std::string_view s, bool remove_linebreaks) {
+   return decode(s, remove_linebreaks);
+}
+
+#endif  // __cplusplus >= 201703L

ext/base64/base64.h

@@ -0,0 +1,35 @@
+//
+//  base64 encoding and decoding with C++.
+//  Version: 2.rc.08 (release candidate)
+//
+
+#ifndef BASE64_H_C0CE2A47_D10E_42C9_A27C_C883944E704A
+#define BASE64_H_C0CE2A47_D10E_42C9_A27C_C883944E704A
+
+#include <string>
+
+#if __cplusplus >= 201703L
+#include <string_view>
+#endif  // __cplusplus >= 201703L
+
+std::string base64_encode     (std::string const& s, bool url = false);
+std::string base64_encode_pem (std::string const& s);
+std::string base64_encode_mime(std::string const& s);
+
+std::string base64_decode(std::string const& s, bool remove_linebreaks = false);
+std::string base64_encode(unsigned char const*, size_t len, bool url = false);
+
+#if __cplusplus >= 201703L
+//
+// Interface with std::string_view rather than const std::string&
+// Requires C++17
+// Provided by Yannic Bonenberger (https://github.com/Yannic)
+//
+std::string base64_encode     (std::string_view s, bool url = false);
+std::string base64_encode_pem (std::string_view s);
+std::string base64_encode_mime(std::string_view s);
+
+std::string base64_decode(std::string_view s, bool remove_linebreaks = false);
+#endif  // __cplusplus >= 201703L
+
+#endif /* BASE64_H_C0CE2A47_D10E_42C9_A27C_C883944E704A */

ext/miniz/CMakeLists.txt

@@ -0,0 +1,5 @@
+add_library(miniz
+	miniz.c miniz.h)
+add_library(External::miniz ALIAS miniz)
+target_include_directories(miniz
+	PUBLIC ${CMAKE_CURRENT_SOURCE_DIR})

ext/rapidxml/CMakeLists.txt

@@ -0,0 +1,4 @@
+add_library(rapidxml INTERFACE)
+add_library(External::rapidxml ALIAS rapidxml)
+target_include_directories(rapidxml
+	INTERFACE ${CMAKE_CURRENT_SOURCE_DIR})

inc/XGetopt.h

@@ -1,23 +0,0 @@
-// XGetopt.h  Version 1.2
-//
-// Author:  Hans Dietrich
-//          hdietrich2@hotmail.com
-//
-// This software is released into the public domain.
-// You are free to use it in any way you like.
-//
-// This software is provided "as is" with no expressed
-// or implied warranty.  I accept no liability for any
-// damage or loss of business that this software may cause.
-//
-///////////////////////////////////////////////////////////////////////////////
-
-#ifndef XGETOPT_H
-#define XGETOPT_H
-
-extern int optind;
-extern char *optarg, *next;
-
-int getopt(int argc, char *argv[], char *optstring);
-
-#endif //XGETOPT_H

inc/base64.h

@@ -1,9 +0,0 @@
-#ifndef __BASE64_H__
-#define __BASE64_H__
-
-#include <string>
-
-std::string base64_encode(unsigned char const* , unsigned int len);
-std::string base64_decode(std::string const& s);
-
-#endif

src/CMakeLists.txt

@@ -0,0 +1,36 @@
+add_executable(tmx2gba
+	argparse.hpp argparse.cpp
+	tmxlayer.hpp
+	tmxobject.hpp
+	tmxreader.hpp tmxreader.cpp
+	tmxtileset.hpp
+	swriter.hpp swriter.cpp
+	headerwriter.hpp headerwriter.cpp
+	tmx2gba.cpp)
+
+set_target_properties(tmx2gba PROPERTIES
+	# C++20 & C99
+	CXX_STANDARD 20
+	C_STANDARD 99)
+
+# Enable strong warnings
+target_compile_options(tmx2gba PRIVATE
+	$<$<CXX_COMPILER_ID:MSVC>:/Wall>
+	$<$<CXX_COMPILER_ID:GNU,Clang,AppleClang>:-Wall -Wextra -pedantic>
+	$<$<CXX_COMPILER_ID:Clang,AppleClang>:-Weverything -Wno-c++98-compat>)
+
+target_link_libraries(tmx2gba
+	External::base64
+	External::miniz
+	External::rapidxml)
+
+if (TMX2GBA_DKP_INSTALL)
+	if (DEFINED ENV{DEVKITPRO})
+		set(TMX2GBA_INSTALL_DESTINATION "$ENV{DEVKITPRO}/tools/bin")
+	else()
+		set(TMX2GBA_INSTALL_DESTINATION /opt/devkitpro/tools/bin)
+	endif()
+else()
+	set(TMX2GBA_INSTALL_DESTINATION bin)
+endif()
+install(TARGETS tmx2gba RUNTIME DESTINATION "${TMX2GBA_INSTALL_DESTINATION}")

src/XGetopt.cpp

@@ -1,212 +0,0 @@
-// XGetopt.cpp  Version 1.2
-//
-// Author:  Hans Dietrich
-//          hdietrich2@hotmail.com
-//
-// Description:
-//     XGetopt.cpp implements getopt(), a function to parse command lines.
-//
-// History
-//     Version 1.2 - 2003 May 17
-//     - Added Unicode support
-//
-//     Version 1.1 - 2002 March 10
-//     - Added example to XGetopt.cpp module header 
-//
-// This software is released into the public domain.
-// You are free to use it in any way you like.
-//
-// This software is provided "as is" with no expressed
-// or implied warranty.  I accept no liability for any
-// damage or loss of business that this software may cause.
-//
-///////////////////////////////////////////////////////////////////////////////
-
-
-///////////////////////////////////////////////////////////////////////////////
-// if you are not using precompiled headers then include these lines:
-#include <stdio.h>
-#include <string.h>
-///////////////////////////////////////////////////////////////////////////////
-
-
-#include "XGetopt.h"
-
-
-///////////////////////////////////////////////////////////////////////////////
-//
-//  X G e t o p t . c p p
-//
-//
-//  NAME
-//       getopt -- parse command line options
-//
-//  SYNOPSIS
-//       int getopt(int argc, char *argv[], char *optstring)
-//
-//       extern char *optarg;
-//       extern int optind;
-//
-//  DESCRIPTION
-//       The getopt() function parses the command line arguments. Its
-//       arguments argc and argv are the argument count and array as
-//       passed into the application on program invocation.  In the case
-//       of Visual C++ programs, argc and argv are available via the
-//       variables __argc and __argv (double underscores), respectively.
-//       getopt returns the next option letter in argv that matches a
-//       letter in optstring.  (Note:  Unicode programs should use
-//       __targv instead of __argv.  Also, all character and string
-//       literals should be enclosed in _T( ) ).
-//
-//       optstring is a string of recognized option letters;  if a letter
-//       is followed by a colon, the option is expected to have an argument
-//       that may or may not be separated from it by white space.  optarg
-//       is set to point to the start of the option argument on return from
-//       getopt.
-//
-//       Option letters may be combined, e.g., "-ab" is equivalent to
-//       "-a -b".  Option letters are case sensitive.
-//
-//       getopt places in the external variable optind the argv index
-//       of the next argument to be processed.  optind is initialized
-//       to 0 before the first call to getopt.
-//
-//       When all options have been processed (i.e., up to the first
-//       non-option argument), getopt returns EOF, optarg will point
-//       to the argument, and optind will be set to the argv index of
-//       the argument.  If there are no non-option arguments, optarg
-//       will be set to nullptr.
-//
-//       The special option "--" may be used to delimit the end of the
-//       options;  EOF will be returned, and "--" (and everything after it)
-//       will be skipped.
-//
-//  RETURN VALUE
-//       For option letters contained in the string optstring, getopt
-//       will return the option letter.  getopt returns a question mark (?)
-//       when it encounters an option letter not included in optstring.
-//       EOF is returned when processing is finished.
-//
-//  BUGS
-//       1)  Long options are not supported.
-//       2)  The GNU double-colon extension is not supported.
-//       3)  The environment variable POSIXLY_CORRECT is not supported.
-//       4)  The + syntax is not supported.
-//       5)  The automatic permutation of arguments is not supported.
-//       6)  This implementation of getopt() returns EOF if an error is
-//           encountered, instead of -1 as the latest standard requires.
-//
-//  EXAMPLE
-//       BOOL CMyApp::ProcessCommandLine(int argc, char *argv[])
-//       {
-//           int c;
-//
-//           while ((c = getopt(argc, argv, _T("aBn:"))) != EOF)
-//           {
-//               switch (c)
-//               {
-//                   case _T('a'):
-//                       TRACE(_T("option a\n"));
-//                       //
-//                       // set some flag here
-//                       //
-//                       break;
-//
-//                   case _T('B'):
-//                       TRACE( _T("option B\n"));
-//                       //
-//                       // set some other flag here
-//                       //
-//                       break;
-//
-//                   case _T('n'):
-//                       TRACE(_T("option n: value=%d\n"), atoi(optarg));
-//                       //
-//                       // do something with value here
-//                       //
-//                       break;
-//
-//                   case _T('?'):
-//                       TRACE(_T("ERROR: illegal option %s\n"), argv[optind-1]);
-//                       return FALSE;
-//                       break;
-//
-//                   default:
-//                       TRACE(_T("WARNING: no handler for option %c\n"), c);
-//                       return FALSE;
-//                       break;
-//               }
-//           }
-//           //
-//           // check for non-option args here
-//           //
-//           return TRUE;
-//       }
-//
-///////////////////////////////////////////////////////////////////////////////
-
-char	*optarg;		// global argument pointer
-char	*next;
-int		optind = 0; 	// global argv index
-
-int getopt(int argc, char *argv[], char *optstring)
-{
-	if (optind == 0)
-		next = nullptr;
-
-	optarg = nullptr;
-
-	if (next == nullptr || *next == '\0')
-	{
-		if (optind == 0)
-			optind++;
-
-		if (optind >= argc || argv[optind][0] != '-' || argv[optind][1] == '\0')
-		{
-			optarg = nullptr;
-			if (optind < argc)
-				optarg = argv[optind];
-			return EOF;
-		}
-
-		if (strcmp(argv[optind], "--") == 0)
-		{
-			optind++;
-			optarg = nullptr;
-			if (optind < argc)
-				optarg = argv[optind];
-			return EOF;
-		}
-
-		next = argv[optind];
-		next++;		// skip past -
-		optind++;
-	}
-
-	char c = *next++;
-	char *cp = strchr(optstring, c);
-
-	if (cp == nullptr || c == ':')
-		return '?';
-
-	cp++;
-	if (*cp == ':')
-	{
-		if (*next != '\0')
-		{
-			optarg = next;
-			next = nullptr;
-		}
-		else if (optind < argc)
-		{
-			optarg = argv[optind];
-			optind++;
-		}
-		else
-		{
-			return '?';
-		}
-	}
-
-	return c;
-}

src/argparse.cpp

@@ -0,0 +1,216 @@
+/* argparse.cpp - Copyright (C) 2024 a dinosaur (zlib, see COPYING.txt) */
+
+#include "argparse.hpp"
+#include <optional>
+#include <cstring>
+#include <filesystem>
+#include <iomanip>
+#include <iostream>
+
+
+ArgParse::ArgParser::ArgParser(
+	const std::string_view argv0,
+	Options options,
+	HandleOption&& handler
+) noexcept :
+	name(std::filesystem::path(argv0).filename().string()),
+	options(options),
+	handler(std::forward<HandleOption>(handler)) {}
+
+
+void ArgParse::Options::ShowShortUsage(const std::string_view name, std::ostream& out) const
+{
+	out << "Usage: " << name;
+	for (const auto& it : options)
+	{
+		if (it.argumentName)
+		{
+			// Option with argument
+			it.required
+				? out << " <-" << it.flag << ' ' << it.argumentName << '>'
+				: out << " [-" << it.flag << ' ' << it.argumentName << ']';
+		}
+		else
+		{
+			// Argument-less flag
+			it.required
+				? out << " <-" << it.flag << '>'
+				: out << " [-" << it.flag << ']';
+		}
+	}
+	out << std::endl;
+}
+
+void ArgParse::Options::ShowHelpUsage(const std::string_view name, std::ostream& out) const
+{
+	// Base usage
+	out << "Usage: " << name << " [-";
+	for (const auto& it : options)
+		if (!it.required)
+			out << it.flag;
+	out << "] <-";
+	for (const auto& it : options)
+		if (it.required)
+			out << it.flag;
+	out << ">" << std::endl;
+
+	// Determine the alignment width from the longest argument
+	auto paramLength = [](const Option& p) -> int { return p.argumentName
+		? static_cast<int>(std::strlen(p.argumentName) + 3)
+		: 1; };
+	auto longestParam = std::max_element(options.begin(), options.end(),
+		[=](auto a, auto b) -> bool { return paramLength(a) < paramLength(b); });
+	auto alignWidth = paramLength(*longestParam) + 3;
+
+	// print argument descriptions
+	for (const auto& it : options)
+	{
+		auto decorateArgument = [=] { return " <" + std::string(it.argumentName) + "> "; };
+		out << "  -" << it.flag
+			<< std::left << std::setw(alignWidth) << std::setfill('-') << (it.argumentName ? decorateArgument() : " ")
+			<< " " << it.helpString << std::endl;
+	}
+	out << std::flush;
+}
+
+
+ArgParse::ParseCtrl ArgParse::ParserState::Next(const std::string_view token)
+{
+	auto getFlag = [](const std::string_view s) { return s[0] == '-' && s[1] ? std::optional<int>(s[1]) : std::nullopt; };
+	auto getOption = [&](int flag) -> std::optional<std::reference_wrapper<const Option>>
+	{
+		for (auto& opt : options.options)
+			if (opt.flag == flag)
+				return std::optional(std::cref(opt));
+		return {};
+	};
+
+	if (expectArg)
+	{
+		expectArg = false;
+		return handler(flagChar, token);
+	}
+	else
+	{
+		auto flag = getFlag(token);
+		if (flag.has_value())
+		{
+			flagChar = flag.value();
+			const auto opt = getOption(flagChar);
+			if (opt.has_value())
+			{
+				bool expect = opt.value().get().argumentName != nullptr;
+				if (token.length() <= 2)
+				{
+					expectArg = expect;
+					if (!expectArg)
+						return handler(flagChar, "");
+				}
+				else
+				{
+					return handler(flagChar, expect ? token.substr(2) : "");
+				}
+			}
+		}
+		else if (!token.empty())
+		{
+			return ParseCtrl::QUIT_ERR_UNEXPECTED;
+		}
+	}
+
+	return ParseCtrl::CONTINUE;
+}
+
+
+void ArgParse::ArgParser::DisplayError(const std::string_view message, bool helpPrompt) const
+{
+	std::cerr << GetName() << ": " << message << std::endl;
+	options.ShowShortUsage(GetName(), std::cerr);
+	if (helpPrompt)
+		std::cerr << "Run '" << GetName() << " -h' to view all available options." << std::endl;
+}
+
+bool ArgParse::ArgParser::CheckParse(ArgParse::ParseErr err) const
+{
+	switch (err)
+	{
+	case ParseErr::OK:
+		return true;
+	case ParseErr::OPT_UNKNOWN:
+		DisplayError("Unrecognised option.");
+		return false;
+	case ParseErr::UNEXPECTED:
+		DisplayError("Unexpected token.");
+		return false;
+	case ParseErr::ARG_EXPECTED:
+		DisplayError("Requires an argument.");
+		return false;
+	case ParseErr::ARG_INVALID:
+		DisplayError("Invalid argument.", false);
+		return false;
+	case ParseErr::ARG_RANGE:
+		DisplayError("Argument out of range.", false);
+		return false;
+	}
+}
+
+
+bool ArgParse::ReadParamFile(std::vector<std::string>& tokens, std::istream& file)
+{
+	bool inQuote = false;
+	std::string quoteStr;
+	const auto store = [&](const std::string_view token, bool quote)
+	{
+		if (quote)
+			quoteStr = token;
+		else
+			tokens.emplace_back(token);
+	};
+
+	while (!file.eof())
+	{
+		if (!inQuote)
+		{
+			std::string token;
+			file >> token;
+			if (!token.empty())
+			{
+				std::string::size_type beg = 0, end;
+				while ((end = token.find_first_of('"', beg)) != std::string::npos)
+				{
+					auto size = end - beg;
+					if (size > 0)
+						store(token.substr(beg, size), !inQuote);
+					inQuote = !inQuote;
+					beg = end + 1;
+				}
+				if (beg > 0)
+				{
+					auto size = token.length() - beg;
+					if (size > 0)
+						token = token.substr(beg);
+					else
+						token.clear();
+				}
+				if (!token.empty())
+					store(token, inQuote);
+			}
+		}
+		else
+		{
+			const int c = file.get();
+			if (c == '"')
+			{
+				tokens.emplace_back(quoteStr);
+				quoteStr.clear();
+				inQuote = false;
+			}
+			else
+			{
+				quoteStr.push_back(static_cast<char>(c));
+			}
+		}
+	}
+
+	return !inQuote;
+}

src/argparse.hpp

@@ -0,0 +1,132 @@
+/* argparse.hpp - Copyright (C) 2024 a dinosaur (zlib, see COPYING.txt) */
+
+#ifndef ARGPARSE_HPP
+#define ARGPARSE_HPP
+
+#include <initializer_list>
+#include <functional>
+#include <string>
+#include <ostream>
+#include <span>
+#include <ranges>
+#include <string_view>
+#include <vector>
+
+namespace ArgParse
+{
+	struct Option
+	{
+		const char* argumentName;
+		const char* helpString;
+		char flag;
+		bool required;
+
+		static constexpr Option Optional(char flag, const char* name, const char* help)
+		{
+			return { name, help, flag, false };
+		}
+		static constexpr Option Required(char flag, const char* name, const char* help)
+		{
+			return { name, help, flag, false };
+		}
+	};
+
+	struct Options
+	{
+		const std::vector<Option> options;
+
+		inline Options(const std::initializer_list<Option>&& rhs) : options(rhs) {}
+
+		void ShowShortUsage(const std::string_view name, std::ostream& out) const;
+		void ShowHelpUsage(const std::string_view name, std::ostream& out) const;
+	};
+
+	enum class ParseCtrl
+	{
+		CONTINUE,
+		QUIT_EARLY,
+		QUIT_ERR_UNKNOWN,
+		QUIT_ERR_UNEXPECTED,
+		QUIT_ERR_EXPECTARG,
+		QUIT_ERR_INVALID,
+		QUIT_ERR_RANGE
+	};
+
+	enum class ParseErr
+	{
+		OK,
+		OPT_UNKNOWN,
+		UNEXPECTED,
+		ARG_EXPECTED,
+		ARG_INVALID,
+		ARG_RANGE
+	};
+
+	using HandleOption = std::function<ParseCtrl(int, const std::string_view)>;
+
+	class ParserState
+	{
+		HandleOption handler;
+		const Options& options;
+		int flagChar;
+		bool expectArg = false;
+
+	public:
+		ParserState(HandleOption handler, const Options& options) noexcept
+			: handler(handler), options(options) {}
+		[[nodiscard]] bool ExpectingArg() const { return expectArg; }
+		[[nodiscard]] ParseCtrl Next(const std::string_view token);
+	};
+
+	class ArgParser
+	{
+		const std::string name;
+		Options options;
+		HandleOption handler;
+
+		[[nodiscard]] bool CheckParse(ArgParse::ParseErr err) const;
+
+	public:
+		explicit ArgParser(const std::string_view argv0, Options options, HandleOption&& handler) noexcept;
+
+		[[nodiscard]] const std::string_view GetName() const noexcept { return name; }
+		void DisplayError(const std::string_view message, bool helpPrompt = true) const;
+
+		template <typename V>
+		[[nodiscard]] bool Parse(V args)
+		{
+			ParserState state(handler, options);
+			for (auto arg : args)
+			{
+				ParseErr err = ParseErr::UNEXPECTED;
+				switch (state.Next(arg))
+				{
+				case ParseCtrl::CONTINUE:            continue;
+				case ParseCtrl::QUIT_EARLY:          err = ParseErr::OK; break;
+				case ParseCtrl::QUIT_ERR_UNKNOWN:    err = ParseErr::OPT_UNKNOWN; break;
+				case ParseCtrl::QUIT_ERR_UNEXPECTED: err = ParseErr::UNEXPECTED; break;
+				case ParseCtrl::QUIT_ERR_EXPECTARG:  err = ParseErr::ARG_EXPECTED; break;
+				case ParseCtrl::QUIT_ERR_INVALID:    err = ParseErr::ARG_INVALID; break;
+				case ParseCtrl::QUIT_ERR_RANGE:      err = ParseErr::ARG_RANGE; break;
+				}
+				if (!CheckParse(err))
+					return false;
+			}
+			return CheckParse(state.ExpectingArg() ? ParseErr::ARG_EXPECTED : ParseErr::OK);
+		}
+
+		[[nodiscard]] inline bool Parse(std::initializer_list<std::string_view> args)
+		{
+			return Parse<std::initializer_list<std::string_view>>(args);
+		}
+
+		[[nodiscard]] inline bool Parse(std::span<char*> args)
+		{
+			return Parse(args | std::views::transform([](char const* v){ return std::string_view(v); }));
+		}
+	};
+
+	[[nodiscard]] extern bool ReadParamFile(std::vector<std::string>& tokens, std::istream& file);
+}
+
+#endif//ARGPARSE_HPP

src/base64.cpp

@@ -1,123 +0,0 @@
-/* 
-   base64.cpp and base64.h
-
-   Copyright (C) 2004-2008 René Nyffenegger
-
-   This source code is provided 'as-is', without any express or implied
-   warranty. In no event will the author be held liable for any damages
-   arising from the use of this software.
-
-   Permission is granted to anyone to use this software for any purpose,
-   including commercial applications, and to alter it and redistribute it
-   freely, subject to the following restrictions:
-
-   1. The origin of this source code must not be misrepresented; you must not
-      claim that you wrote the original source code. If you use this source code
-      in a product, an acknowledgment in the product documentation would be
-      appreciated but is not required.
-
-   2. Altered source versions must be plainly marked as such, and must not be
-      misrepresented as being the original source code.
-
-   3. This notice may not be removed or altered from any source distribution.
-
-   René Nyffenegger rene.nyffenegger@adp-gmbh.ch
-
-*/
-
-#include "base64.h"
-#include <iostream>
-
-static const std::string base64_chars = 
-             "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
-             "abcdefghijklmnopqrstuvwxyz"
-             "0123456789+/";
-
-
-static inline bool is_base64(unsigned char c) {
-  return (isalnum(c) || (c == '+') || (c == '/'));
-}
-
-std::string base64_encode(unsigned char const* bytes_to_encode, unsigned int in_len) {
-  std::string ret;
-  int i = 0;
-  int j = 0;
-  unsigned char char_array_3[3];
-  unsigned char char_array_4[4];
-
-  while (in_len--) {
-    char_array_3[i++] = *(bytes_to_encode++);
-    if (i == 3) {
-      char_array_4[0] = (char_array_3[0] & 0xfc) >> 2;
-      char_array_4[1] = ((char_array_3[0] & 0x03) << 4) + ((char_array_3[1] & 0xf0) >> 4);
-      char_array_4[2] = ((char_array_3[1] & 0x0f) << 2) + ((char_array_3[2] & 0xc0) >> 6);
-      char_array_4[3] = char_array_3[2] & 0x3f;
-
-      for(i = 0; (i <4) ; i++)
-        ret += base64_chars[char_array_4[i]];
-      i = 0;
-    }
-  }
-
-  if (i)
-  {
-    for(j = i; j < 3; j++)
-      char_array_3[j] = '\0';
-
-    char_array_4[0] = (char_array_3[0] & 0xfc) >> 2;
-    char_array_4[1] = ((char_array_3[0] & 0x03) << 4) + ((char_array_3[1] & 0xf0) >> 4);
-    char_array_4[2] = ((char_array_3[1] & 0x0f) << 2) + ((char_array_3[2] & 0xc0) >> 6);
-    char_array_4[3] = char_array_3[2] & 0x3f;
-
-    for (j = 0; (j < i + 1); j++)
-      ret += base64_chars[char_array_4[j]];
-
-    while((i++ < 3))
-      ret += '=';
-
-  }
-
-  return ret;
-
-}
-
-std::string base64_decode(std::string const& encoded_string) {
-  int in_len = encoded_string.size();
-  int i = 0;
-  int j = 0;
-  int in_ = 0;
-  unsigned char char_array_4[4], char_array_3[3];
-  std::string ret;
-
-  while (in_len-- && ( encoded_string[in_] != '=') && is_base64(encoded_string[in_])) {
-    char_array_4[i++] = encoded_string[in_]; in_++;
-    if (i ==4) {
-      for (i = 0; i <4; i++)
-        char_array_4[i] = base64_chars.find(char_array_4[i]);
-
-      char_array_3[0] = (char_array_4[0] << 2) + ((char_array_4[1] & 0x30) >> 4);
-      char_array_3[1] = ((char_array_4[1] & 0xf) << 4) + ((char_array_4[2] & 0x3c) >> 2);
-      char_array_3[2] = ((char_array_4[2] & 0x3) << 6) + char_array_4[3];
-
-      for (i = 0; (i < 3); i++)
-        ret += char_array_3[i];
-      i = 0;
-    }
-  }
-
-  if (i) {
-    for (j = i; j <4; j++)
-      char_array_4[j] = 0;
-
-    for (j = 0; j <4; j++)
-      char_array_4[j] = base64_chars.find(char_array_4[j]);
-
-    char_array_3[0] = (char_array_4[0] << 2) + ((char_array_4[1] & 0x30) >> 4);
-    char_array_3[1] = ((char_array_4[1] & 0xf) << 4) + ((char_array_4[2] & 0x3c) >> 2);
-    char_array_3[2] = ((char_array_4[2] & 0x3) << 6) + char_array_4[3];
-
-    for (j = 0; (j < i - 1); j++) ret += char_array_3[j];
-  }
-
-  return ret;
-}

src/headerwriter.cpp

@@ -0,0 +1,93 @@
+/* headerwriter.cpp - Copyright (C) 2015-2024 a dinosaur (zlib, see COPYING.txt) */
+
+#include "headerwriter.hpp"
+
+
+template <typename T> static constexpr std::string_view DatType();
+template <> constexpr std::string_view DatType<uint8_t>() { return "unsigned char"; }
+template <> constexpr std::string_view DatType<uint16_t>() { return "unsigned short"; }
+template <> constexpr std::string_view DatType<uint32_t>() { return "unsigned int"; }
+
+void HeaderWriter::WriteSize(int width, int height)
+{
+	stream << std::endl;
+	WriteDefine(mName + "Width", width);
+	WriteDefine(mName + "Height", height);
+}
+
+void HeaderWriter::WriteCharacterMap(const std::span<uint16_t> charData)
+{
+	stream << std::endl;
+	WriteDefine(mName + "TilesLen", charData.size() * 2);
+	WriteSymbol(mName + "Tiles", DatType<uint16_t>(), charData.size());
+}
+
+void HeaderWriter::WriteCollision(const std::span<uint8_t> collisionData)
+{
+	stream << std::endl;
+	WriteDefine(mName + "CollisionLen", collisionData.size());
+	WriteSymbol(mName + "Collision", DatType<uint8_t>(), collisionData.size());
+}
+
+void HeaderWriter::WriteObjects(const std::span<uint32_t> objData)
+{
+	stream << std::endl;
+	WriteDefine(mName + "ObjCount", objData.size() / 3);
+	WriteDefine(mName + "ObjdatLen", objData.size() * sizeof(int));
+	WriteSymbol(mName + "Objdat", DatType<uint32_t>(), objData.size());
+}
+
+
+static std::string GuardName(const std::string_view name)
+{
+	auto upper = std::string(name);
+	for (auto& c: upper)
+		c = static_cast<char>(toupper(c));
+	return "TMX2GBA_" + upper;
+}
+
+
+void HeaderWriter::WriteGuardStart()
+{
+	const std::string guard = GuardName(mName);
+	stream << "#ifndef " << guard << std::endl;
+	stream << "#define " << guard << std::endl;
+}
+
+void HeaderWriter::WriteGuardEnd()
+{
+	const std::string guard = GuardName(mName);
+	stream << std::endl << "#endif//" << guard << std::endl;
+}
+
+
+HeaderWriter::~HeaderWriter()
+{
+	if (stream.is_open())
+	{
+		WriteGuardEnd();
+		stream.close();
+	}
+}
+
+
+bool HeaderWriter::Open(const std::filesystem::path& path, const std::string_view name)
+{
+	mName = name;
+	stream.open(path);
+	if (!stream.is_open())
+		return false;
+
+	WriteGuardStart();
+	return true;
+}
+
+void HeaderWriter::WriteDefine(const std::string_view name, const std::string_view value)
+{
+	stream << "#define " << name << " " << value << std::endl;
+}
+
+void HeaderWriter::WriteSymbol(const std::string_view name, const std::string_view type, std::size_t count)
+{
+	stream << "extern const " << type << " " << name << "[" << count << "];" << std::endl;
+}

src/headerwriter.hpp

@@ -0,0 +1,46 @@
+/* headerwriter.hpp - Copyright (C) 2015-2024 a dinosaur (zlib, see COPYING.txt) */
+
+#ifndef HEADERWRITER_HPP
+#define HEADERWRITER_HPP
+
+#include <cstddef>
+#include <cstdint>
+#include <string>
+#include <string_view>
+#include <span>
+#include <concepts>
+#include <fstream>
+#include <filesystem>
+
+template <typename T>
+concept NumericType = std::integral<T> || std::floating_point<T>;
+
+class HeaderWriter
+{
+	std::ofstream stream;
+	std::string mName;
+
+	void WriteGuardStart();
+	void WriteGuardEnd();
+
+public:
+	~HeaderWriter();
+
+	[[nodiscard]] bool Open(const std::filesystem::path& path, const std::string_view name);
+
+	void WriteDefine(const std::string_view name, const std::string_view value);
+	void WriteSymbol(const std::string_view name, const std::string_view type, std::size_t count);
+
+	template <NumericType T>
+	void WriteDefine(const std::string_view name, T value)
+	{
+		WriteDefine(name, std::to_string(value));
+	}
+
+	void WriteSize(int width, int height);
+	void WriteCharacterMap(const std::span<uint16_t> charData);
+	void WriteCollision(const std::span<uint8_t> collisionData);
+	void WriteObjects(const std::span<uint32_t> objData);
+};
+
+#endif//HEADERWRITER_HPP

src/swriter.cpp

@@ -0,0 +1,163 @@
+/* swwriter.cpp - Copyright (C) 2024 a dinosaur (zlib, see COPYING.txt) */
+
+#include "swriter.hpp"
+#include <type_traits>
+#include <limits>
+
+#define GNU_STYLE  0
+#define MASM_STYLE 1
+
+#define HEX_STYLE GNU_STYLE
+
+
+static inline constexpr char HexU(uint8_t h) { return "0123456789ABCDEF"[h >> 4]; }
+static inline constexpr char HexL(uint8_t l) { return "0123456789ABCDEF"[l & 15]; }
+
+#if HEX_STYLE == GNU_STYLE
+template <typename T> static void CHex(std::ostream& s, T x);
+template <> void CHex(std::ostream& s, uint8_t x)
+{
+	if (x >  9) s << "0x";
+	if (x > 15) s << HexU(x);
+	s << HexL(x);
+}
+template <> void CHex(std::ostream& s, uint16_t x)
+{
+	if (x >    9) s << "0x";
+	if (x > 4095) s << HexU(static_cast<uint8_t>(x >> 8));
+	if (x >  255) s << HexL(static_cast<uint8_t>(x >> 8));
+	if (x >   15) s << HexU(static_cast<uint8_t>(x));
+	s << HexL(static_cast<uint8_t>(x));
+}
+template <> void CHex(std::ostream& s, uint32_t x)
+{
+	if (x >         9) s << "0x";
+	if (x > 0xFFFFFFF) s << HexU(static_cast<uint8_t>(x >> 24));
+	if (x >  0xFFFFFF) s << HexL(static_cast<uint8_t>(x >> 24));
+	if (x >   0xFFFFF) s << HexU(static_cast<uint8_t>(x >> 16));
+	if (x >     65535) s << HexL(static_cast<uint8_t>(x >> 16));
+	if (x >      4095) s << HexU(static_cast<uint8_t>(x >>  8));
+	if (x >       255) s << HexL(static_cast<uint8_t>(x >>  8));
+	if (x >        15) s << HexU(static_cast<uint8_t>(x));
+	s << HexL(static_cast<uint8_t>(x));
+}
+#elif HEX_STYLE == MASM_STYLE
+template <typename T> static void MHex(std::ostream& s, T x);
+template <> void MHex(std::ostream& s, uint8_t x)
+{
+	if (x > 159) s << "0";
+	if (x >  15) s << HexU(x); else if (x > 9) s << "0";
+	s << HexL(x);
+	if (x >   9) s << "h";
+}
+template <> void MHex(std::ostream& s, uint16_t x)
+{
+	if (x > 40959) s << "0";
+	if (x >  4095) s << HexU(static_cast<uint8_t>(x >> 8)); else if (x > 2559) s << "0";
+	if (x >   255) s << HexL(static_cast<uint8_t>(x >> 8)); else if (x >  159) s << "0";
+	if (x >    15) s << HexU(static_cast<uint8_t>(x));      else if (x >    9) s << "0";
+	s << HexL(static_cast<uint8_t>(x));
+	if (x >     9) s << "h";
+}
+template <> void MHex(std::ostream& s, uint32_t x)
+{
+	if (x > 0x9FFFFFFF) s << "0";
+	if (x >  0xFFFFFFF) s << HexU(static_cast<uint8_t>(x >> 24)); else if (x > 0x9FFFFFF) s << "0";
+	if (x >   0xFFFFFF) s << HexL(static_cast<uint8_t>(x >> 24)); else if (x >  0x9FFFFF) s << "0";
+	if (x >    0xFFFFF) s << HexU(static_cast<uint8_t>(x >> 16)); else if (x >    655359) s << "0";
+	if (x >      65535) s << HexL(static_cast<uint8_t>(x >> 16)); else if (x >     40959) s << "0";
+	if (x >       4095) s << HexU(static_cast<uint8_t>(x >>  8)); else if (x >      2559) s << "0";
+	if (x >        255) s << HexL(static_cast<uint8_t>(x >>  8)); else if (x >       159) s << "0";
+	if (x >         15) s << HexU(static_cast<uint8_t>(x));       else if (x >         9) s << "0";
+	s << HexL(static_cast<uint8_t>(x));
+	if (x >          9) s << "h";
+}
+#else
+# error "Unknown hex style"
+#endif
+
+
+template <typename T> static constexpr const std::string_view DataType();
+template <> constexpr const std::string_view DataType<uint8_t>()  { return ".byte"; }
+template <> constexpr const std::string_view DataType<uint16_t>() { return ".hword"; }
+template <> constexpr const std::string_view DataType<uint32_t>() { return ".word"; }
+
+template <typename I>
+static void WriteArrayDetail(std::ostream& s, const I beg, const I end, int perCol)
+{
+	typedef typename std::iterator_traits<I>::value_type Element;
+
+	int col = 0;
+	for (auto it = beg;;)
+	{
+		if (col == 0)
+			s << "\t" << DataType<Element>() << " ";
+
+		const Element e = *it;
+#if HEX_STYLE == MASM_STYLE
+		MHex(s, e);
+#elif HEX_STYLE == GNU_STYLE
+		CHex(s, e);
+#endif
+
+		if (++it == end)
+			break;
+
+		if (++col < perCol)
+		{
+			s << ",";
+		}
+		else
+		{
+			s << std::endl;
+			col = 0;
+		}
+	}
+	s << std::endl;
+}
+
+
+void SWriter::WriteSymbol(const std::string_view suffix)
+{
+	if (writes++ != 0)
+		stream << std::endl;
+	stream << "\t.section .rodata" << std::endl;
+	stream << "\t.align 2" << std::endl;
+	stream << "\t.global " << mName << suffix << std::endl;
+	stream << "\t.hidden " << mName << suffix << std::endl;
+	stream << mName << suffix << ":" << std::endl;
+}
+
+void SWriter::WriteArray(const std::string_view suffix, std::span<uint8_t> data, int numCols)
+{
+	WriteSymbol(suffix);
+	WriteArrayDetail(stream, data.begin(), data.end(), numCols);
+}
+
+void SWriter::WriteArray(const std::string_view suffix, std::span<uint16_t> data, int numCols)
+{
+	WriteSymbol(suffix);
+	WriteArrayDetail(stream, data.begin(), data.end(), numCols);
+}
+
+void SWriter::WriteArray(const std::string_view suffix, std::span<uint32_t> data, int numCols)
+{
+	WriteSymbol(suffix);
+	WriteArrayDetail(stream, data.begin(), data.end(), numCols);
+}
+
+
+SWriter::~SWriter()
+{
+	if (stream.is_open())
+	{
+		stream.close();
+	}
+}
+
+bool SWriter::Open(const std::filesystem::path& path, const std::string_view name)
+{
+	mName = name;
+	stream.open(path);
+	return stream.is_open();
+}

src/swriter.hpp

@@ -0,0 +1,32 @@
+/* swwriter.hpp - Copyright (C) 2024 a dinosaur (zlib, see COPYING.txt) */
+
+#ifndef SWRITER_HPP
+#define SWRITER_HPP
+
+#include <cstddef>
+#include <cstdint>
+#include <string>
+#include <string_view>
+#include <span>
+#include <fstream>
+#include <filesystem>
+
+class SWriter
+{
+	std::ofstream stream;
+	std::string mName;
+	int writes = 0;
+
+	void WriteSymbol(const std::string_view suffix);
+
+public:
+	~SWriter();
+
+	bool Open(const std::filesystem::path& path, const std::string_view name);
+
+	void WriteArray(const std::string_view suffix, std::span<uint8_t> data, int numCols = 16);
+	void WriteArray(const std::string_view suffix, std::span<uint16_t> data, int numCols = 16);
+	void WriteArray(const std::string_view suffix, std::span<uint32_t> data, int numCols = 16);
+};
+
+#endif//SWRITER_HPP

src/tmx2gba.cpp

@@ -1,165 +1,116 @@
-/* tmx2gba.cpp
+/* tmx2gba.cpp - Copyright (C) 2015-2024 a dinosaur (zlib, see COPYING.txt) */
 
-  Copyright (C) 2015-2018 Nicholas Curtis (a dinosaur)
+#include "argparse.hpp"
-
+#include "tmxreader.hpp"
-  This software is provided 'as-is', without any express or implied
+#include "tmxlayer.hpp"
-  warranty.  In no event will the authors be held liable for any damages
+#include "tmxobject.hpp"
-  arising from the use of this software.
+#include "headerwriter.hpp"
-
+#include "swriter.hpp"
-  Permission is granted to anyone to use this software for any purpose,
-  including commercial applications, and to alter it and redistribute it
-  freely, subject to the following restrictions:
-
-  1. The origin of this software must not be misrepresented; you must not
-     claim that you wrote the original software. If you use this software
-     in a product, an acknowledgment in the product documentation would be
-     appreciated but is not required.
-  2. Altered source versions must be plainly marked as such, and must not be
-     misrepresented as being the original software.
-  3. This notice may not be removed or altered from any source distribution.
-
-*/
-
-#include "tmxreader.h"
-#include "tmxlayer.h"
-#include "tmxobject.h"
 #include <iostream>
-#include <iomanip>
-#include <fstream>
-#include <vector>
 #include <map>
-#include <string>
-#include <cstdint>
 #include <algorithm>
 
-#ifdef _MSC_VER
-  #include <XGetopt.h>
-#else
-  #include <getopt.h>
-#endif
 
-using std::cout;
+static const char* versionStr = "tmx2gba version 0.3, (c) 2015-2022 a dinosaur";
-using std::cerr;
-using std::endl;
-using std::string;
-using std::vector;
-using std::map;
-using std::ifstream;
-using std::ofstream;
-using std::stoi;
-using std::min;
-using std::max;
-using std::ios;
 
-static const string g_strUsage = "Usage: tmx2gba [-h] [-f file] [-r offset] [-lyc name] [-p 0-15] [-m name;id] <-i inpath> <-o outpath>";
+struct Arguments
-static const string g_strHelpShort = "Run 'tmx2gba -h' to view all available options.";
-static const string g_strHelpFull = R"(
--h ------------ Display this help & command info.
--l <name> ----- Name of layer to use (default first layer in TMX).
--y <name> ----- Layer for palette mappings.
--c <name> ----- Output a separate 8bit collision map of the specified layer.
--r <offset> --- Offset tile indices (default 0).
--p <0-15> ----- Select which palette to use for 4-bit tilesets.
--m <name;id> -- Map an object name to an ID, will enable object exports.
--i <path> ----- Path to input TMX file.
--o <path> ----- Path to output files.
--f <file> ----- Specify a file to use for flags, will override any options specified on the command line.)";
-
-struct SParams
 {
-	bool help = false;
+	bool help = false, showVersion = false;
-	string inPath, outPath;
+	std::string inPath, outPath;
-	string layer, collisionlay, paletteLay;
+	std::string layer, collisionlay, paletteLay;
-	string flagFile;
+	std::string flagFile;
 	int offset = 0;
 	int palette = 0;
-	vector<string> objMappings;
+	std::vector<std::string> objMappings;
-	bool objExport = false;
 };
 
-void GetoptClear ()
+using ArgParse::Option;
-{
-	optarg = nullptr;
-	optind = 0;
-#ifdef _MSC_VER
-	next = nullptr;
-#endif
-}
 
-void ParseArgs ( int argc, char** argv, SParams* params )
+static const ArgParse::Options options =
 {
-	char cOption;
+	Option::Optional('h', nullptr,   "Display this help & command info"),
-	GetoptClear ();
+	Option::Optional('v', nullptr,   "Display version & quit"),
-	while ( ( cOption = (char)getopt ( argc, argv, "hr:l:c:p:y:m:i:o:f:" ) ) > 0 )
+	Option::Optional('l', "name",    "Name of layer to use (default first layer in TMX)"),
+	Option::Optional('y', "name",    "Layer for palette mappings"),
+	Option::Optional('c', "name",    "Output a separate 8bit collision map of the specified layer"),
+	Option::Optional('r', "offset",  "Offset tile indices (default 0)"),
+	Option::Optional('p', "0-15",    "Select which palette to use for 4-bit tilesets"),
+	Option::Optional('m', "name;id", "Map an object name to an ID, will enable object exports"),
+	Option::Required('i', "inpath",  "Path to input TMX file"),
+	Option::Required('o', "outpath", "Path to output files"),
+	Option::Optional('f', "file",    "Specify a file to use for flags, will override any options"
+	                                 " specified on the command line")
+};
+
+bool ParseArgs(int argc, char** argv, Arguments& params)
+{
+	auto parser = ArgParse::ArgParser(argv[0], options, [&](int opt, const std::string_view arg)
+		-> ArgParse::ParseCtrl
 	{
-		switch ( cOption )
+		using ArgParse::ParseCtrl;
+		try
 		{
-		case ( 'h' ):
+			switch (opt)
-			params->help = true;
+			{
-			return;
+			case 'h': params.help = true;        return ParseCtrl::QUIT_EARLY;
+			case 'v': params.showVersion = true; return ParseCtrl::QUIT_EARLY;
+			case 'l': params.layer = arg;        return ParseCtrl::CONTINUE;
+			case 'c': params.collisionlay = arg; return ParseCtrl::CONTINUE;
+			case 'y': params.paletteLay = arg;   return ParseCtrl::CONTINUE;
+			case 'r': params.offset = std::stoi(std::string(arg));  return ParseCtrl::CONTINUE;
+			case 'p': params.palette = std::stoi(std::string(arg)); return ParseCtrl::CONTINUE;
+			case 'm': params.objMappings.emplace_back(arg);         return ParseCtrl::CONTINUE;
+			case 'i': params.inPath = arg;       return ParseCtrl::CONTINUE;
+			case 'o': params.outPath = arg;      return ParseCtrl::CONTINUE;
+			case 'f': params.flagFile = arg;     return ParseCtrl::CONTINUE;
 
-		case ( 'l' ):
+			default: return ParseCtrl::QUIT_ERR_UNKNOWN;
-			params->layer = optarg;
+			}
-			break;
-
-		case ( 'c' ):
-			params->collisionlay = optarg;
-			break;
-
-		case ( 'y' ):
-			params->paletteLay = optarg;
-			break;
-
-		case ( 'r' ):
-			params->offset = stoi ( optarg );
-			break;
-
-		case ( 'p' ):
-			params->palette = stoi ( optarg );
-			break;
-
-		case ( 'm' ):
-			params->objExport = true;
-			params->objMappings.push_back ( optarg );
-			break;
-
-		case ( 'i' ):
-			params->inPath = optarg;
-			break;
-
-		case ( 'o' ):
-			params->outPath = optarg;
-			break;
-
-		case ( 'f' ):
-			params->flagFile = optarg;
-			break;
-
-		default:
-			break;
 		}
-	}
+		catch (std::invalid_argument const&) { return ParseCtrl::QUIT_ERR_INVALID; }
-}
+		catch (std::out_of_range const&) { return ParseCtrl::QUIT_ERR_RANGE; }
+	});
 
-bool CheckArgs ( const SParams* params )
+	if (!parser.Parse(std::span(argv + 1, argc - 1)))
-{
+		return false;
-	// Check my paranoia.
+
-	if ( params->inPath.empty () )
+	if (params.help || params.showVersion)
+		return true;
+
+	if (!params.flagFile.empty())
 	{
-		cerr << "No input file specified." << endl;
+		std::ifstream paramFile(params.flagFile);
-		cout << g_strUsage << endl << g_strHelpShort << endl;
+		if (!paramFile.is_open())
+		{
+			std::cerr << "Failed to open param file." << std::endl;
+			return false;
+		}
+
+		std::vector<std::string> tokens;
+		if (!ArgParse::ReadParamFile(tokens, paramFile))
+		{
+			std::cerr << "Failed to read param file: Unterminated quote string." << std::endl;
+			return false;
+		}
+
+		if (!parser.Parse(tokens))
+			return false;
+	}
+
+	// Check my paranoia
+	if (params.inPath.empty())
+	{
+		parser.DisplayError("No input file specified.");
 		return false;
 	}
-	if ( params->outPath.empty () )
+	if (params.outPath.empty())
 	{
-		cerr << "No output file specified." << endl;
+		parser.DisplayError("No output file specified.");
-		cout << g_strUsage << endl << g_strHelpShort << endl;
 		return false;
 	}
-	if ( params->palette < 0 || params->palette > 15 )
+	if (params.palette < 0 || params.palette > 15)
 	{
-		cerr << "Invalid palette index." << endl;
+		parser.DisplayError("Invalid palette index.");
-		cout << g_strUsage << endl << g_strHelpShort << endl;
 		return false;
 	}
 
@@ -167,343 +118,183 @@ bool CheckArgs ( const SParams* params )
 }
 
 
-template <typename T>
+int main(int argc, char** argv)
-void WriteArray ( ofstream& a_fout, const vector<T>& a_dat, int a_perCol = 16 )
 {
-	const int w = sizeof(T) * 2;
+	Arguments p;
-	int col = 0;
+	if (!ParseArgs(argc, argv, p))
-
+		return 1;
-	string datType = "ERR";
+	if (p.help)
-	if ( sizeof(T) == 1 )
 	{
-		datType = ".byte";
+		options.ShowHelpUsage(argv[0], std::cout);
+		return 0;
 	}
-	else
+	if (p.showVersion)
-	if ( sizeof(T) == 2 )
 	{
-		datType = ".hword";
+		std::cout << versionStr << std::endl;
-	}
-	else
-	if ( sizeof(T) == 4 )
-	{
-		datType = ".word";
-	}
-
-	a_fout.setf ( ios::hex, ios::basefield );
-	a_fout.setf ( ios::showbase );
-
-	size_t i = 0;
-	for ( T element : a_dat )
-	{
-		if ( col == 0 )
-		{
-			a_fout << "\t" << datType << " ";
-		}
-
-		a_fout << std::hex << (int)element;
-
-		if ( i < a_dat.size () - 1 )
-		{
-			if ( ++col < a_perCol )
-			{
-				a_fout << ",";
-			}
-			else
-			{
-				a_fout << "" << endl;
-				col = 0;
-			}
-		}
-
-		++i;
-	}
-}
-
-int main ( int argc, char** argv )
-{
-	SParams params;
-	ParseArgs ( argc, argv, &params );
-
-	if ( params.help )
-	{
-		cout << g_strUsage << endl << g_strHelpFull << endl;
 		return 0;
 	}
 
-	if ( params.flagFile.length () != 0 )
+	// Object mappings
+	std::map<std::string, uint32_t> objMapping;
+	if (!p.objMappings.empty())
 	{
-		ifstream paramFile ( params.flagFile );
+		for (const auto& objToken : p.objMappings)
-		if ( !paramFile.is_open () )
 		{
-			cerr << "Failed to open param file." << endl;
+			auto splitter = objToken.find_last_of(';');
-			return -1;
+			if (splitter == std::string::npos)
-		}
-		
-		vector<string> fileArgTokens;
-		fileArgTokens.push_back ( "auu~~" );
-		bool carry = false;
-		string token;
-		while ( !paramFile.eof () )
-		{
-			if ( carry )
 			{
-				string tmp;
+				std::cerr << "Malformed mapping (missing a splitter)." << std::endl;
-				paramFile >> tmp;
+				return 1;
-				token += " ";
-				token += tmp;
-			}
-			else
-			{
-				token.clear ();
-				paramFile >> token;
-			}
-
-			if ( token == "" )
-			{
-				continue;
-			}
-
-			bool qFr = token[0] == '"';
-			bool qBk = token[token.length () - 1] == '"';
-			if ( qFr && qBk )
-			{
-				fileArgTokens.push_back ( token.substr ( 1, token.length () - 2 ) );
-			}
-			else
-			if ( qFr )
-			{
-				fileArgTokens.push_back ( token.substr ( 1, token.length () - 1 ) );
-				carry = true;
-			}
-			else
-			if ( qBk )
-			{
-				fileArgTokens.push_back ( token.substr ( 0, token.length () - 1 ) );
-				carry = false;
-			}
-			else
-			{
-				fileArgTokens.push_back ( token );
-			}
-		}
-
-		vector<const char*> fileArgs;
-		fileArgs.reserve ( fileArgTokens.size () );
-		for ( auto& token : fileArgTokens )
-		{
-			fileArgs.push_back ( token.c_str () );
-		}
-
-		ParseArgs ( fileArgs.size (), (char**)fileArgs.data (), &params );
-	}
-
-	if ( !CheckArgs ( &params ) )
-	{
-		return -1;
-	}
-
-	// Object mappings.
-	map<string, uint32_t> objMapping;
-	if ( params.objExport )
-	{
-		for ( auto token : params.objMappings )
-		{
-			int splitter = token.find_last_of ( ';' );
-			if ( splitter == -1 )
-			{
-				cerr << "Malformed mapping (missing a splitter)." << endl;
-				return -1;
 			}
 
 			try
 			{
-				string name = token.substr ( 0, splitter );
+				std::string name = objToken.substr(0, splitter);
-				int id = stoi ( token.substr ( splitter + 1 ) );
+				int id = std::stoi(objToken.substr(splitter + 1));
 
 				objMapping[name] = id;
 			}
-			catch ( std::exception )
+			catch (std::exception&)
 			{
-				cerr << "Malformed mapping, make sure id is numeric." << endl;
+				std::cerr << "Malformed mapping, make sure id is numeric." << std::endl;
 			}
 		}
 	}
 
-	// Open & read input file.
+	// Open & read input file
-	CTmxReader tmx;
+	TmxReader tmx;
-	ifstream fin ( params.inPath );
+	std::ifstream fin(p.inPath);
-	if ( !fin.is_open () )
+	if (!fin.is_open())
 	{
-		cerr << "Failed to open input file." << endl;
+		std::cerr << "Failed to open input file." << std::endl;
-		return -1;
+		return 1;
 	}
-	tmx.Open ( fin );
+	tmx.Open(fin);
 
-	// Get layers.
+	// Get layers
-	if ( tmx.GetLayerCount () == 0 )
+	if (tmx.GetLayerCount() == 0)
 	{
-		cerr << "No layers found." << endl; 
+		std::cerr << "No layers found." << std::endl;
-		return -1;
+		return 1;
 	}
-	const CTmxLayer* pLayerGfx = params.layer.empty () ? tmx.GetLayer ( 0 ) : tmx.GetLayer ( params.layer );
+	const TmxLayer* layerGfx = p.layer.empty()
-	const CTmxLayer* pLayerCls = params.collisionlay.empty () ? nullptr : tmx.GetLayer ( params.collisionlay );
+		? tmx.GetLayer(0)
-	const CTmxLayer* pLayerPal = params.paletteLay.empty () ? nullptr : tmx.GetLayer ( params.paletteLay );
+		: tmx.GetLayer(p.layer);
+	const TmxLayer* layerCls = p.collisionlay.empty()
+		? nullptr
+		: tmx.GetLayer(p.collisionlay);
+	const TmxLayer* layerPal = p.paletteLay.empty()
+		? nullptr
+		: tmx.GetLayer(p.paletteLay);
 
-	if ( pLayerGfx == nullptr )
+	if (layerGfx == nullptr)
 	{
-		cerr << "Input layer not found." << endl;
+		std::cerr << "Input layer not found." << std::endl;
-		return -1;
+		return 1;
 	}
 
-	// Open output files.
+	// Get name from file
-	ofstream foutS ( params.outPath + ".s" );
+	//TODO: properly sanitise
-	ofstream foutH ( params.outPath + ".h" );
+	int slashPos = std::max(
-	if ( !foutS.is_open () || !foutH.is_open () )
+		static_cast<int>(p.outPath.find_last_of('/')),
+		static_cast<int>(p.outPath.find_last_of('\\')));
+	std::string name = p.outPath;
+	if (slashPos != -1)
+		name = name.substr(slashPos + 1);
+
+	// Open output files
+	SWriter outS; HeaderWriter outH;
+	if (!outS.Open(p.outPath + ".s", name))
 	{
-		cerr << "Failed to create output file(s).";
+		std::cerr << "Failed to create output file \"" << p.outPath << ".s\".";
-		return -1;
+		return 1;
+	}
+	if (!outH.Open(p.outPath + ".h", name))
+	{
+		std::cerr << "Failed to create output file \"" << p.outPath << ".h\".";
+		return 1;
 	}
 
-	int slashPos = max ( (int)params.outPath.find_last_of ( '/' ), (int)params.outPath.find_last_of ( '\\' ) );
+	// Convert to GBA-friendly charmap data
-	string name = params.outPath;
+	const uint32_t* gfxTiles = layerGfx->GetData();
-	if ( slashPos != -1 )
+	const uint32_t* palTiles = (layerPal == nullptr) ? nullptr : layerPal->GetData();
+	std::vector<uint16_t> charDat;
+	const size_t numTiles = static_cast<size_t>(layerGfx->GetWidth()) * static_cast<size_t>(layerGfx->GetHeight());
+	charDat.reserve(numTiles);
+	for (size_t i = 0; i < numTiles; ++i)
 	{
-		name = name.substr ( slashPos + 1 );
+		uint32_t read = (*gfxTiles++);
-	}
 
-	// Write header guards.
+		uint16_t tile = std::max(0, static_cast<int>(tmx.LidFromGid(read & ~TmxLayer::FLIP_MASK)) + p.offset);
-	string guard = "__TMX2GBA_" + name + "__";
+		uint8_t flags = 0x0;
-	for ( auto& c: guard ) c = (char)toupper ( (int)c );
-	foutH << "#ifndef " << guard << endl;
-	foutH << "#define " << guard << endl;
-	foutH << endl;
-	foutH << "#define " << name << "Width " << tmx.GetWidth () << endl;
-	foutH << "#define " << name << "Height " << tmx.GetHeight () << endl;
-	foutH << endl;
-
-	// Convert to GBA-friendly charmap data.
-	const uint32_t* pRead		= pLayerGfx->GetData ();
-	const uint32_t* pPalRead	= pLayerPal == nullptr ? nullptr : pLayerPal->GetData ();
-	vector<uint16_t> vucCharDat;
-	vucCharDat.reserve ( pLayerGfx->GetWidth () * pLayerGfx->GetHeight () );
-	for ( size_t i = 0; i < size_t(pLayerGfx->GetWidth () * pLayerGfx->GetHeight ()); ++i )
-	{
-		uint32_t uiRead = (*pRead++);
-
-		uint16_t usTile	= (uint16_t)max<int32_t> ( 0, tmx.LidFromGid ( uiRead & ~FLIP_MASK ) + params.offset );
-		uint8_t ucFlags	= 0x0;
 
 		// Get flipped!
-		ucFlags |= ( uiRead & FLIP_HORZ ) ? 0x4 : 0x0;
+		flags |= (read & TmxLayer::FLIP_HORZ) ? 0x4 : 0x0;
-		ucFlags |= ( uiRead & FLIP_VERT ) ? 0x8 : 0x0;
+		flags |= (read & TmxLayer::FLIP_VERT) ? 0x8 : 0x0;
 
-		// Determine palette ID.
+		// Determine palette ID
-		uint32_t uiIndex = 0;
+		uint32_t idx = 0;
-		if ( pPalRead != nullptr )
+		if (palTiles != nullptr)
-		{
+			idx = tmx.LidFromGid((*palTiles++) & ~TmxLayer::FLIP_MASK);
-			uiIndex = tmx.LidFromGid ( (*pPalRead++) & ~FLIP_MASK );
+		if (idx == 0)
-		}
+			idx = p.palette + 1;
-		if ( uiIndex == 0 )
+		flags |= static_cast<uint8_t>(idx - 1) << 4;
-		{
-			uiIndex = params.palette + 1;
-		}
-		ucFlags |= (uint8_t)(uiIndex - 1) << 4;
 
-		vucCharDat.push_back ( usTile | ( uint16_t(ucFlags) << 8 ) );
+		charDat.push_back(tile | (static_cast<uint16_t>(flags) << 8));
 	}
 
-	// Save out charmap.
+	// Write out charmap
-	foutH << "#define " << name << "TilesLen " << vucCharDat.size () * 2 << endl;
+	outH.WriteSize(tmx.GetWidth(), tmx.GetHeight());
-	foutH << "extern const unsigned short " << name << "Tiles[" << vucCharDat.size () << "];" << endl;
+	outH.WriteCharacterMap(charDat);
-	foutH << endl;
+	outS.WriteArray("Tiles", charDat);
 
-	foutS << "\t.section .rodata" << endl;
+	// Convert collision map & write it out
-	foutS << "\t.align 2" << endl;
+	if (layerCls != nullptr)
-	foutS << "\t.global " << name << "Tiles" << endl;
-	foutS << "\t.hidden " << name << "Tiles" << endl;
-	foutS << name << "Tiles" << ":" << endl;
-	WriteArray<uint16_t> ( foutS, vucCharDat );
-	foutS << endl;
-
-	// Convert collision map & save it out.
-	if ( pLayerCls != nullptr )
 	{
-		vector<uint8_t> vucCollisionDat;
+		std::vector<uint8_t> collisionDat;
-		vucCollisionDat.reserve ( pLayerCls->GetWidth () * pLayerCls->GetHeight () );
+		collisionDat.reserve(layerCls->GetWidth() * layerCls->GetHeight());
 
-		pRead = pLayerCls->GetData ();
+		gfxTiles = layerCls->GetData();
-		for ( int i = 0; i < pLayerCls->GetWidth () * pLayerCls->GetHeight (); ++i )
+		for (int i = 0; i < layerCls->GetWidth() * layerCls->GetHeight(); ++i)
 		{
-			uint8_t ucTile = (uint8_t)tmx.LidFromGid ( (*pRead++) & ~FLIP_MASK );
+			uint8_t ucTile = static_cast<uint8_t>(tmx.LidFromGid((*gfxTiles++) & ~TmxLayer::FLIP_MASK));
-			vucCollisionDat.push_back ( ucTile );
+			collisionDat.push_back(ucTile);
 		}
 
-		// Try to nicely append "_collision" to the output name.
+		// Try to nicely append "_collision" to the output name
-		string strPath;
+		std::string path;
-		size_t extPos = params.outPath.find_last_of ( '.' );
+		size_t extPos = p.outPath.find_last_of('.');
-		if ( extPos != string::npos )
+		if (extPos != std::string::npos)
-		{
+			path = p.outPath.insert(extPos, "_collision");
-			strPath = params.outPath.insert ( extPos, "_collision" );
-		}
 		else
-		{
+			path = p.outPath + "_collision";
-			strPath = params.outPath + "_collision";
-		}
 
-		// Save it out.
+		// Write collision
-		foutH << "#define " << name << "CollisionLen " << vucCollisionDat.size () << endl;
+		outH.WriteCollision(collisionDat);
-		foutH << "extern const unsigned char " << name << "Collision[" << vucCollisionDat.size () << "];" << endl;
+		outS.WriteArray("Collision", collisionDat, 32);
-		foutH << endl;
-
-		foutS << endl;
-		foutS << "\t.section .rodata" << endl;
-		foutS << "\t.align 2" << endl;
-		foutS << "\t.global " << name << "Collision" << endl;
-		foutS << "\t.hidden " << name << "Collision" << endl;
-		foutS << name << "Collision" << ":" << endl;
-		WriteArray<uint8_t> ( foutS, vucCollisionDat );
-		foutS << endl;
 	}
 
-	if ( params.objExport )
+	if (!p.objMappings.empty())
 	{
-		vector<uint32_t> objDat;
+		std::vector<uint32_t> objDat;
-		for ( int i = 0; i < tmx.GetObjectCount (); ++i )
+		for (size_t i = 0; i < tmx.GetObjectCount(); ++i)
 		{
-			auto obj = tmx.GetObject ( i );
+			auto obj = tmx.GetObject(i);
-			auto it = objMapping.find ( obj->GetName () );
+			auto it = objMapping.find(obj->GetName());
-			if ( it == objMapping.end () )
+			if (it == objMapping.end())
-			{
 				continue;
-			}
 
 			float x, y;
-			obj->GetPos ( &x, &y );
+			obj->GetPos(x, y);
-			objDat.push_back ( it->second );
+			objDat.push_back(it->second);
-			objDat.push_back ( (int)( x * 256.0f ) );
+			objDat.push_back(static_cast<int>(x * 256.0f));
-			objDat.push_back ( (int)( y * 256.0f ) );
+			objDat.push_back(static_cast<int>(y * 256.0f));
 		}
 
-		// Save it out.
+		// Write objects
-		foutH << "#define " << name << "ObjCount " << objDat.size () / 3 << endl;
+		outH.WriteObjects(objDat);
-		foutH << "#define " << name << "ObjdatLen " << objDat.size () * sizeof(int) << endl;
+		outS.WriteArray("Objdat", objDat);
-		foutH << "extern const unsigned int " << name << "Objdat[" << objDat.size () << "];" << endl;
-		foutH << endl;
-
-		foutS << endl;
-		foutS << "\t.section .rodata" << endl;
-		foutS << "\t.align 2" << endl;
-		foutS << "\t.global " << name << "Objdat" << endl;
-		foutS << "\t.hidden " << name << "Objdat" << endl;
-		foutS << name << "Objdat" << ":" << endl;
-		WriteArray<uint32_t> ( foutS, objDat );
-		foutS << endl;
 	}
 
-	foutH << "#endif//" << guard << endl;
-
-	foutH.close ();
-	foutS.close ();
-
 	return 0;
 }

src/tmxlayer.cpp

@@ -1,68 +0,0 @@
-/* tmxlayer.cpp
-
-  Copyright (C) 2015 Nicholas Curtis
-
-  This software is provided 'as-is', without any express or implied
-  warranty.  In no event will the authors be held liable for any damages
-  arising from the use of this software.
-
-  Permission is granted to anyone to use this software for any purpose,
-  including commercial applications, and to alter it and redistribute it
-  freely, subject to the following restrictions:
-
-  1. The origin of this software must not be misrepresented; you must not
-     claim that you wrote the original software. If you use this software
-     in a product, an acknowledgment in the product documentation would be
-     appreciated but is not required.
-  2. Altered source versions must be plainly marked as such, and must not be
-     misrepresented as being the original software.
-  3. This notice may not be removed or altered from any source distribution.
-
-*/
-
-#include "tmxlayer.h"
-
-
-CTmxLayer::CTmxLayer () :
-	m_iWidth ( 0 ),
-	m_iHeight ( 0 ),
-	m_strName ( "" ),
-	m_pTileDat ( nullptr )
-{
-
-}
-
-CTmxLayer::CTmxLayer ( int a_iWidth, int a_iHeight, const char* a_szName, uint32_t* a_pTileDat ) :
-	m_iWidth ( a_iWidth ),
-	m_iHeight ( a_iHeight ),
-	m_strName ( a_szName ),
-	m_pTileDat ( a_pTileDat )
-{
-
-}
-
-CTmxLayer::~CTmxLayer ()
-{
-	delete[] m_pTileDat;
-}
-
-
-const std::string& CTmxLayer::GetName () const
-{
-	return m_strName;
-}
-
-int CTmxLayer::GetWidth () const
-{
-	return m_iWidth;
-}
-
-int CTmxLayer::GetHeight () const
-{
-	return m_iHeight;
-}
-
-const uint32_t* CTmxLayer::GetData () const
-{
-	return m_pTileDat;
-}

src/tmxlayer.h

@@ -1,31 +0,0 @@
-#ifndef __TMXLAYER_H__
-#define __TMXLAYER_H__
-
-#include <string>
-#include <cstdint>
-
-const uint32_t FLIP_HORZ = 0x80000000;
-const uint32_t FLIP_VERT = 0x40000000;
-const uint32_t FLIP_DIAG = 0x20000000;
-const uint32_t FLIP_MASK = 0xE0000000;
-
-class CTmxLayer
-{
-public:
-	CTmxLayer ();
-	CTmxLayer ( int a_iWidth, int a_iHeight, const char* a_szName, uint32_t* a_pTileDat );
-	~CTmxLayer ();
-
-	const			std::string& GetName () const;
-	int				GetWidth	() const;
-	int				GetHeight	() const;
-	const uint32_t*	GetData		() const;
-
-private:
-	std::string m_strName;
-	int m_iWidth, m_iHeight;
-	uint32_t* m_pTileDat;
-
-};
-
-#endif//__TMXLAYER_H__

src/tmxlayer.hpp

@@ -0,0 +1,34 @@
+/* tmxlayer.hpp - Copyright (C) 2015-2022 a dinosaur (zlib, see COPYING.txt) */
+
+#ifndef TMXLAYER_HPP
+#define TMXLAYER_HPP
+
+#include <string>
+#include <cstdint>
+#include <utility>
+
+class TmxLayer
+{
+public:
+	static constexpr uint32_t FLIP_HORZ = 0x80000000;
+	static constexpr uint32_t FLIP_VERT = 0x40000000;
+	static constexpr uint32_t FLIP_DIAG = 0x20000000;
+	static constexpr uint32_t FLIP_MASK = 0xE0000000;
+
+	TmxLayer() : mWidth(0), mHeight(0), mTileDat(nullptr) {}
+	TmxLayer(int aWidth, int aHeight, std::string aName, uint32_t* aTileDat)
+		: mName(std::move(aName)), mWidth(aWidth), mHeight(aHeight), mTileDat(aTileDat) {}
+	inline ~TmxLayer() { delete[] mTileDat; }
+
+	constexpr const std::string& GetName() const { return mName; }
+	constexpr int GetWidth() const { return mWidth; }
+	constexpr int GetHeight() const { return mHeight; }
+	constexpr const uint32_t* GetData() const { return mTileDat; }
+
+private:
+	std::string mName;
+	int mWidth, mHeight;
+	uint32_t* mTileDat;
+};
+
+#endif//TMXLAYER_HPP

src/tmxobject.cpp

@@ -1,61 +0,0 @@
-/* tmxobject.cpp
-
-  Copyright (C) 2015 Nicholas Curtis
-
-  This software is provided 'as-is', without any express or implied
-  warranty.  In no event will the authors be held liable for any damages
-  arising from the use of this software.
-
-  Permission is granted to anyone to use this software for any purpose,
-  including commercial applications, and to alter it and redistribute it
-  freely, subject to the following restrictions:
-
-  1. The origin of this software must not be misrepresented; you must not
-     claim that you wrote the original software. If you use this software
-     in a product, an acknowledgment in the product documentation would be
-     appreciated but is not required.
-  2. Altered source versions must be plainly marked as such, and must not be
-     misrepresented as being the original software.
-  3. This notice may not be removed or altered from any source distribution.
-
-*/
-
-#include "tmxobject.h"
-
-CTmxObject::CTmxObject () :
-	m_name ( "" ),
-	m_x ( 0.0f ), m_y ( 0.0f )
-{
-
-}
-
-CTmxObject::CTmxObject ( const std::string& a_name, float a_x, float a_y ) :
-	m_name ( a_name ),
-	m_x ( a_x ), m_y ( a_y )
-{
-
-}
-
-CTmxObject::~CTmxObject ()
-{
-
-}
-
-
-const std::string& CTmxObject::GetName () const
-{
-	return m_name;
-}
-
-void CTmxObject::GetPos ( float* a_outX, float* a_outY ) const
-{
-	if ( a_outX != nullptr )
-	{
-		*a_outX = m_x;
-	}
-	if ( a_outY != nullptr )
-	{
-		*a_outY = m_y;
-	}
-}
-

src/tmxobject.h

@@ -1,21 +0,0 @@
-#ifndef __TMXOBJECT_H__
-#define __TMXOBJECT_H__
-
-#include <string>
-
-class CTmxObject
-{
-public:
-	CTmxObject ();
-	CTmxObject ( const std::string& a_name, float a_x, float a_y );
-	~CTmxObject ();
-
-	const std::string& GetName () const;
-	void GetPos ( float* a_outX, float* a_outY ) const;
-
-private:
-	std::string m_name;
-	float m_x, m_y;
-};
-
-#endif//__TMXOBJECT_H__

src/tmxobject.hpp

@@ -0,0 +1,25 @@
+/* tmxobject.hpp - Copyright (C) 2015-2022 a dinosaur (zlib, see COPYING.txt) */
+
+#ifndef TMXOBJECT_HPP
+#define TMXOBJECT_HPP
+
+#include <string>
+#include <utility>
+
+class TmxObject
+{
+public:
+	TmxObject() : mX(0.0f), mY(0.0f) {}
+	TmxObject(std::string aName, float aX, float aY)
+		: mName(std::move(aName)), mX(aX), mY(aY) {}
+	~TmxObject() = default;
+
+	constexpr const std::string& GetName() const { return mName; }
+	inline void GetPos(float& aOutX, float& aOutY) const { aOutX = mX; aOutY = mY; }
+
+private:
+	std::string mName;
+	float mX, mY;
+};
+
+#endif//TMXOBJECT_HPP

src/tmxreader.cpp

@@ -1,29 +1,9 @@
-/* tmxreader.cpp
+/* tmxreader.cpp - Copyright (C) 2015-2022 a dinosaur (zlib, see COPYING.txt) */
 
-  Copyright (C) 2015 Nicholas Curtis
+#include "tmxreader.hpp"
-
+#include "tmxtileset.hpp"
-  This software is provided 'as-is', without any express or implied
+#include "tmxobject.hpp"
-  warranty.  In no event will the authors be held liable for any damages
+#include "tmxlayer.hpp"
-  arising from the use of this software.
-
-  Permission is granted to anyone to use this software for any purpose,
-  including commercial applications, and to alter it and redistribute it
-  freely, subject to the following restrictions:
-
-  1. The origin of this software must not be misrepresented; you must not
-     claim that you wrote the original software. If you use this software
-     in a product, an acknowledgment in the product documentation would be
-     appreciated but is not required.
-  2. Altered source versions must be plainly marked as such, and must not be
-     misrepresented as being the original software.
-  3. This notice may not be removed or altered from any source distribution.
-
-*/
-
-#include "tmxreader.h"
-#include "tmxtileset.h"
-#include "tmxobject.h"
-#include "tmxlayer.h"
 #include <cstdint>
 #include <sstream>
 #include <algorithm>
@@ -32,296 +12,216 @@
 #include <miniz.h>
 
 
-CTmxReader::CTmxReader ()
+TmxReader::~TmxReader()
 {
-
+	// Delete old tilesets
-}
+	for (auto pTileset : mTilesets)
-
-CTmxReader::~CTmxReader ()
-{
-	// Delete old tilesets.
-	for ( auto pTileset : m_tileset )
-	{
 		delete pTileset;
-	}
+	mTilesets.clear();
-	m_tileset.clear ();
 
-	// Delete old layers.
+	// Delete old layers
-	for ( auto pLay : m_layers )
+	for (auto pLay : mLayers)
-	{
 		delete pLay;
-	}
+	mLayers.clear();
-	m_layers.clear ();
 }
 
 
-bool CTmxReader::DecodeMap ( uint32_t* a_pOut, size_t a_outSize, const std::string& a_strIn )
+bool TmxReader::DecodeMap(uint32_t* aOut, size_t aOutSize, const std::string& aBase64Dat)
 {
-	// Decode base64 string.
+	// Cut leading & trailing whitespace (including newlines)
-	size_t cutTheCrap = a_strIn.find_first_not_of ( " \t\n\r" );
+	auto beg = std::find_if_not(aBase64Dat.begin(), aBase64Dat.end(), ::isspace);
-	std::string strDec = base64_decode ( a_strIn.substr ( cutTheCrap ) );
+	if (beg == std::end(aBase64Dat))
-
+		return false;
-	// Decompress compressed data.
+	auto end = std::find_if_not(aBase64Dat.rbegin(), aBase64Dat.rend(), ::isspace);
-	mz_ulong uiDstSize = a_outSize;
+	std::size_t begOff = std::distance(aBase64Dat.begin(), beg);
-	int iRes = uncompress (
+	std::size_t endOff = std::distance(end, aBase64Dat.rend()) - begOff;
-			(unsigned char*)a_pOut, &uiDstSize,
+	auto trimmed = aBase64Dat.substr(begOff, endOff);
-			(const unsigned char*)strDec.data (), strDec.size ()
+
-		);
+	// Decode base64 string
-	strDec.clear ();
+	std::string decoded = base64_decode(trimmed);
-	if ( iRes < 0 )
+
-	{
+	// Decompress compressed data
+	auto dstSize = static_cast<mz_ulong>(aOutSize);
+	int res = uncompress(
+		reinterpret_cast<unsigned char*>(aOut),
+		&dstSize,
+		reinterpret_cast<const unsigned char*>(decoded.data()),
+		static_cast<mz_ulong>(decoded.size()));
+	decoded.clear();
+	if (res < 0)
 		return false;
-	}
 
 	return true;
 }
 
-void CTmxReader::ReadTileset ( rapidxml::xml_node<>* a_xNode )
+void TmxReader::ReadTileset(rapidxml::xml_node<>* aXNode)
 {
 	rapidxml::xml_attribute<>* xAttrib;
 
-	const char*	szName		= "";
+	const char*	name = "";
-	const char*	szSource	= "";
+	const char*	source = "";
-	uint32_t	uiFirstGid	= 0;
+	uint32_t firstGid = 0;
 
-	// Read name.
+	// Read name
-	xAttrib = a_xNode->first_attribute ( "name" );
+	xAttrib = aXNode->first_attribute("name");
-	if ( xAttrib != nullptr )
+	if (xAttrib != nullptr)
-	{
+		name = xAttrib->value();
-		szName = xAttrib->value ();
-	}
 
-	// Read source.
+	// Read source
-	xAttrib = a_xNode->first_attribute ( "source" );
+	xAttrib = aXNode->first_attribute("source");
-	if ( xAttrib != nullptr )
+	if (xAttrib != nullptr)
-	{
+		source = xAttrib->value();
-		szSource = xAttrib->value ();
-	}
 
-	// Read first global ID.
+	// Read first global ID
-	xAttrib = a_xNode->first_attribute ( "firstgid" );
+	xAttrib = aXNode->first_attribute("firstgid");
-	if ( xAttrib != nullptr )
+	if (xAttrib != nullptr)
-	{
+		firstGid = static_cast<uint32_t>(std::stoul(xAttrib->value()));
-		uiFirstGid = std::stoul ( xAttrib->value () );
-	}
 
-	m_tileset.push_back ( new CTmxTileset ( szName, szSource, uiFirstGid ) );
+	mTilesets.push_back(new TmxTileset(name, source, firstGid));
 }
 
-void CTmxReader::ReadLayer ( rapidxml::xml_node<>* a_xNode )
+void TmxReader::ReadLayer(rapidxml::xml_node<>* aXNode)
 {
 	rapidxml::xml_attribute<>* xAttrib;
-	const char*	szName		= "";
+	const char* name    = "";
-	int			iWidth		= 0;
+	int         width   = 0;
-	int			iHeight		= 0;
+	int         height  = 0;
-	uint32_t*	pTileDat	= nullptr;
+	uint32_t*   tileDat = nullptr;
 
-	// Read name.
+	// Read name
-	xAttrib = a_xNode->first_attribute ( "name" );
+	xAttrib = aXNode->first_attribute("name");
-	if ( xAttrib != nullptr )
+	if (xAttrib != nullptr)
-	{
+		name = xAttrib->value();
-		szName = xAttrib->value ();
-	}
 
-	// Read width.
+	// Read width
-	xAttrib = a_xNode->first_attribute ( "width" );
+	xAttrib = aXNode->first_attribute("width");
-	if ( xAttrib != nullptr )
+	if (xAttrib != nullptr)
-	{
+		width = std::stoi(xAttrib->value());
-		iWidth = std::stoi ( xAttrib->value () );
-	}
 
-	// Read height.
+	// Read height
-	xAttrib = a_xNode->first_attribute ( "height" );
+	xAttrib = aXNode->first_attribute("height");
-	if ( xAttrib != nullptr )
+	if (xAttrib != nullptr)
-	{
+		height = std::stoi(xAttrib->value());
-		iHeight = std::stoi ( xAttrib->value () );
-	}
 
-	// Read tile data.
+	// Read tile data
-	auto xData = a_xNode->first_node ( "data" );
+	auto xData = aXNode->first_node("data");
-	if ( xData != nullptr )
+	if (xData != nullptr)
 	{
-		// TODO: don't assume base64 & zlib.
+		// TODO: don't assume base64 & zlib
-		pTileDat = new uint32_t[iWidth * iHeight];
+		tileDat = new uint32_t[width * height];
-		if ( !DecodeMap ( pTileDat, iWidth * iHeight * sizeof(uint32_t), std::string ( xData->value () ) ) )
+		if (!DecodeMap(tileDat, width * height * sizeof(uint32_t), std::string(xData->value())))
 		{
-			delete[] pTileDat;
+			delete[] tileDat;
-			pTileDat = nullptr;
+			tileDat = nullptr;
 		}
 	}
 
-	m_layers.push_back ( new CTmxLayer ( iWidth, iHeight, szName, pTileDat ) );
+	mLayers.push_back(new TmxLayer(width, height, name, tileDat));
 }
 
-void CTmxReader::ReadObjects ( rapidxml::xml_node<>* a_xNode )
+void TmxReader::ReadObjects(rapidxml::xml_node<>* aXNode)
 {
-	for ( auto xNode = a_xNode->first_node (); xNode != nullptr; xNode = xNode->next_sibling () )
+	for (auto xNode = aXNode->first_node(); xNode != nullptr; xNode = xNode->next_sibling())
 	{
-		if ( strcmp ( xNode->name (), "object" ) != 0 )
+		if (strcmp(xNode->name(), "object") != 0)
-		{
 			continue;
-		}
 
 		rapidxml::xml_attribute<>* xAttrib;
-		const char*	name	= "";
+		const char*	name = "";
-		float		x		= 0.0f;
+		float x = 0.0f;
-		float		y		= 0.0f;
+		float y = 0.0f;
 
-		// Read name.
+		// Read name
-		xAttrib = xNode->first_attribute ( "name" );
+		xAttrib = xNode->first_attribute("name");
-		if ( xAttrib != nullptr )
+		if (xAttrib != nullptr)
-		{
+			name = xAttrib->value();
-			name = xAttrib->value ();
-		}
 
-		// Read X pos.
+		// Read X pos
-		xAttrib = xNode->first_attribute ( "x" );
+		xAttrib = xNode->first_attribute("x");
-		if ( xAttrib != nullptr )
+		if (xAttrib != nullptr)
-		{
+			x = std::stof(xAttrib->value());
-			x = std::stof ( xAttrib->value () );
-		}
 
-		// Read Y pos.
+		// Read Y pos
-		xAttrib = xNode->first_attribute ( "y" );
+		xAttrib = xNode->first_attribute("y");
-		if ( xAttrib != nullptr )
+		if (xAttrib != nullptr)
-		{
+			y = std::stof(xAttrib->value());
-			y = std::stof ( xAttrib->value () );
-		}
 
-		m_objects.push_back ( new CTmxObject ( name, x, y ) );
+		mObjects.push_back(new TmxObject(name, x, y));
 	}
 }
 
-void CTmxReader::Open ( std::istream& a_in )
+void TmxReader::Open(std::istream& aIn)
 {
-	// Delete old tilesets.
+	// Delete old tilesets
-	for ( auto pTileset : m_tileset )
+	for (auto tileset : mTilesets)
-	{
+		delete tileset;
-		delete pTileset;
+	mTilesets.clear();
-	}
-	m_tileset.clear ();
 
-	// Delete old layers.
+	// Delete old layers
-	for ( auto pLay : m_layers )
+	for (auto layer : mLayers)
-	{
+		delete layer;
-		delete pLay;
+	mLayers.clear();
-	}
-	m_layers.clear ();
 
-	m_gidTable.clear ();
+	mGidTable.clear();
 
-	// Read string into a buffer.
+	// Read string into a buffer
 	std::stringstream buf;
-	buf << a_in.rdbuf ();
+	buf << aIn.rdbuf();
-	std::string strXml = buf.str ();
+	std::string strXml = buf.str();
-	buf.clear ();
+	buf.clear();
 
-	// Parse document.
+	// Parse document
 	rapidxml::xml_document<> xDoc;
-	xDoc.parse<0> ( (char*)strXml.c_str () );
+	xDoc.parse<0>(const_cast<char*>(strXml.c_str()));
 
-	// Get map node.
+	// Get map node
-	auto xMap = xDoc.first_node ( "map" );
+	auto xMap = xDoc.first_node("map");
-	if ( xMap == nullptr )
+	if (xMap == nullptr)
-	{
 		return;
-	}
 
-	// Read map attribs.
+	// Read map attribs
 	rapidxml::xml_attribute<>* xAttrib = nullptr;
-	if ( ( xAttrib = xMap->first_attribute ( "width" ) ) != nullptr )
+	if ((xAttrib = xMap->first_attribute("width")) != nullptr)
-	{
+		mWidth = std::stoi(xAttrib->value());
-		m_width = std::stoi ( xAttrib->value () );
+	if ((xAttrib = xMap->first_attribute("height")) != nullptr)
-	}
+		mHeight = std::stoi(xAttrib->value());
-	if ( ( xAttrib = xMap->first_attribute ( "height" ) ) != nullptr )
-	{
-		m_height = std::stoi ( xAttrib->value () );
-	}
 
-	// Read nodes.
+	// Read nodes
-	for ( auto xNode = xMap->first_node (); xNode != nullptr; xNode = xNode->next_sibling () )
+	for (auto xNode = xMap->first_node(); xNode != nullptr; xNode = xNode->next_sibling())
 	{
-		// Read layer nodes.
+		// Read layer nodes
-		if ( strcmp ( xNode->name (), "layer" ) == 0 )
+		if (strcmp(xNode->name(), "layer") == 0)
-		{
+			ReadLayer(xNode);
-			ReadLayer ( xNode );
-		}
 		else
-		if ( strcmp ( xNode->name (), "tileset" ) == 0 )
+		if (strcmp(xNode->name(), "tileset") == 0)
-		{
+			ReadTileset(xNode);
-			ReadTileset ( xNode );
-		}
 		else
-		if ( strcmp ( xNode->name (), "objectgroup" ) == 0 )
+		if (strcmp(xNode->name(), "objectgroup") == 0)
-		{
+			ReadObjects(xNode);
-			ReadObjects ( xNode );
-		}
 	}
 
-	// Generate global id table.
+	// Generate global id table
-	for ( auto pTileset : m_tileset )
+	for (auto tileset : mTilesets)
+		mGidTable.push_back(tileset->GetFirstGid());
+	std::sort(mGidTable.rbegin(), mGidTable.rend());
+}
+
+const TmxLayer* TmxReader::GetLayer(const std::string& aName) const
+{
+	for (auto layer : mLayers)
 	{
-		m_gidTable.push_back ( pTileset->GetFirstGid () );
+		if (layer->GetName() == aName)
+			return layer;
 	}
-	std::sort ( m_gidTable.rbegin (), m_gidTable.rend () );
-}
-
-
-int CTmxReader::GetWidth () const
-{
-	return m_width;
-}
-
-int CTmxReader::GetHeight () const
-{
-	return m_height;
-}
-
-
-const CTmxLayer* CTmxReader::GetLayer ( int a_id ) const
-{
-	return m_layers[a_id];
-}
-
-const CTmxLayer* CTmxReader::GetLayer ( std::string a_strName ) const
-{
-	for ( auto pLay : m_layers )
-	{
-		if ( pLay->GetName ().compare ( a_strName ) == 0 )
-		{
-			return pLay;
-		}
-	}
-
 	return nullptr;
 }
 
-int CTmxReader::GetLayerCount () const
+uint32_t TmxReader::LidFromGid(uint32_t aGid)
 {
-	return m_layers.size ();
+	for (uint32_t first : mGidTable)
-}
-
-
-const CTmxObject* CTmxReader::GetObject ( int a_id ) const
-{
-	return m_objects[a_id];
-}
-
-int CTmxReader::GetObjectCount () const
-{
-	return m_objects.size ();
-}
-
-
-uint32_t CTmxReader::LidFromGid ( uint32_t a_uiGid )
-{
-	for ( uint32_t uiFirst : m_gidTable )
 	{
-		if ( uiFirst <= a_uiGid )
+		if (first <= aGid)
-		{
+			return aGid - (first - 1);
-			return a_uiGid - ( uiFirst - 1 );
-		}
 	}
-
+	return aGid;
-	return a_uiGid;
 }

src/tmxreader.h

@@ -1,48 +0,0 @@
-#ifndef __TMXREADER_H__
-#define __TMXREADER_H__
-
-#include <istream>
-#include <vector>
-#include <string>
-#include <cstdint>
-#include <rapidxml/rapidxml.hpp>
-
-class CTmxTileset;
-class CTmxLayer;
-class CTmxObject;
-
-class CTmxReader
-{
-public:
-	CTmxReader ();
-	~CTmxReader ();
-
-	void Open ( std::istream& a_in );
-
-	int GetWidth () const;
-	int GetHeight () const;
-
-	const CTmxLayer* GetLayer ( int a_id ) const;
-	const CTmxLayer* GetLayer ( std::string a_strName ) const;
-	int GetLayerCount () const;
-
-	const CTmxObject* GetObject ( int a_id ) const;
-	int GetObjectCount () const;
-
-	uint32_t LidFromGid ( uint32_t a_gid );
-
-private:
-	bool DecodeMap ( uint32_t* a_out, size_t a_outSize, const std::string& a_strIn );
-	void ReadTileset ( rapidxml::xml_node<>* a_xNode );
-	void ReadLayer ( rapidxml::xml_node<>* a_xNode );
-	void ReadObjects ( rapidxml::xml_node<>* a_xNode );
-
-	int m_width, m_height;
-	std::vector<CTmxTileset*>	m_tileset;
-	std::vector<CTmxLayer*>		m_layers;
-	std::vector<CTmxObject*>	m_objects;
-	std::vector<uint32_t>		m_gidTable;
-
-};
-
-#endif//__TMXREADER_H__

src/tmxreader.hpp

@@ -0,0 +1,50 @@
+/* tmxreader.hpp - Copyright (C) 2015-2022 a dinosaur (zlib, see COPYING.txt) */
+
+#ifndef TMXREADER_HPP
+#define TMXREADER_HPP
+
+#include <istream>
+#include <vector>
+#include <string>
+#include <cstdint>
+#include <rapidxml/rapidxml.hpp>
+
+class TmxTileset;
+class TmxLayer;
+class TmxObject;
+
+class TmxReader
+{
+public:
+	TmxReader() = default;
+	~TmxReader();
+
+	void Open(std::istream& aIn);
+
+	constexpr int GetWidth() const { return mWidth; }
+	constexpr int GetHeight() const { return mHeight; }
+
+	inline const TmxLayer* GetLayer(size_t aId) const { return mLayers.at(aId); }
+	const TmxLayer* GetLayer(const std::string& aName) const;
+	inline std::size_t GetLayerCount() const { return mLayers.size(); }
+
+	inline const TmxObject* GetObject(size_t aId) const { return mObjects.at(aId); }
+	inline size_t GetObjectCount() const { return mObjects.size(); }
+
+	uint32_t LidFromGid(uint32_t aGid);
+
+private:
+	static bool DecodeMap(uint32_t* aOut, size_t aOutSize, const std::string& aBase64Dat);
+	void ReadTileset(rapidxml::xml_node<>* aXNode);
+	void ReadLayer(rapidxml::xml_node<>* aXNode);
+	void ReadObjects(rapidxml::xml_node<>* aXNode);
+
+	int mWidth, mHeight;
+	std::vector<TmxTileset*> mTilesets;
+	std::vector<TmxLayer*>   mLayers;
+	std::vector<TmxObject*>  mObjects;
+	std::vector<uint32_t>    mGidTable;
+
+};
+
+#endif//TMXREADER_HPP

src/tmxtileset.cpp

@@ -1,60 +0,0 @@
-/* tmxtileset.cpp
-
-  Copyright (C) 2015 Nicholas Curtis
-
-  This software is provided 'as-is', without any express or implied
-  warranty.  In no event will the authors be held liable for any damages
-  arising from the use of this software.
-
-  Permission is granted to anyone to use this software for any purpose,
-  including commercial applications, and to alter it and redistribute it
-  freely, subject to the following restrictions:
-
-  1. The origin of this software must not be misrepresented; you must not
-     claim that you wrote the original software. If you use this software
-     in a product, an acknowledgment in the product documentation would be
-     appreciated but is not required.
-  2. Altered source versions must be plainly marked as such, and must not be
-     misrepresented as being the original software.
-  3. This notice may not be removed or altered from any source distribution.
-
-*/
-
-#include "tmxtileset.h"
-
-CTmxTileset::CTmxTileset () :
-	m_strName		( "" ),
-	m_strSource		( "" ),
-	m_uiFirstGid	( 0 )
-{
-
-}
-
-CTmxTileset::CTmxTileset ( const char* a_szName, const char* a_szSource, uint32_t a_uiFirstGid ) :
-	m_strName		( a_szName ),
-	m_strSource		( a_szSource ),
-	m_uiFirstGid	( a_uiFirstGid )
-{
-
-}
-
-CTmxTileset::~CTmxTileset ()
-{
-
-}
-
-
-const std::string& CTmxTileset::GetName () const
-{
-	return m_strName;
-}
-
-const std::string& CTmxTileset::GetSource () const
-{
-	return m_strSource;
-}
-
-uint32_t CTmxTileset::GetFirstGid () const
-{
-	return m_uiFirstGid;
-}

src/tmxtileset.h

@@ -1,25 +0,0 @@
-#ifndef __TMXTILESET_H__
-#define __TMXTILESET_H__
-
-#include <string>
-#include <cstdint>
-
-class CTmxTileset
-{
-public:
-	CTmxTileset ();
-	CTmxTileset ( const char* a_szName, const char* a_szSource, uint32_t a_uiFirstGid );
-	~CTmxTileset ();
-
-	const std::string& GetName () const;
-	const std::string& GetSource () const;
-	uint32_t GetFirstGid () const;
-
-private:
-	std::string m_strName;
-	std::string m_strSource;
-	uint32_t m_uiFirstGid;
-
-};
-
-#endif//__TMXTILESET_H__

src/tmxtileset.hpp

@@ -0,0 +1,28 @@
+/* tmxtileset.hpp - Copyright (C) 2015-2022 a dinosaur (zlib, see COPYING.txt) */
+
+#ifndef TMXTILESET_HPP
+#define TMXTILESET_HPP
+
+#include <string>
+#include <cstdint>
+#include <utility>
+
+class TmxTileset
+{
+public:
+	TmxTileset() : mFirstGid(0) {}
+	TmxTileset(std::string aName, std::string aSource, uint32_t aFirstGid)
+		: mName(std::move(aName)), mSource(std::move(aSource)), mFirstGid(aFirstGid) {}
+	~TmxTileset() = default;
+
+	constexpr const std::string& GetName() const { return mName; }
+	constexpr const std::string& GetSource() const { return mSource; }
+	constexpr uint32_t GetFirstGid() const { return mFirstGid; }
+
+private:
+	std::string mName;
+	std::string mSource;
+	uint32_t mFirstGid;
+};
+
+#endif//TMXTILESET_HPP

tmx2gba.sln

@@ -1,22 +0,0 @@
-
-Microsoft Visual Studio Solution File, Format Version 12.00
-# Visual Studio 2013
-VisualStudioVersion = 12.0.40629.0
-MinimumVisualStudioVersion = 10.0.40219.1
-Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "tmx2gba", "tmx2gba.vcxproj", "{F8167A3B-4B77-4449-B730-F19ECB133A08}"
-EndProject
-Global
-	GlobalSection(SolutionConfigurationPlatforms) = preSolution
-		Debug|Win32 = Debug|Win32
-		Release|Win32 = Release|Win32
-	EndGlobalSection
-	GlobalSection(ProjectConfigurationPlatforms) = postSolution
-		{F8167A3B-4B77-4449-B730-F19ECB133A08}.Debug|Win32.ActiveCfg = Debug|Win32
-		{F8167A3B-4B77-4449-B730-F19ECB133A08}.Debug|Win32.Build.0 = Debug|Win32
-		{F8167A3B-4B77-4449-B730-F19ECB133A08}.Release|Win32.ActiveCfg = Release|Win32
-		{F8167A3B-4B77-4449-B730-F19ECB133A08}.Release|Win32.Build.0 = Release|Win32
-	EndGlobalSection
-	GlobalSection(SolutionProperties) = preSolution
-		HideSolutionNode = FALSE
-	EndGlobalSection
-EndGlobal

tmx2gba.vcxproj

@@ -1,93 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<Project DefaultTargets="Build" ToolsVersion="14.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
-  <ItemGroup Label="ProjectConfigurations">
-    <ProjectConfiguration Include="Debug|Win32">
-      <Configuration>Debug</Configuration>
-      <Platform>Win32</Platform>
-    </ProjectConfiguration>
-    <ProjectConfiguration Include="Release|Win32">
-      <Configuration>Release</Configuration>
-      <Platform>Win32</Platform>
-    </ProjectConfiguration>
-  </ItemGroup>
-  <PropertyGroup Label="Globals">
-    <ProjectGuid>{F8167A3B-4B77-4449-B730-F19ECB133A08}</ProjectGuid>
-    <RootNamespace>tmx2gba</RootNamespace>
-  </PropertyGroup>
-  <Import Project="$(VCTargetsPath)\Microsoft.Cpp.Default.props" />
-  <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'" Label="Configuration">
-    <ConfigurationType>Application</ConfigurationType>
-    <UseDebugLibraries>true</UseDebugLibraries>
-    <PlatformToolset>v140</PlatformToolset>
-    <CharacterSet>MultiByte</CharacterSet>
-  </PropertyGroup>
-  <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'" Label="Configuration">
-    <ConfigurationType>Application</ConfigurationType>
-    <UseDebugLibraries>false</UseDebugLibraries>
-    <PlatformToolset>v140</PlatformToolset>
-    <WholeProgramOptimization>true</WholeProgramOptimization>
-    <CharacterSet>MultiByte</CharacterSet>
-  </PropertyGroup>
-  <Import Project="$(VCTargetsPath)\Microsoft.Cpp.props" />
-  <ImportGroup Label="ExtensionSettings">
-  </ImportGroup>
-  <ImportGroup Label="PropertySheets" Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">
-    <Import Project="$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props" Condition="exists('$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props')" Label="LocalAppDataPlatform" />
-  </ImportGroup>
-  <ImportGroup Label="PropertySheets" Condition="'$(Configuration)|$(Platform)'=='Release|Win32'">
-    <Import Project="$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props" Condition="exists('$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props')" Label="LocalAppDataPlatform" />
-  </ImportGroup>
-  <PropertyGroup Label="UserMacros" />
-  <PropertyGroup />
-  <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">
-    <ClCompile>
-      <WarningLevel>Level4</WarningLevel>
-      <Optimization>Disabled</Optimization>
-      <SDLCheck>true</SDLCheck>
-      <AdditionalIncludeDirectories>inc\</AdditionalIncludeDirectories>
-    </ClCompile>
-    <Link>
-      <GenerateDebugInformation>true</GenerateDebugInformation>
-    </Link>
-  </ItemDefinitionGroup>
-  <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'">
-    <ClCompile>
-      <WarningLevel>Level4</WarningLevel>
-      <Optimization>MaxSpeed</Optimization>
-      <FunctionLevelLinking>true</FunctionLevelLinking>
-      <IntrinsicFunctions>true</IntrinsicFunctions>
-      <SDLCheck>true</SDLCheck>
-      <AdditionalIncludeDirectories>inc\</AdditionalIncludeDirectories>
-    </ClCompile>
-    <Link>
-      <GenerateDebugInformation>true</GenerateDebugInformation>
-      <EnableCOMDATFolding>true</EnableCOMDATFolding>
-      <OptimizeReferences>true</OptimizeReferences>
-    </Link>
-  </ItemDefinitionGroup>
-  <ItemGroup>
-    <ClCompile Include="src\base64.cpp" />
-    <ClCompile Include="src\tmx2gba.cpp" />
-    <ClCompile Include="src\tmxlayer.cpp" />
-    <ClCompile Include="src\tmxobject.cpp" />
-    <ClCompile Include="src\tmxreader.cpp" />
-    <ClCompile Include="src\XGetopt.cpp" />
-    <ClCompile Include="src\tmxtileset.cpp" />
-  </ItemGroup>
-  <ItemGroup>
-    <ClInclude Include="inc\base64.h" />
-    <ClInclude Include="inc\miniz.h" />
-    <ClInclude Include="inc\rapidxml\rapidxml.hpp" />
-    <ClInclude Include="inc\rapidxml\rapidxml_iterators.hpp" />
-    <ClInclude Include="inc\rapidxml\rapidxml_print.hpp" />
-    <ClInclude Include="inc\rapidxml\rapidxml_utils.hpp" />
-    <ClInclude Include="inc\XGetopt.h" />
-    <ClInclude Include="src\tmxlayer.h" />
-    <ClInclude Include="src\tmxobject.h" />
-    <ClInclude Include="src\tmxreader.h" />
-    <ClInclude Include="src\tmxtileset.h" />
-  </ItemGroup>
-  <Import Project="$(VCTargetsPath)\Microsoft.Cpp.targets" />
-  <ImportGroup Label="ExtensionTargets">
-  </ImportGroup>
-</Project>

tmx2gba.vcxproj.filters

@@ -1,75 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
-  <ItemGroup>
-    <Filter Include="Source Files">
-      <UniqueIdentifier>{4FC737F1-C7A5-4376-A066-2A32D752A2FF}</UniqueIdentifier>
-      <Extensions>cpp;c;cc;cxx;def;odl;idl;hpj;bat;asm;asmx</Extensions>
-    </Filter>
-    <Filter Include="Header Files">
-      <UniqueIdentifier>{93995380-89BD-4b04-88EB-625FBE52EBFB}</UniqueIdentifier>
-      <Extensions>h;hh;hpp;hxx;hm;inl;inc;xsd</Extensions>
-    </Filter>
-    <Filter Include="Resource Files">
-      <UniqueIdentifier>{67DA6AB6-F800-4c08-8B7A-83BB121AAD01}</UniqueIdentifier>
-      <Extensions>rc;ico;cur;bmp;dlg;rc2;rct;bin;rgs;gif;jpg;jpeg;jpe;resx;tiff;tif;png;wav;mfcribbon-ms</Extensions>
-    </Filter>
-  </ItemGroup>
-  <ItemGroup>
-    <ClCompile Include="src\tmx2gba.cpp">
-      <Filter>Source Files</Filter>
-    </ClCompile>
-    <ClCompile Include="src\base64.cpp">
-      <Filter>Source Files</Filter>
-    </ClCompile>
-    <ClCompile Include="src\XGetopt.cpp">
-      <Filter>Source Files</Filter>
-    </ClCompile>
-    <ClCompile Include="src\tmxreader.cpp">
-      <Filter>Source Files</Filter>
-    </ClCompile>
-    <ClCompile Include="src\tmxlayer.cpp">
-      <Filter>Source Files</Filter>
-    </ClCompile>
-    <ClCompile Include="src\tmxtileset.cpp">
-      <Filter>Source Files</Filter>
-    </ClCompile>
-    <ClCompile Include="src\tmxobject.cpp">
-      <Filter>Source Files</Filter>
-    </ClCompile>
-  </ItemGroup>
-  <ItemGroup>
-    <ClInclude Include="inc\base64.h">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="inc\rapidxml\rapidxml.hpp">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="inc\rapidxml\rapidxml_iterators.hpp">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="inc\rapidxml\rapidxml_print.hpp">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="inc\rapidxml\rapidxml_utils.hpp">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="inc\miniz.h">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="inc\XGetopt.h">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="src\tmxreader.h">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="src\tmxlayer.h">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="src\tmxtileset.h">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="src\tmxobject.h">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-  </ItemGroup>
-</Project>