Migrate to std::string_view

This removes string_view lite as C++17 supports string views natively.

For convenience (and to make the replacement simpler) ToString helper function is kept.
Unfortunately C++17 has no starts/ends_with, helpers were added for this.

Author: Ghabry

CMakeLists.txt

@@ -290,7 +290,6 @@ set(LCF_HEADERS
 	src/generated/lcf/rpg/trooppagecondition.h
 	src/generated/lcf/rpg/variable.h
 	src/lcf/third_party/span.h
-	src/lcf/third_party/string_view.h
 )
 
 set(LCF_SUPPORT_INI 0)

Makefile.am

@@ -322,8 +322,7 @@ lcfrpginclude_HEADERS = \
 	src/generated/lcf/rpg/variable.h
 
 lcfthirdpartyinclude_HEADERS = \
-	src/lcf/third_party/span.h \
-	src/lcf/third_party/string_view.h
+	src/lcf/third_party/span.h
 
 nodist_lcfinclude_HEADERS = src/lcf/config.h
 

README.md

@@ -117,7 +117,7 @@ See the file [COPYING] for copying conditions.
 liblcf includes code of the following 3rd party software:
 
 - [inih] under New BSD license.
-- [string-view-lite] and [span-lite] under Boost Software License, Version 1.0.
+- [span-lite] under Boost Software License, Version 1.0.
 
 See the source code comment headers for license details.
 

generator/csv/functions.csv

@@ -1,4 +1,4 @@
 Structure,Method,Static,Headers
 Actor,void Setup(bool is2k3),f,
 Parameters,void Setup(int final_level),f,
-Terms,"std::string TermOrDefault(const DBString& db_term, StringView default_term)",t,
+Terms,"std::string TermOrDefault(const DBString& db_term, std::string_view default_term)",t,

src/generated/lcf/rpg/terms.h

@@ -28,7 +28,7 @@ namespace rpg {
 		// Sentinel name used to denote that the default hardcoded term should be used.
 		static constexpr const char* kDefaultTerm = "default_term";
 
-		static std::string TermOrDefault(const DBString& db_term, StringView default_term);
+		static std::string TermOrDefault(const DBString& db_term, std::string_view default_term);
 		DBString encounter;
 		DBString special_combat;
 		DBString escape_success;

src/lcf/context.h

@@ -19,10 +19,10 @@ namespace lcf {
  */
 struct ContextNameBase {
 	/** Constructor */
-	constexpr ContextNameBase(StringView n, int i) : name(n), index(i) {}
+	constexpr ContextNameBase(std::string_view n, int i) : name(n), index(i) {}
 
 	/** Name of the enumerated field */
-	StringView name;
+	std::string_view name;
 
 	/** Array index when the object is part of a list (-1 when not) */
 	int index = -1;
@@ -38,7 +38,7 @@ struct ContextStructBase : ContextNameBase {
 	using StructType_t = StructType;
 
 	/** Constructor */
-	constexpr ContextStructBase(StringView n, int i, StructType_t* o)
+	constexpr ContextStructBase(std::string_view n, int i, StructType_t* o)
 		: ContextNameBase(n, i), obj(o) {}
 
 	/** Object instance (cast to appropriate RPG-type */
@@ -58,7 +58,7 @@ struct Context : ContextStructBase<StructType> {
 	/** The type of the parent context */
 	using ParentCtxType_t = ParentCtxType;
 
-	constexpr Context(StringView n, int i, StructType* o, const ParentCtxType_t* pctx)
+	constexpr Context(std::string_view n, int i, StructType* o, const ParentCtxType_t* pctx)
 		: ContextStructBase<StructType>{n, i, o}, parent(pctx) {}
 
 	/** Context of the parent (nullptr when no parent) */

src/lcf/dbstring.h

@@ -14,11 +14,9 @@
 #include <iterator>
 #include <cstdint>
 #include <cstring>
-#include <limits>
 #include <algorithm>
 #include <ostream>
 
-#include "lcf/string_view.h"
 #include "lcf/dbarrayalloc.h"
 
 namespace lcf {
@@ -43,17 +41,17 @@ class DBString {
 		static constexpr size_type npos = size_type(-1);
 
 		constexpr DBString() = default;
-		explicit DBString(StringView s) : _storage(construct_sv(s.data(), s.size())) {}
+		explicit DBString(std::string_view s) : _storage(construct_sv(s.data(), s.size())) {}
 		explicit DBString(const std::string& s) : _storage(construct_z(s.c_str(), s.size())) {}
 
 		// Explicit construct for general const char*
-		explicit DBString(const char* s) : DBString(StringView(s)) {}
+		explicit DBString(const char* s) : DBString(std::string_view(s)) {}
 		// Implicit constructor to capture string literals
 		template <size_t N>
 			DBString(const char(&literal)[N]) : _storage(construct_z(literal, N - 1)) {}
-		DBString(const char* s, size_t len) : DBString(StringView(s, len)) {}
+		DBString(const char* s, size_t len) : DBString(std::string_view(s, len)) {}
 
-		DBString(const DBString& o) : DBString(StringView(o)) {}
+		DBString(const DBString& o) : DBString(std::string_view(o)) {}
 		DBString(DBString&& o) noexcept { swap(o); }
 
 		DBString& operator=(const DBString&);
@@ -66,7 +64,7 @@ class DBString {
 		~DBString() { destroy(); }
 
 		explicit operator std::string() const { return std::string(data(), size()); }
-		operator StringView() const { return StringView(data(), size()); }
+		operator std::string_view() const { return std::string_view(data(), size()); }
 
 		char& operator[](size_type i) { return data()[i]; }
 		char operator[](size_type i) const { return data()[i]; }
@@ -123,21 +121,21 @@ inline std::string ToString(const DBString& s) {
 }
 
 #define LCF_DBSTRING_MAKE_CMP_OPS(LTYPE, RTYPE) \
-inline bool operator==(LTYPE l, RTYPE r) { return StringView(l) == StringView(r); }\
-inline bool operator!=(LTYPE l, RTYPE r) { return StringView(l) != StringView(r); }\
-inline bool operator<(LTYPE l, RTYPE r) { return StringView(l) < StringView(r); }\
-inline bool operator>(LTYPE l, RTYPE r) { return StringView(l) > StringView(r); }\
-inline bool operator<=(LTYPE l, RTYPE r) { return StringView(l) <= StringView(r); }\
-inline bool operator>=(LTYPE l, RTYPE r) { return StringView(l) >= StringView(r); }\
+inline bool operator==(LTYPE l, RTYPE r) { return std::string_view(l) == std::string_view(r); }\
+inline bool operator!=(LTYPE l, RTYPE r) { return std::string_view(l) != std::string_view(r); }\
+inline bool operator<(LTYPE l, RTYPE r) { return std::string_view(l) < std::string_view(r); }\
+inline bool operator>(LTYPE l, RTYPE r) { return std::string_view(l) > std::string_view(r); }\
+inline bool operator<=(LTYPE l, RTYPE r) { return std::string_view(l) <= std::string_view(r); }\
+inline bool operator>=(LTYPE l, RTYPE r) { return std::string_view(l) >= std::string_view(r); }\
 
 LCF_DBSTRING_MAKE_CMP_OPS(const DBString&, const DBString&);
-LCF_DBSTRING_MAKE_CMP_OPS(StringView, const DBString&);
-LCF_DBSTRING_MAKE_CMP_OPS(const DBString&, StringView);
+LCF_DBSTRING_MAKE_CMP_OPS(std::string_view, const DBString&);
+LCF_DBSTRING_MAKE_CMP_OPS(const DBString&, std::string_view);
 LCF_DBSTRING_MAKE_CMP_OPS(const char*, const DBString&);
 LCF_DBSTRING_MAKE_CMP_OPS(const DBString&, const char*);
 
 inline std::ostream& operator<<(std::ostream& os, const DBString& s) {
-	os << StringView(s);
+	os << std::string_view(s);
 	return os;
 }
 
@@ -147,7 +145,7 @@ namespace std {
 
 template <> struct hash<lcf::DBString> {
 	size_t operator()(const lcf::DBString& s) const {
-		return std::hash<lcf::StringView>()(lcf::StringView(s));
+		return std::hash<std::string_view>()(std::string_view(s));
 	}
 };
 

src/lcf/ldb/reader.h

@@ -30,32 +30,32 @@ namespace LDB_Reader {
 	/**
 	 * Loads Database.
 	 */
-	std::unique_ptr<lcf::rpg::Database> Load(StringView filename, StringView encoding = "");
+	std::unique_ptr<lcf::rpg::Database> Load(std::string_view filename, std::string_view encoding = "");
 
 	/**
 	 * Saves Database.
 	 */
-	bool Save(StringView filename, const lcf::rpg::Database& db, StringView encoding = "", SaveOpt opt = SaveOpt::eNone);
+	bool Save(std::string_view filename, const lcf::rpg::Database& db, std::string_view encoding = "", SaveOpt opt = SaveOpt::eNone);
 
 	/**
 	 * Saves Database as XML.
 	 */
-	bool SaveXml(StringView filename, const lcf::rpg::Database& db);
+	bool SaveXml(std::string_view filename, const lcf::rpg::Database& db);
 
 	/**
 	 * Load Database as XML.
 	 */
-	std::unique_ptr<lcf::rpg::Database> LoadXml(StringView filename);
+	std::unique_ptr<lcf::rpg::Database> LoadXml(std::string_view filename);
 
 	/**
 	 * Loads Database.
 	 */
-	std::unique_ptr<lcf::rpg::Database> Load(std::istream& filestream, StringView encoding = "");
+	std::unique_ptr<lcf::rpg::Database> Load(std::istream& filestream, std::string_view encoding = "");
 
 	/**
 	 * Saves Database.
 	 */
-	bool Save(std::ostream& filestream, const lcf::rpg::Database& db, StringView encoding = "", SaveOpt opt = SaveOpt::eNone);
+	bool Save(std::ostream& filestream, const lcf::rpg::Database& db, std::string_view encoding = "", SaveOpt opt = SaveOpt::eNone);
 
 	/**
 	 * Saves Database as XML.

src/lcf/lmt/reader.h

@@ -25,32 +25,32 @@ namespace LMT_Reader {
 	/**
 	 * Loads Map Tree.
 	 */
-	std::unique_ptr<lcf::rpg::TreeMap> Load(StringView filename, StringView encoding = "");
+	std::unique_ptr<lcf::rpg::TreeMap> Load(std::string_view filename, std::string_view encoding = "");
 
 	/**
 	 * Saves Map Tree.
 	 */
-	bool Save(StringView filename, const lcf::rpg::TreeMap& tmap, EngineVersion engine, StringView encoding = "", SaveOpt opt = SaveOpt::eNone);
+	bool Save(std::string_view filename, const lcf::rpg::TreeMap& tmap, EngineVersion engine, std::string_view encoding = "", SaveOpt opt = SaveOpt::eNone);
 
 	/**
 	 * Saves Map Tree as XML.
 	 */
-	bool SaveXml(StringView filename, const lcf::rpg::TreeMap& tmap, EngineVersion engine);
+	bool SaveXml(std::string_view filename, const lcf::rpg::TreeMap& tmap, EngineVersion engine);
 
 	/**
 	 * Loads Map Tree as XML.
 	 */
-	std::unique_ptr<lcf::rpg::TreeMap> LoadXml(StringView filename);
+	std::unique_ptr<lcf::rpg::TreeMap> LoadXml(std::string_view filename);
 
 	/**
 	 * Loads Map Tree.
 	 */
-	std::unique_ptr<lcf::rpg::TreeMap> Load(std::istream& filestream, StringView encoding = "");
+	std::unique_ptr<lcf::rpg::TreeMap> Load(std::istream& filestream, std::string_view encoding = "");
 
 	/**
 	 * Saves Map Tree.
 	 */
-	bool Save(std::ostream& filestream, const lcf::rpg::TreeMap& tmap, EngineVersion engine, StringView encoding = "", SaveOpt opt = SaveOpt::eNone);
+	bool Save(std::ostream& filestream, const lcf::rpg::TreeMap& tmap, EngineVersion engine, std::string_view encoding = "", SaveOpt opt = SaveOpt::eNone);
 
 	/**
 	 * Saves Map Tree as XML.

src/lcf/lmu/reader.h

@@ -30,32 +30,32 @@ namespace LMU_Reader {
 	/**
 	 * Loads map.
 	 */
-	std::unique_ptr<rpg::Map> Load(StringView filename, StringView encoding = "");
+	std::unique_ptr<rpg::Map> Load(std::string_view filename, std::string_view encoding = "");
 
 	/**
 	 * Saves map.
 	 */
-	bool Save(StringView filename, const rpg::Map& map, EngineVersion engine, StringView encoding = "", SaveOpt opt = SaveOpt::eNone);
+	bool Save(std::string_view filename, const rpg::Map& map, EngineVersion engine, std::string_view encoding = "", SaveOpt opt = SaveOpt::eNone);
 
 	/**
 	 * Saves map as XML.
 	 */
-	bool SaveXml(StringView filename, const rpg::Map& map, EngineVersion engine);
+	bool SaveXml(std::string_view filename, const rpg::Map& map, EngineVersion engine);
 
 	/**
 	 * Loads map as XML.
 	 */
-	std::unique_ptr<rpg::Map> LoadXml(StringView filename);
+	std::unique_ptr<rpg::Map> LoadXml(std::string_view filename);
 
 	/**
 	 * Loads map.
 	 */
-	std::unique_ptr<rpg::Map> Load(std::istream& filestream, StringView encoding = "");
+	std::unique_ptr<rpg::Map> Load(std::istream& filestream, std::string_view encoding = "");
 
 	/**
 	 * Saves map.
 	 */
-	bool Save(std::ostream& filestream, const rpg::Map& map, EngineVersion engine, StringView encoding = "", SaveOpt opt = SaveOpt::eNone);
+	bool Save(std::ostream& filestream, const rpg::Map& map, EngineVersion engine, std::string_view encoding = "", SaveOpt opt = SaveOpt::eNone);
 
 	/**
 	 * Saves map as XML.