renderer: simplify builtin image name parsing

Author: illwieckz

src/engine/renderer/tr_image.cpp

@@ -2736,39 +2736,39 @@ void R_CreateBuiltinImages()
 	imageParams.filterType = filterType_t::FT_LINEAR;
 	imageParams.wrapType = wrapTypeEnum_t::WT_REPEAT;
 
-	tr.whiteImage = R_CreateImage( "_white", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
+	tr.whiteImage = R_CreateImage( "$whiteimage", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
 
 	// we use a solid black image instead of disabling texturing
 	Vector4Set( data, 0, 0, 0, 255 );
 
-	tr.blackImage = R_CreateImage( "_black", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
+	tr.blackImage = R_CreateImage( "$blackimage", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
 
 	// red
 	Vector4Set( data, 255, 0, 0, 255 );
 
-	tr.redImage = R_CreateImage( "_red", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
+	tr.redImage = R_CreateImage( "$redimage", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
 
 	// green
 	Vector4Set( data, 0, 255, 0, 255 );
 
-	tr.greenImage = R_CreateImage( "_green", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
+	tr.greenImage = R_CreateImage( "$greenimage", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
 
 	// blue
 	Vector4Set( data, 0, 0, 255, 255 );
 
-	tr.blueImage = R_CreateImage( "_blue", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
+	tr.blueImage = R_CreateImage( "$blueimage", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
 
 	// null
 	memset( data, 0, sizeof( data ) );
 
-	tr.nullImage = R_CreateImage( "_null", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
+	tr.nullImage = R_CreateImage( "$nullimage", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
 
 	// generate a default normalmap with a fully opaque heightmap (no displacement)
 	Vector4Set( data, 128, 128, 255, 255 );
 
 	imageParams.bits = IF_NOPICMIP | IF_NORMALMAP;
 
-	tr.flatImage = R_CreateImage( "_flat", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
+	tr.flatImage = R_CreateImage( "$flatimage", ( const byte ** ) &dataPtr, 1, 1, 1, imageParams );
 
 	imageParams.bits = IF_NOPICMIP;
 	imageParams.wrapType = wrapTypeEnum_t::WT_CLAMP;

src/engine/renderer/tr_shader.cpp

@@ -1434,48 +1434,12 @@ static bool LoadMap( shaderStage_t *stage, const char *buffer, stageType_t type,
 		return true;
 	}
 
-	if ( !Q_stricmp( token, "$whiteimage" ) || !Q_stricmp( token, "$white" )
-		|| !Q_stricmp( token, "_white" ) || !Q_stricmp( token, "*white" ) )
+	// Quake III backward compatibility.
+	if ( !Q_stricmp( token, "*white" ) )
 	{
 		stage->bundle[ bundleIndex ].image[ 0 ] = tr.whiteImage;
 		return true;
 	}
-	else if ( !Q_stricmp( token, "$blackimage" ) || !Q_stricmp( token, "$black" )
-		|| !Q_stricmp( token, "_black" ) || !Q_stricmp( token, "*black" ) )
-	{
-		stage->bundle[ bundleIndex ].image[ 0 ] = tr.blackImage;
-		return true;
-	}
-	else if ( !Q_stricmp( token, "$redimage" ) || !Q_stricmp( token, "$red" )
-		|| !Q_stricmp( token, "_red" ) || !Q_stricmp( token, "*red" ) )
-	{
-		stage->bundle[ bundleIndex ].image[ 0 ] = tr.redImage;
-		return true;
-	}
-	else if ( !Q_stricmp( token, "$greenimage" ) || !Q_stricmp( token, "$green" )
-		|| !Q_stricmp( token, "_green" ) || !Q_stricmp( token, "*green" ) )
-	{
-		stage->bundle[ bundleIndex ].image[ 0 ] = tr.greenImage;
-		return true;
-	}
-	else if ( !Q_stricmp( token, "$blueimage" ) || !Q_stricmp( token, "$blue" )
-		|| !Q_stricmp( token, "_blue" ) || !Q_stricmp( token, "*blue" ) )
-	{
-		stage->bundle[ bundleIndex ].image[ 0 ] = tr.blueImage;
-		return true;
-	}
-	else if ( !Q_stricmp( token, "$nullimage" ) || !Q_stricmp( token, "$null" )
-		|| !Q_stricmp( token, "_null" ) || !Q_stricmp( token, "*null" ) )
-	{
-		stage->bundle[ bundleIndex ].image[ 0 ] = tr.nullImage;
-		return true;
-	}
-	else if ( !Q_stricmp( token, "$flatimage" ) || !Q_stricmp( token, "$flat" )
-		|| !Q_stricmp( token, "_flat" ) || !Q_stricmp( token, "*flat" ) )
-	{
-		stage->bundle[ bundleIndex ].image[ 0 ] = tr.flatImage;
-		return true;
-	}
 
 	else if ( !Q_stricmp( token, "$lightmap" ) )
 	{