⁠
joe: Include much more detail in inspection descriptions
- <!--
- Minecraft Development for IntelliJ
-
- https://mcdev.io/
-
- Copyright (C) 2026 minecraft-dev
-
- This program is free software: you can redistribute it and/or modify
- it under the terms of the GNU Lesser General Public License as published
- by the Free Software Foundation, version 3.0 only.
-
- This program is distributed in the hope that it will be useful,
- but WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- GNU General Public License for more details.
-
- You should have received a copy of the GNU Lesser General Public License
- along with this program. If not, see <https://www.gnu.org/licenses/>.
- -->
-
- <html>
- <body>
- Reports when INVOKE_ASSIGN could be replaced with a MixinExtras expression. Expressions are preferred over INVOKE_ASSIGN because they fail when an assignment doesn't exist.
- </body>
- </html>
- <!--
- Minecraft Development for IntelliJ
-
- https://mcdev.io/
-
- Copyright (C) 2026 minecraft-dev
-
- This program is free software: you can redistribute it and/or modify
- it under the terms of the GNU Lesser General Public License as published
- by the Free Software Foundation, version 3.0 only.
-
- This program is distributed in the hope that it will be useful,
- but WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- GNU General Public License for more details.
-
- You should have received a copy of the GNU Lesser General Public License
- along with this program. If not, see <https://www.gnu.org/licenses/>.
- -->
-
- <html>
- <body>
- <p>
- Reports when <code>INVOKE_ASSIGN</code> could be replaced with a MixinExtras expression.
- </p>
- <!-- tooltip end -->
- <p>
- Expressions are preferred over <code>INVOKE_ASSIGN</code> because they fail when an assignment doesn't exist.
- </p>
- <pre><code>
- // bad
- @Inject(method = "foo", at = @At(value = "INVOKE_ASSIGN", target = "Lcom/example/Foo;getName()Ljava/lang/String;"))
-
- // good
- @Definition(id = "getName", method = "Lcom/example/Foo;getName()Ljava/lang/String;")
- @Expression("? = ?.getName()")
- @Inject(method = "foo", at = @At(value = "MIXINEXTRAS:EXPRESSION", shift = At.Shift.AFTER))
- </code></pre>
- </body>
- </html>