jMonkeyEngine / jMonkeyEngine/sdk
ReadOnlyPrimitives hint in codechecks not firing
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 348
- Forks
- 104
- Avg merge
- 4d 13h
- Merged PRs (30d)
- 3
Description
I was working on #37 when I figured out that ReadOnlyPrimitivesHint does not work.
As far as I understand it it's purpose is to prevent people from using stuff like
node.getLocalTranslation.set(/*Whatever*/);
I tried it on my fork. Didn't work. Tried it on this repo. Didn't work.
Now I could go fix it myself, but ATM I'm still moving strings to bundles.
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.
Research direction
Start by locating ReadOnlyPrimitivesHint and the codechecks that invoke it. Reproduce the reported example, node.getLocalTranslation.set(/Whatever/);, in the SDK or a fork and trace why the hint does not fire. Done means the hint reliably detects this use and an appropriate check verifies the behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- tooling
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100