Primitive class wrappers are not unwrapped
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 45/100
Research direction
Start with the InputMapping clinit example in the issue and locate the corresponding decompiled output. Trace how primitive class literals are rendered, then verify that the output uses int.class and boolean.class rather than Integer.TYPE and Boolean.TYPE.
Written by the indexing model from the issue text.
Description
(Originally opened as https://github.com/MinecraftForge/ForgeGradle/issues/713 on the wrong repo)
This is a very minor enchancement, but it should be pretty easy to implement
See InputMapping clinit:
MethodType methodtype = MethodType.methodType(Boolean.TYPE);
MethodHandle methodhandle = null;
int i = 0;
try {
methodhandle = lookup.findStatic(GLFW.class, "glfwRawMouseMotionSupported", methodtype);
MethodHandle methodhandle1 = lookup.findStaticGetter(GLFW.class, "GLFW_RAW_MOUSE_MOTION", Integer.TYPE);
Integer.TYPE and Boolean.TYPE should be replaced with int.class and boolean.class respectively, as that's more easy to understand (the compiler converts any primitive .class call to the respective wrapper class .TYPE)
- Dominant language
- No language data
- Stars
- 110
- Forks
- 49
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from MinecraftForge/ForgeFlower
-
Difficulty 4/5 3-5 days Newbie friendliness 42/100
MinecraftForge/ForgeFlower#131 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
MinecraftForge/ForgeFlower#86 · 3 comments ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
MinecraftForge/ForgeFlower#85 · 3 comments ·
-
Difficulty 4/5 3-5 days Newbie friendliness 42/100
-
Difficulty 4/5 3-5 days Newbie friendliness 30/100
MinecraftForge/ForgeFlower#76 · 1 comment · 3 reactions ·
All issues in MinecraftForge/ForgeFlower
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
JakeChampion/lang#10213 ·
-
enhancement good first issue needs testing
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
bradcypert/plum#58 ·
-
flang:fir-hlfir
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
llvm/llvm-project#225935 ·
-
area:cards hand-tail ready-for-agent
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
fil-donadoni/tolaria#4446 ·