RGBELoader: Simplify texture configuration.

[deepkolos]

Fixed #24017

Description

clean RGBELoader unused code

0. unused switch case
1. unused format variable

[mrdoob]

Without studying the code in depth... Wouldn't this break this if texture.type ==== UnsignedByteType?

[deepkolos]

@mrdoob RGBELoader support Float or HalfFloat only

[mrdoob]

I see...

How about this clean up then?

switch ( texture.type ) {

	case FloatType:
	case HalfFloatType:

		texture.encoding = LinearEncoding;
		texture.minFilter = LinearFilter;
		texture.magFilter = LinearFilter;
		texture.generateMipmaps = false;
		texture.flipY = true;

		break;

}

[deepkolos]

@mrdoob the current clean up will remove switch check too, because of the type check already in the parse function

[mrdoob]

Right, but my suggestion would only change the properties in the cases we know that need to be changed.
I think it makes the code more readable and more future proof, wouldn't you agree?

[deepkolos]

@mrdoob 👌🏻

[mrdoob]

Thanks!