Yarn: move C2S/S2C to the front of names

Created on 16 Feb 2019  路  7Comments  路  Source: FabricMC/yarn

ex. TitleS2CPacket -> S2CTitlePacket

discussion
Source
picture Boundarybreaker
👍5 👎2

All 7 comments

And also rename them to anything that doesn't contain a number in the name

Runemoro picture Runemoro on 16 Feb 2019
👍1

S2C is fine as it is IMO (at the start tho ofc)

Blayyke picture Blayyke on 16 Feb 2019

Let's have that debate in a separate issue. It's not the goal of this, so it's not worth clogging up this issue with.

Boundarybreaker picture Boundarybreaker on 16 Feb 2019

And also rename them to anything that doesn't contain a number in the name

Fastutil first.

asiekierka picture asiekierka on 16 Feb 2019
😄1

Any more input for this? I feel like this is really unnecessary and doesn't improve the names (and also makes it harder to find classes in tree view)

YanisBft picture YanisBft on 27 Mar 2021

Same thing as Block suffix vs prefix imo... I am against this.

Technici4n picture Technici4n on 27 Mar 2021
👍1

Closing this as people seem to not want such a refactor. If someone thinks this should still happen, please open a new issue/PR

YanisBft picture YanisBft on 28 Mar 2021
Was this page helpful?
0 / 5 - 0 ratings

Related issues

quat1024 picture quat1024  路  6Comments
Draylar picture Draylar  路  6Comments
Sollace picture Sollace  路  5Comments
haykam821 picture haykam821  路  4Comments
Bixilon picture Bixilon  路  5Comments