Generated methods CSV doesn't support record method names

[Su5eD]

Generated methods CSV doesn't support record method names

[LexManos]

To make this clear, it has nothing to do with things being synthetic or not. This is a bug in FML, or at least a misunderstanding of design.
The SRG names are meant to be unique and treated as a lookup in a single dictionary
Having them split by FML means that record names do not get the field names as they are intended.

The fix would be to update FML's Naming Service, but it can also be fixed on our end by duplicating the data into both csv files. So feel free to PR FG to add "f_" to the list of methods so it's retroactive.

[cpw]

The record field accessor methods are likely marked as synthetic by the compiler, which likely means they should not even be in the decompile.

Are you trying to reflect them at runtime and hitting an error, or is it a "they seem to be missing"? Because I would generally expect them to be missing. Note that at runtime deobf, they probably deobf to their field name because they're synthetic.

[Su5eD]

Runtime deobf doesn't work at all because the default modlauncher naming service loads its method mappings from methods.csv. And since record method names are filtered out from that file due to this bug, remap calls will always fall back to the obfuscated name.

All the required method names are already present in the SRG input of OfficialChannelProvider, they just get filtered out because they begin with the field name prefix f_.

[cpw]

Fixing name service is quite doable. We'd need to identify the special case where the method is a field name somehow..

[LexManos]

The point is they shouldn't be special cased. It'd just need to load both the methods and fields csv into a single dictionary.