ZDoom

Ok, forgot about that. It's still broken beyond repair, though.

[quote="Graf Zahl"]The entire player class selection is done via display name. If that gets replaced by a LANGUAGE lookup it'd all fall apart.
I have no idea why Grubber implemented it this way as it's completely stupid.[/quote]
[url=http://forum.zdoom.org/viewtopic.php?f=7&t=27507&p=530946#p530946]Yes you do.[/url] Do I have to remind you every 5 years? :-)

(Sorry for bumping the thread.)

That would be just fine with me. I'd even retract my feature suggestion from last year to make Tag default to DisplayName, if it would be considered the new 'proper' method for nice names.

I'd be all for such a change. It'd remove a persistent thorn for sure.

I think you are correct. The best option would probably be to officially deprecate the 'displayname' property, wrap all accesses into some functions, add a new property solely for display and use the class name in other places and the old 'displayname' only in the places where it's necessary but fall back to the class name if none is specified. But that'd require some serious thinking.

Perhaps this could be better resolved by the addition of Player.NiceName or making Tag functional on player classes, which will override the name on the menu if defined? It's definitely kind of silly that that's a direction this would be required to take, but the simpler option is clearly unavailable due to poor implementation.

Ah, I see. How disappointing... :(

That's basically fine. The language lookup is only used in the menu.
The problem I was talking about is that I cannot retroactively change current player names because it'd break all mods out there, for the simple reason that some parts of the code use the 'display'-name for identification purposes - and expose this to some definition lumps!

[enu default]
MYPLAYERNAME="My Player";

ACTOR NewPlayer : PlayerPawn
{
	Player.DisplayName "$MYPLAYERNAME"
	...
}

I do just what I was hoping to see done to DoomPlayer (and the other internal player classes)
In LANGUAGE:
[code][enu default]
MYPLAYERNAME="My Player";[/code]
In DECORATE:
[code]ACTOR NewPlayer : PlayerPawn
{
Player.DisplayName "$MYPLAYERNAME"
...
}[/code]

I've seen no problems so far, but if you know of something I should be looking for...

What precisely do you do?

I'll have to take your word for it, then... So would you recommend I not use LANGUAGE for my classes' names?

The entire player class selection is done via display name. If that gets replaced by a LANGUAGE lookup it'd all fall apart.
I have no idea why Grubber implemented it this way as it's completely stupid.

What kind of things would break? I've successfully made player classes that reference a LANGUAGE string in their DisplayName, and didn't notice problems. What should I be looking out for?

Sadly not. Despite the term 'Displayname' this property is used for other purposes as well which should have been using the class name in the first place.
I'd really like to change this design bug, but that'd break pretty much any player mod out there.

Excellent point; I forgot to even consider that.