cpp/openttd-patchpack/source Changeset - r27779:653ba52c002e

Changeset - r27779:653ba52c002e

Parent rev.

Child rev.

[Not reviewed]

master

0 3 0

Patric Stout - 10 months ago 2023-08-11 12:53:51
truebrain@openttd.org

Codechange: make no assumptions on how the internals of TileIndex work (#11183)

Basically, avoid ".value", and just cast it to its original type
if you want to retrieve this.

3 files changed with 5 insertions and 5 deletions:

src/map_func.h

src/misc/endian_buffer.hpp

src/script/squirrel_helper.hpp

0 comments (0 inline, 0 general)

src/map_func.h

➞

Show inline comments

@@ @@ -423,23 +423,23 @@ debug_inline static TileIndex TileVirtXY @@
  * Get the X component of a tile
  * @param tile the tile to get the X component of
  * @return the X component
  */
 debug_inline static uint TileX(TileIndex tile)
+{
-	return tile.value & Map::MaxX();
+	return static_cast<uint32_t>(tile) & Map::MaxX();
+}
 /**
  * Get the Y component of a tile
  * @param tile the tile to get the Y component of
  * @return the Y component
  */
 debug_inline static uint TileY(TileIndex tile)
+{
-	return tile.value >> Map::LogX();
+	return static_cast<uint32_t>(tile) >> Map::LogX();
+}
 /**
  * Return the offset between two tiles from a TileIndexDiffC struct.
+ *
  * This function works like #TileDiffXY(int, int) and returns the

src/misc/endian_buffer.hpp

➞

Show inline comments

@@ @@ -50,13 +50,13 @@ public: @@
 	template <class T, std::enable_if_t<std::disjunction_v<std::negation<std::is_class<T>>, std::is_base_of<StrongTypedefBase, T>>, int> = 0>
 	EndianBufferWriter &operator <<(const T data)
+	{
 		if constexpr (std::is_enum_v<T>) {
 			this->Write(static_cast<std::underlying_type_t<const T>>(data));
 		} else if constexpr (std::is_base_of_v<StrongTypedefBase, T>) {
-			this->Write(data.value);
+			this->Write(static_cast<typename T::Type>(data));
 		} else {
 			this->Write(data);
+		}
 		return *this;
+	}
@@ @@ -143,13 +143,13 @@ public: @@
 	template <class T, std::enable_if_t<std::disjunction_v<std::negation<std::is_class<T>>, std::is_base_of<StrongTypedefBase, T>>, int> = 0>
 	EndianBufferReader &operator >>(T &data)
+	{
 		if constexpr (std::is_enum_v<T>) {
 			data = static_cast<T>(this->Read<std::underlying_type_t<T>>());
 		} else if constexpr (std::is_base_of_v<StrongTypedefBase, T>) {
-			data.value = this->Read<decltype(data.value)>();
+			data = this->Read<typename T::Type>();
 		} else {
 			data = this->Read<T>();
+		}
 		return *this;
+	}

src/script/squirrel_helper.hpp

➞

Show inline comments

@@ @@ -34,13 +34,13 @@ namespace SQConvert { @@
 	template <> struct Return<uint32_t>       { static inline int Set(HSQUIRRELVM vm, uint32_t res)      { sq_pushinteger(vm, (int32_t)res); return 1; } };
 	template <> struct Return<int8_t>         { static inline int Set(HSQUIRRELVM vm, int8_t res)        { sq_pushinteger(vm, res); return 1; } };
 	template <> struct Return<int16_t>        { static inline int Set(HSQUIRRELVM vm, int16_t res)       { sq_pushinteger(vm, res); return 1; } };
 	template <> struct Return<int32_t>        { static inline int Set(HSQUIRRELVM vm, int32_t res)       { sq_pushinteger(vm, res); return 1; } };
 	template <> struct Return<int64_t>        { static inline int Set(HSQUIRRELVM vm, int64_t res)       { sq_pushinteger(vm, res); return 1; } };
 	template <> struct Return<Money>        { static inline int Set(HSQUIRRELVM vm, Money res)       { sq_pushinteger(vm, res); return 1; } };
-	template <> struct Return<TileIndex>    { static inline int Set(HSQUIRRELVM vm, TileIndex res)   { sq_pushinteger(vm, (int32_t)res.value); return 1; } };
+	template <> struct Return<TileIndex>    { static inline int Set(HSQUIRRELVM vm, TileIndex res)   { sq_pushinteger(vm, (int32_t)static_cast<uint32_t>(res)); return 1; } };
 	template <> struct Return<bool>         { static inline int Set(HSQUIRRELVM vm, bool res)        { sq_pushbool   (vm, res); return 1; } };
 	template <> struct Return<char *>       { /* Do not use char *, use std::optional<std::string> instead. */ };
 	template <> struct Return<const char *> { /* Do not use const char *, use std::optional<std::string> instead. */ };
 	template <> struct Return<void *>       { static inline int Set(HSQUIRRELVM vm, void *res)       { sq_pushuserpointer(vm, res); return 1; } };
 	template <> struct Return<HSQOBJECT>    { static inline int Set(HSQUIRRELVM vm, HSQOBJECT res)   { sq_pushobject(vm, res); return 1; } };

0 comments (0 inline, 0 general)