{"action":"created","issue":{"url":"https://api.github.com/repos/SpongePowered/Mixin/issues/67","labels_url":"https://api.github.com/repos/SpongePowered/Mixin/issues/67/labels{/name}","comments_url":"https://api.github.com/repos/SpongePowered/Mixin/issues/67/comments","events_url":"https://api.github.com/repos/SpongePowered/Mixin/issues/67/events","html_url":"https://github.com/SpongePowered/Mixin/issues/67","id":106227425,"number":67,"title":"Support needed for remapping parameters but not methods","user":{"login":"Zidane","id":744482,"avatar_url":"https://avatars.githubusercontent.com/u/744482?v=3","gravatar_id":"","url":"https://api.github.com/users/Zidane","html_url":"https://github.com/Zidane","followers_url":"https://api.github.com/users/Zidane/followers","following_url":"https://api.github.com/users/Zidane/following{/other_user}","gists_url":"https://api.github.com/users/Zidane/gists{/gist_id}","starred_url":"https://api.github.com/users/Zidane/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Zidane/subscriptions","organizations_url":"https://api.github.com/users/Zidane/orgs","repos_url":"https://api.github.com/users/Zidane/repos","events_url":"https://api.github.com/users/Zidane/events{/privacy}","received_events_url":"https://api.github.com/users/Zidane/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/SpongePowered/Mixin/labels/accepted","name":"accepted","color":"55FF55"},{"url":"https://api.github.com/repos/SpongePowered/Mixin/labels/enhancement","name":"enhancement","color":"ffff55"}],"state":"open","locked":false,"assignee":{"login":"Mumfrey","id":1147071,"avatar_url":"https://avatars.githubusercontent.com/u/1147071?v=3","gravatar_id":"","url":"https://api.github.com/users/Mumfrey","html_url":"https://github.com/Mumfrey","followers_url":"https://api.github.com/users/Mumfrey/followers","following_url":"https://api.github.com/users/Mumfrey/following{/other_user}","gists_url":"https://api.github.com/users/Mumfrey/gists{/gist_id}","starred_url":"https://api.github.com/users/Mumfrey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Mumfrey/subscriptions","organizations_url":"https://api.github.com/users/Mumfrey/orgs","repos_url":"https://api.github.com/users/Mumfrey/repos","events_url":"https://api.github.com/users/Mumfrey/events{/privacy}","received_events_url":"https://api.github.com/users/Mumfrey/received_events","type":"User","site_admin":false},"milestone":null,"comments":8,"created_at":"2015-09-13T17:03:03Z","updated_at":"2015-09-14T13:39:47Z","closed_at":null,"body":"One thing that we've hit in SpongeForge is the fact that we are injecting into GameData#registerBlock(Block, String, int);int and therefore we can't remap this as the method name is obviously not obfuscated BUT the Block parameter needs to be. Sooo\r\n\r\n* We do remap=false but at production the injection fails as it expects registerBlock(atr, String, int);int where atr = Block.class\r\n* we do remap = true but at compile-time the mixin AP fails because it finds no obfuscation mapping for registerBlock (as there wouldn't be one since it isn't obfuscated at all).\r\n\r\nWith the latter option, I tried shoehoring an entry into extraSrg that basically just maps registerBlock back onto the same thing with:\r\n\r\nMD: net/minecraftforge/fml/common/registry/GameData/registerBlock (Lnet/minecraft/block/Block;Ljava/lang/String;I)I net/minecraftforge/fml/common/registry/GameData/registerBlock (Lnet/minecraft/block/Block;Ljava/lang/String;I)I\r\n\r\nUnless that mapping is incorrect...\r\n\r\nWe need guidance or a fix for this ASAP."},"comment":{"url":"https://api.github.com/repos/SpongePowered/Mixin/issues/comments/140078418","html_url":"https://github.com/SpongePowered/Mixin/issues/67#issuecomment-140078418","issue_url":"https://api.github.com/repos/SpongePowered/Mixin/issues/67","id":140078418,"user":{"login":"Zidane","id":744482,"avatar_url":"https://avatars.githubusercontent.com/u/744482?v=3","gravatar_id":"","url":"https://api.github.com/users/Zidane","html_url":"https://github.com/Zidane","followers_url":"https://api.github.com/users/Zidane/followers","following_url":"https://api.github.com/users/Zidane/following{/other_user}","gists_url":"https://api.github.com/users/Zidane/gists{/gist_id}","starred_url":"https://api.github.com/users/Zidane/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Zidane/subscriptions","organizations_url":"https://api.github.com/users/Zidane/orgs","repos_url":"https://api.github.com/users/Zidane/repos","events_url":"https://api.github.com/users/Zidane/events{/privacy}","received_events_url":"https://api.github.com/users/Zidane/received_events","type":"User","site_admin":false},"created_at":"2015-09-14T13:39:47Z","updated_at":"2015-09-14T13:39:47Z","body":"@Mumfrey \r\n\r\nI came to that realization last night as well. I thought a potential fix is to accept that PREINIT cannot touch SRG classes and make a new state between INIT and DEFAULT that would be able to acquire the SRG name now available from the FMLDeobfTweaker but still be early enough to mix it. \r\n\r\nI am not sure the correct spot to put this at..."}}