cpp/openttd-patchpack/source Changeset - r22518:bf65c0efa303

Changeset - r22518:bf65c0efa303

Parent rev.

Child rev.

[Not reviewed]

master

0 4 0

frosch - 7 years ago 2017-01-07 21:36:25
frosch@openttd.org

(svn r27725) -Codechange: Remove IsOpenTTDBaseGRF and test for GCF_CONFIG instead, which does the same in all use-cases.

4 files changed with 1 insertions and 14 deletions:

src/newgrf.cpp

src/newgrf.h

src/newgrf_config.cpp

src/newgrf_config.h

0 comments (0 inline, 0 general)

src/newgrf.cpp

➞

Show inline comments

@@ @@ -5621,7 +5621,7 @@ static void GraphicsNew(ByteReader *buf) @@
 	uint16 offset = HasBit(type, 7) ? buf->ReadExtendedByte() : 0;
 	ClrBit(type, 7); // Clear the high bit as that only indicates whether there is an offset.
-	if ((type == 0x0D) && (num == 10) && _cur.grffile->is_ottdfile) {
+	if ((type == 0x0D) && (num == 10) && HasBit(_cur.grfconfig->flags, GCF_SYSTEM)) {
 		/* Special not-TTDP-compatible case used in openttd.grf
 		 * Missing shore sprites and initialisation of SPR_SHORE_BASE */
 		grfmsg(2, "GraphicsNew: Loading 10 missing shore sprites from extra grf.");
@@ @@ -8858,7 +8858,6 @@ void LoadNewGRFFile(GRFConfig *config, u @@
 		if (_cur.grffile == NULL) usererror("File '%s' lost in cache.\n", filename);
 		if (stage == GLS_RESERVE && config->status != GCS_INITIALISED) return;
 		if (stage == GLS_ACTIVATION && !HasBit(config->flags, GCF_RESERVED)) return;
 		_cur.grffile->is_ottdfile = config->IsOpenTTDBaseGRF();
+	}
 	if (file_index > LAST_GRF_SLOT) {

src/newgrf.h

➞

Show inline comments

@@ @@ -103,7 +103,6 @@ struct GRFLabel { @@
 /** Dynamic data of a loaded NewGRF */
 struct GRFFile : ZeroedMemoryAllocator {
 	char *filename;
 	bool is_ottdfile;
 	uint32 grfid;
 	byte grf_version;

src/newgrf_config.cpp

➞

Show inline comments

@@ @@ -916,15 +916,6 @@ char *GRFBuildParamList(char *dst, const @@
 static const uint32 OPENTTD_GRAPHICS_BASE_GRF_ID = BSWAP32(0xFF4F5400);
 /**
  * Checks whether this GRF is a OpenTTD base graphic GRF.
  * @return true if and only if it is a base GRF.
  */
 bool GRFConfig::IsOpenTTDBaseGRF() const
+{
 	return (this->ident.grfid & 0x00FFFFFF) == OPENTTD_GRAPHICS_BASE_GRF_ID;
+}
 /**
  * Search a textfile file next to this NewGRF.
  * @param type The type of the textfile to search for.
  * @return The filename for the textfile, \c NULL otherwise.

src/newgrf_config.h

➞

Show inline comments

@@ @@ -179,8 +179,6 @@ struct GRFConfig : ZeroedMemoryAllocator @@
 	void CopyParams(const GRFConfig &src);
 	bool IsOpenTTDBaseGRF() const;
 	const char *GetTextfile(TextfileType type) const;
 	const char *GetName() const;
 	const char *GetDescription() const;

0 comments (0 inline, 0 general)