deps/ox/src/ox/clargs/clargs.cpp

@@ -11,19 +11,18 @@
 
 namespace ox {
 
-ClArgs::ClArgs(int argc, const char **args) noexcept: ClArgs({args, static_cast<size_t>(argc)}) {}
-
-ClArgs::ClArgs(ox::SpanView<const char*> args) noexcept {
-	for (auto i = 0u; i < args.size(); ++i) {
-		auto arg = StringView(args[i]);
+ClArgs::ClArgs(int argc, const char **args) noexcept {
+	auto const argv = ox::SpanView{args, static_cast<size_t>(argc)};
+	for (auto i = 0u; i < static_cast<unsigned>(argc); ++i) {
+		auto arg = StringView(argv[i]);
 		if (arg[0] == '-') {
 			while (arg[0] == '-' && arg.len()) {
 				arg = substr(arg, 1);
 			}
 			m_bools[arg] = true;
 			// parse additional arguments
-			if (i < args.size() && args[i + 1]) {
-				auto val = String(args[i + 1]);
+			if (i < static_cast<unsigned>(argc) && argv[i + 1]) {
+				auto val = String(argv[i + 1]);
 				if (val.len() && val[i] != '-') {
 					if (val == "false") {
 						m_bools[arg] = false;

deps/ox/src/ox/clargs/clargs.hpp

@@ -9,7 +9,6 @@
 #pragma once
 
 #include <ox/std/hashmap.hpp>
-#include <ox/std/span.hpp>
 #include <ox/std/string.hpp>
 
 namespace ox {
@@ -23,8 +22,6 @@ class ClArgs {
 	public:
 		ClArgs(int argc, const char **args) noexcept;
 
-		ClArgs(ox::SpanView<const char*> args) noexcept;
-
 		[[nodiscard]]
 		bool getBool(ox::StringViewCR arg, bool defaultValue) const noexcept;
 

deps/ox/src/ox/fs/test/tests.cpp

@@ -231,7 +231,7 @@ int main(int argc, const char **argv) {
 		oxError("Must specify test to run");
 		return -1;
 	}
-	auto const args = ox::Span{argv, static_cast<size_t>(argc)};
+	auto const args = ox::SpanView{argv, static_cast<size_t>(argc)};
 	ox::StringView const testName = args[1];
 	ox::StringView const testArg = argc >= 3 ? args[2] : nullptr;
 	auto const func = tests.find(testName);

deps/ox/src/ox/fs/tool.cpp

@@ -39,7 +39,7 @@ static ox::Result<ox::UniquePtr<ox::FileSystem>> loadFs(const char *path) noexce
 	return {ox::make_unique<ox::FileSystem32>(buff.data, buff.size)};
 }
 
-static ox::Error runLs(ox::FileSystem *fs, ox::Span<const char*> args) noexcept {
+static ox::Error runLs(ox::FileSystem *fs, ox::SpanView<const char*> args) noexcept {
 	if (args.size() < 2) {
 		oxErr("Must provide a directory to ls\n");
 		return OxError(1);
@@ -51,7 +51,7 @@ static ox::Error runLs(ox::FileSystem *fs, ox::Span<const char*> args) noexcept
 	return OxError(0);
 }
 
-static ox::Error runRead(ox::FileSystem *fs, ox::Span<const char*> args) noexcept {
+static ox::Error runRead(ox::FileSystem *fs, ox::SpanView<const char*> args) noexcept {
 	if (args.size() < 2) {
 		oxErr("Must provide a path to a file to read\n");
 		return OxError(1);
@@ -66,7 +66,7 @@ static ox::Error run(int argc, const char **argv) noexcept {
 		oxErr("OxFS file and subcommand arguments are required\n");
 		return OxError(1);
 	}
-	auto const args = ox::Span{argv, static_cast<size_t>(argc)};
+	auto const args = ox::SpanView{argv, static_cast<size_t>(argc)};
 	auto const fsPath = args[1];
 	ox::String subCmd(args[2]);
 	oxRequire(fs, loadFs(fsPath));

deps/ox/src/ox/std/buffer.hpp

@@ -21,7 +21,7 @@ namespace ox {
 extern template class Vector<char>;
 
 using Buffer = Vector<char>;
-using BufferView = Span<const char>;
+using BufferView = SpanView<char>;
 
 class BufferWriter {
 	private:

deps/ox/src/ox/std/span.hpp

@@ -18,6 +18,126 @@ OX_CLANG_NOWARN_BEGIN(-Wunsafe-buffer-usage)
 
 namespace ox {
 
+template<typename T>
+class SpanView {
+
+	private:
+		T const*m_items{};
+		std::size_t m_size{};
+
+	public:
+		using value_type = T;
+		using size_type = std::size_t;
+
+		template<typename RefType = T const&, typename PtrType = T const*, bool reverse = false>
+		using iterator = SpanIterator<T, RefType, PtrType, reverse>;
+
+		constexpr SpanView() noexcept {}
+
+		template<std::size_t sz>
+		constexpr SpanView(ox::Array<T, sz> const&a) noexcept:
+				m_items(a.data()),
+				m_size(a.size()) {
+		}
+
+		template<std::size_t sz, typename Allocator>
+		constexpr SpanView(ox::Vector<T, sz, Allocator> const&v) noexcept:
+				m_items(v.data()),
+				m_size(v.size()) {
+		}
+
+		template<std::size_t sz>
+		constexpr SpanView(const T (&a)[sz]) noexcept:
+				m_items(a),
+				m_size(sz) {
+		}
+
+		constexpr SpanView(const T *a, std::size_t sz) noexcept:
+				m_items(a),
+				m_size(sz) {
+		}
+
+		constexpr iterator<> begin() noexcept {
+			return iterator<>(m_items, 0, m_size);
+		}
+
+		constexpr iterator<> end() noexcept {
+			return iterator<>(m_items, m_size, m_size);
+		}
+
+		[[nodiscard]]
+		constexpr iterator<const T&, const T*> begin() const noexcept {
+			return iterator<const T&, const T*>(m_items, 0, m_size);
+		}
+
+		[[nodiscard]]
+		constexpr iterator<const T&, const T*> end() const noexcept {
+			return iterator<const T&, const T*>(m_items, m_size, m_size);
+		}
+
+		[[nodiscard]]
+		constexpr iterator<const T&, const T*> cbegin() const noexcept {
+			return iterator<const T&, const T*>(m_items, 0, m_size);
+		}
+
+		[[nodiscard]]
+		constexpr iterator<const T&, const T*> cend() const noexcept {
+			return iterator<const T&, const T*>(m_items, m_size, m_size);
+		}
+
+		[[nodiscard]]
+		constexpr iterator<const T&, const T*, true> crbegin() const noexcept {
+			return iterator<const T&, const T*, true>(m_items, m_size - 1, m_size);
+		}
+
+		[[nodiscard]]
+		constexpr iterator<const T&, const T*, true> crend() const noexcept {
+			return iterator<const T&, const T*, true>(m_items, MaxValue<size_type>, m_size);
+		}
+
+		[[nodiscard]]
+		constexpr iterator<const T&, const T*, true> rbegin() const noexcept {
+			return iterator<const T&, const T*, true>(m_items, m_size - 1, m_size);
+		}
+
+		[[nodiscard]]
+		constexpr iterator<const T&, const T*, true> rend() const noexcept {
+			return iterator<const T&, const T*, true>(m_items, MaxValue<size_type>, m_size);
+		}
+
+		constexpr const T &operator[](std::size_t i) const noexcept {
+			ox::primitiveAssert(__FILE__, __LINE__, i < size(), "Span access overflow");
+			return m_items[i];
+		}
+
+		constexpr SpanView operator+(size_t i) const noexcept {
+			ox::primitiveAssert(__FILE__, __LINE__, i < size(), "Span access overflow");
+			return {m_items + i, m_size - i};
+		}
+
+		constexpr SpanView operator+=(size_t i) noexcept {
+			ox::primitiveAssert(__FILE__, __LINE__, i < size(), "Span access overflow");
+			m_items += i;
+			m_size -= i;
+			return *this;
+		}
+
+		[[nodiscard]]
+		constexpr T const*data() const noexcept {
+			return m_items;
+		}
+
+		[[nodiscard]]
+		constexpr std::size_t size() const noexcept {
+			return m_size;
+		}
+
+		[[nodiscard]]
+		constexpr bool empty() const noexcept {
+			return m_size == 0;
+		}
+
+};
 
 template<typename T>
 class Span {
@@ -33,32 +153,18 @@ class Span {
 		template<typename RefType = T&, typename PtrType = T*, bool reverse = false>
 		using iterator = SpanIterator<T, RefType, PtrType, reverse>;
 
-		constexpr Span() noexcept = default;
-
 		template<std::size_t sz>
 		constexpr Span(ox::Array<T, sz> &a) noexcept:
 			m_items(a.data()),
 			m_size(a.size()) {
 		}
 
-		template<std::size_t sz>
-		constexpr Span(ox::Array<ox::remove_const_t<T>, sz> const&a) noexcept:
-			m_items(a.data()),
-			m_size(a.size()) {
-		}
-
 		template<std::size_t sz, typename Allocator>
 		constexpr Span(ox::Vector<T, sz, Allocator> &v) noexcept:
 			m_items(v.data()),
 			m_size(v.size()) {
 		}
 
-		template<std::size_t sz, typename Allocator>
-		constexpr Span(ox::Vector<ox::remove_const_t<T>, sz, Allocator> const&v) noexcept:
-			m_items(v.data()),
-			m_size(v.size()) {
-		}
-
 		template<std::size_t sz>
 		constexpr Span(T (&a)[sz]) noexcept:
 			m_items(a),
@@ -167,9 +273,6 @@ class Span {
 
 };
 
-template<typename T>
-using SpanView = Span<const T>;
-
 }
 
 OX_CLANG_NOWARN_END

deps/ox/src/ox/std/test/tests.cpp

@@ -477,7 +477,7 @@ int main(int argc, const char **argv) {
 		oxError("Must specify test to run");
 		return -1;
 	}
-	auto const args = ox::Span{argv, static_cast<size_t>(argc)};
+	auto const args = ox::SpanView{argv, static_cast<size_t>(argc)};
 	auto const testName = args[1];
 	auto const func = tests.find(testName);
 	if (func != tests.end()) {

deps/ox/src/ox/std/typetraits.hpp

@@ -269,41 +269,6 @@ template<class T>
 constexpr bool is_move_constructible_v = detail::is_move_constructible<T>(0);
 
 
-template<typename T>
-struct remove_cv {
-	using type = T;
-};
-
-template<typename T>
-struct remove_cv<const T> {
-	using type = T;
-};
-
-template<typename T>
-struct remove_cv<volatile T> {
-	using type = T;
-};
-
-template<typename T>
-struct remove_cv<const volatile T> {
-	using type = T;
-};
-
-
-template<typename T>
-struct remove_const {
-	using type = T;
-};
-
-template<typename T>
-struct remove_const<const T> {
-	using type = T;
-};
-
-template<typename T>
-using remove_const_t = typename remove_const<T>::type;
-
-
 // is String?
 
 template<std::size_t SmallStringSize>

deps/ox/src/ox/std/vector.hpp

@@ -20,7 +20,7 @@
 #include "types.hpp"
 #include "utility.hpp"
 
-OX_ALLOW_UNSAFE_BUFFERS_BEGIN
+OX_CLANG_NOWARN_BEGIN(-Wunsafe-buffer-usage)
 
 namespace ox {
 
@@ -712,4 +712,4 @@ constexpr auto alignOf(const Vector<T>&) noexcept {
 
 }
 
-OX_ALLOW_UNSAFE_BUFFERS_END
+OX_CLANG_NOWARN_END

src/nostalgia/modules/core/src/studio/tilesheeteditor/commands/deletetilescommand.cpp

@@ -2,8 +2,6 @@
  * Copyright 2016 - 2024 Gary Talent (gary@drinkingtea.net). All rights reserved.
  */
 
-#include <ox/std/algorithm.hpp>
-
 #include "deletetilescommand.hpp"
 
 namespace nostalgia::core {
@@ -22,9 +20,11 @@ core::DeleteTilesCommand::DeleteTilesCommand(
 	// copy pixels to be erased
 	{
 		auto &s = getSubSheet(m_img, m_idx);
-		auto dst = m_deletedPixels.begin();
-		auto src = s.pixels.begin() + m_deletePos;
-		ox::copy_n(src, m_deleteSz, dst);
+		auto &p = s.pixels;
+		auto dst = m_deletedPixels.data();
+		auto src = &p[m_deletePos];
+		const auto sz = m_deleteSz * sizeof(decltype(p[0]));
+		ox::memcpy(dst, src, sz);
 	}
 }
 
@@ -32,9 +32,9 @@ ox::Error core::DeleteTilesCommand::redo() noexcept {
 	auto &s = getSubSheet(m_img, m_idx);
 	auto &p = s.pixels;
 	auto srcPos = m_deletePos + m_deleteSz;
-	auto const src = &p[srcPos];
-	auto const dst1 = &p[m_deletePos];
-	auto const dst2 = &p[(p.size() - m_deleteSz)];
+	const auto src = &p[srcPos];
+	const auto dst1 = &p[m_deletePos];
+	const auto dst2 = &p[(p.size() - m_deleteSz)];
 	ox::memmove(dst1, src, p.size() - srcPos);
 	ox::memset(dst2, 0, m_deleteSz * sizeof(decltype(p[0])));
 	return {};
@@ -43,10 +43,10 @@ ox::Error core::DeleteTilesCommand::redo() noexcept {
 ox::Error DeleteTilesCommand::undo() noexcept {
 	auto &s = getSubSheet(m_img, m_idx);
 	auto &p = s.pixels;
-	auto const src = &p[m_deletePos];
-	auto const dst1 = &p[m_deletePos + m_deleteSz];
-	auto const dst2 = src;
-	auto const sz = p.size() - m_deletePos - m_deleteSz;
+	const auto src = &p[m_deletePos];
+	const auto dst1 = &p[m_deletePos + m_deleteSz];
+	const auto dst2 = src;
+	const auto sz = p.size() - m_deletePos - m_deleteSz;
 	ox::memmove(dst1, src, sz);
 	ox::memcpy(dst2, m_deletedPixels.data(), m_deletedPixels.size());
 	return {};

src/nostalgia/modules/core/src/studio/tilesheeteditor/commands/inserttilescommand.cpp

@@ -21,9 +21,10 @@ core::InsertTilesCommand::InsertTilesCommand(
 	{
 		auto &s = getSubSheet(m_img, m_idx);
 		auto &p = s.pixels;
-		auto dst = m_deletedPixels.begin();
-		auto src = p.begin() + p.size() - m_insertCnt;
-		ox::copy_n(src, m_insertCnt, dst);
+		auto dst = m_deletedPixels.data();
+		auto src = &p[p.size() - m_insertCnt];
+		const auto sz = m_insertCnt * sizeof(decltype(p[0]));
+		ox::memcpy(dst, src, sz);
 	}
 }
 

src/nostalgia/modules/core/test/tests.cpp

@@ -27,7 +27,7 @@ static std::map<ox::StringView, ox::Error(*)()> tests = {
 int main(int argc, const char **argv) {
 	int retval = -1;
 	if (argc > 0) {
-		auto const args = ox::Span{argv, static_cast<size_t>(argc)};
+		auto const args = ox::SpanView{argv, static_cast<size_t>(argc)};
 		auto const testName = ox::StringView(args[1]);
 		if (tests.find(testName) != tests.end()) {
 			retval = static_cast<int>(tests[testName]());

src/olympic/applib/applib.cpp

@@ -41,7 +41,8 @@ ox::Error run(
 		ox::StringView project,
 		ox::StringView appName,
 		ox::StringView projectDataDir,
-		ox::SpanView<char const*> argv) noexcept;
+		int argc,
+		char const**argv) noexcept;
 
 }
 
@@ -64,11 +65,7 @@ int main(int argc, char const**argv) {
 #if OLYMPIC_LOAD_STUDIO_MODULES
 	OLYMPIC_PROJECT_NAMESPACE::registerStudioModules();
 #endif
-	auto const err = olympic::run(
-			OLYMPIC_PROJECT_NAME,
-			OLYMPIC_APP_NAME,
-			OLYMPIC_PROJECT_DATADIR,
-			{argv, static_cast<size_t>(argc)});
+	auto const err = olympic::run(OLYMPIC_PROJECT_NAME, OLYMPIC_APP_NAME, OLYMPIC_PROJECT_DATADIR, argc, argv);
 	oxAssert(err, "Something went wrong...");
 	if (err) {
 		oxErrf("Failure: {}\n", toStr(err));

src/olympic/keel/src/module.cpp

@@ -9,9 +9,7 @@ namespace keel {
 static ox::Vector<Module const*> mods;
 
 void registerModule(Module const*mod) noexcept {
-	if (mod) {
 	mods.emplace_back(mod);
-	}
 }
 
 [[nodiscard]]

src/olympic/keel/src/pack-applib.cpp

@@ -81,8 +81,8 @@ static ox::Error pack(
 	return {};
 }
 
-static ox::Error run(ox::SpanView<char const*> argv, ox::StringView projectDataDir) noexcept {
-	ox::ClArgs const args(argv);
+static ox::Error run(int argc, char const**argv, ox::StringView projectDataDir) noexcept {
+	ox::ClArgs const args(argc, argv);
 	auto const argSrc = args.getString("src", "");
 	auto const argRomBin = args.getString("rom-bin", "");
 	auto const argManifest = args.getString("manifest", "");
@@ -103,8 +103,9 @@ ox::Error run(
 		[[maybe_unused]] ox::StringView project,
 		[[maybe_unused]] ox::StringView appName,
 		ox::StringView projectDataDir,
-		ox::SpanView<char const*> argv) noexcept {
-	return ::run(argv, projectDataDir);
+		int argc,
+		char const**argv) noexcept {
+	return ::run(argc, argv, projectDataDir);
 }
 
 }

src/olympic/keel/test/tests.cpp

@@ -28,7 +28,7 @@ static std::map<ox::StringView, ox::Error(*)()> tests = {
 int main(int argc, const char **argv) {
 	int retval = -1;
 	if (argc > 0) {
-		auto const args = ox::Span{argv, static_cast<size_t>(argc)};
+		auto const args = ox::SpanView{argv, static_cast<size_t>(argc)};
 		auto testName = args[1];
 		if (tests.find(testName) != tests.end()) {
 			retval = static_cast<int>(tests[testName]());

src/olympic/studio/applib/src/main.cpp

@@ -52,7 +52,8 @@ static ox::Error runApp(
 static ox::Error run(
 		ox::StringViewCR appName,
 		ox::StringViewCR projectDataDir,
-		ox::SpanView<const char*>) {
+		int,
+		char const**) {
 	// seed UUID generator
 	auto const time = std::time(nullptr);
 	ox::UUID::seedGenerator({
@@ -73,8 +74,9 @@ ox::Error run(
 		ox::StringView project,
 		ox::StringView appName,
 		ox::StringView projectDataDir,
-		ox::SpanView<char const*> args) noexcept {
-	return studio::run(ox::sfmt("{} {}", project, appName), projectDataDir, args);
+		int argc,
+		char const**argv) noexcept {
+	return studio::run(ox::sfmt("{} {}", project, appName), projectDataDir, argc, argv);
 }
 
 }

src/olympic/studio/applib/src/studioapp.cpp

@@ -21,9 +21,7 @@ namespace studio {
 static ox::Vector<studio::Module const*> modules;
 
 void registerModule(studio::Module const*mod) noexcept {
-	if (mod) {
 	modules.emplace_back(mod);
-	}
 }
 
 
@@ -240,18 +238,18 @@ void StudioUI::loadEditorMaker(studio::EditorMaker const&editorMaker) noexcept {
 	}
 }
 
-void StudioUI::loadModule(studio::Module const&mod) noexcept {
-	for (auto const&editorMaker : mod.editors(m_sctx)) {
+void StudioUI::loadModule(studio::Module const*mod) noexcept {
+	for (auto const&editorMaker : mod->editors(m_sctx)) {
 		loadEditorMaker(editorMaker);
 	}
-	for (auto &im : mod.itemMakers(m_sctx)) {
+	for (auto &im : mod->itemMakers(m_sctx)) {
 		m_newMenu.addItemMaker(std::move(im));
 	}
 }
 
 void StudioUI::loadModules() noexcept {
 	for (auto const mod : modules) {
-		loadModule(*mod);
+		loadModule(mod);
 	}
 }
 

src/olympic/studio/applib/src/studioapp.hpp

@@ -69,7 +69,7 @@ class StudioUI: public ox::SignalHandler {
 
 		void loadEditorMaker(studio::EditorMaker const&editorMaker) noexcept;
 
-		void loadModule(studio::Module const&mod) noexcept;
+		void loadModule(studio::Module const*mod) noexcept;
 
 		void loadModules() noexcept;
 

src/olympic/studio/modlib/include/studio/imguiutil.hpp

@@ -189,7 +189,7 @@ bool BeginPopup(turbine::Context &ctx, ox::CStringView popupName, bool &show, Im
  * @param selectedIdx
  * @return true if new value selected, false otherwise
  */
-bool ComboBox(ox::CStringView lbl, ox::Span<const ox::String> list, size_t &selectedIdx) noexcept;
+bool ComboBox(ox::CStringView lbl, ox::SpanView<ox::String> list, size_t &selectedIdx) noexcept;
 
 /**
  *
@@ -223,7 +223,7 @@ bool ListBox(
  * @param selIdx
  * @return true if new value selected, false otherwise
  */
-bool ListBox(ox::CStringView name, ox::Span<const ox::String> const&list, size_t &selIdx) noexcept;
+bool ListBox(ox::CStringView name, ox::SpanView<ox::String> const&list, size_t &selIdx) noexcept;
 
 class FilePicker {
 	private:

src/olympic/studio/modlib/src/imguiutil.cpp

@@ -87,7 +87,7 @@ bool BeginPopup(turbine::Context &ctx, ox::CStringView popupName, bool &show, Im
 
 bool ComboBox(
 		ox::CStringView lbl,
-		ox::Span<const ox::String> list,
+		ox::SpanView<ox::String> list,
 		size_t &selectedIdx) noexcept {
 	bool out{};
 	auto const first = selectedIdx < list.size() ? list[selectedIdx].c_str() : "";
@@ -155,7 +155,7 @@ bool ListBox(
 	return out;
 }
 
-bool ListBox(ox::CStringView name, ox::Span<const ox::String> const&list, size_t &selIdx) noexcept {
+bool ListBox(ox::CStringView name, ox::SpanView<ox::String> const&list, size_t &selIdx) noexcept {
 	return ListBox(name, [list](size_t i) -> ox::CStringView {
 		return list[i];
 	}, list.size(), selIdx);