Merge http://github.com/rheit/zdoom

src/textures/multipatchtexture.cpp

@@ -297,8 +297,6 @@ FMultiPatchTexture::FMultiPatchTexture (const void *texdef, FPatchLookup *patchl
 		Printf ("Texture %s is left without any patches\n", Name.GetChars());
 	}
 
-	CheckForHacks ();
-
 	DefinitionLump = deflumpnum;
 }
 
@@ -1328,7 +1326,7 @@ void FMultiPatchTexture::ResolvePatches()
 			if (texno == id)	// we found ourselves. Try looking for another one with the same name which is not a multipatch texture itself.
 			{
 				TArray<FTextureID> list;
-				TexMan.ListTextures(Inits[i].TexName, list);
+				TexMan.ListTextures(Inits[i].TexName, list, true);
 				for (int i = list.Size() - 1; i >= 0; i--)
 				{
 					if (list[i] != id && !TexMan[list[i]]->bMultiPatch)
@@ -1342,6 +1340,11 @@ void FMultiPatchTexture::ResolvePatches()
 					if (Inits[i].HasLine) Inits[i].sc.Message(MSG_WARNING, "Texture '%s' references itself as patch\n", Inits[i].TexName.GetChars());
 					else Printf(TEXTCOLOR_YELLOW  "Texture '%s' references itself as patch\n", Inits[i].TexName.GetChars());
 				}
+				else
+				{
+					// If it could be resolved, just print a developer warning.
+					DPrintf(DMSG_WARNING, "Resolved self-referencing texture by picking an older entry for %s", Inits[i].TexName.GetChars());
+				}
 			}
 
 			if (!texno.isValid())
@@ -1372,6 +1375,8 @@ void FMultiPatchTexture::ResolvePatches()
 	delete[] Inits;
 	Inits = nullptr;
 
+	CheckForHacks();
+
 	// If this texture is just a wrapper around a single patch, we can simply
 	// forward GetPixels() and GetColumn() calls to that patch.
 

src/textures/texturemanager.cpp

@@ -267,7 +267,7 @@ FTextureID FTextureManager::CheckForTexture (const char *name, int usetype, BITF
 //
 //==========================================================================
 
-int FTextureManager::ListTextures (const char *name, TArray<FTextureID> &list)
+int FTextureManager::ListTextures (const char *name, TArray<FTextureID> &list, bool listall)
 {
 	int i;
 
@@ -293,12 +293,15 @@ int FTextureManager::ListTextures (const char *name, TArray<FTextureID> &list)
 			// NULL textures must be ignored.
 			if (tex->UseType!=FTexture::TEX_Null) 
 			{
-				unsigned int j;
+				unsigned int j = list.Size();
+				if (!listall)
+				{
 					for (j = 0; j < list.Size(); j++)
 					{
 						// Check for overriding definitions from newer WADs
 						if (Textures[list[j].GetIndex()].Texture->UseType == tex->UseType) break;
 					}
+				}
 				if (j==list.Size()) list.Push(FTextureID(i));
 			}
 		}

src/textures/textures.h

@@ -433,7 +433,7 @@ public:
 
 	FTextureID CheckForTexture (const char *name, int usetype, BITFIELD flags=TEXMAN_TryAny);
 	FTextureID GetTexture (const char *name, int usetype, BITFIELD flags=0);
-	int ListTextures (const char *name, TArray<FTextureID> &list);
+	int ListTextures (const char *name, TArray<FTextureID> &list, bool listall = false);
 
 	void AddTexturesLump (const void *lumpdata, int lumpsize, int deflumpnum, int patcheslump, int firstdup=0, bool texture1=false);
 	void AddTexturesLumps (int lump1, int lump2, int patcheslump);