Closed Braiam23 closed 4 days ago
[!CAUTION]
Review failed
The pull request is closed.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
By the way, I will not put this change in the Changelog because it is not fundamental, it does not drastically change the mod, it is just a small improvement.
By the way, if you have no idea what to do with the mod, let the designers' work revive the mod or let me manage the project for a few days XD.
Now if you don't accept the PR I don't know how I can help you with the mod either.
You are already lead designer
By the way, I will not put this change in the Changelog because it is not fundamental, it does not drastically change the mod, it is just a small improvement.
Changelog is here to log changes, no matter how small it may be, anyway I will do it myself I guess.
Thanks for the PR
21679 new problems were found
Inspection name | Severity | Problems |
---|---|---|
Unchecked warning |
🔶 Warning | 10614 |
Unnecessary 'this' qualifier |
🔶 Warning | 4020 |
Deprecated API usage |
🔶 Warning | 1566 |
Auto-boxing |
🔶 Warning | 892 |
@NotNull/@Nullable problems |
🔶 Warning | 417 |
Statement with empty body |
🔶 Warning | 361 |
Field can be local |
🔶 Warning | 313 |
Lombok @Getter may be used |
🔶 Warning | 276 |
Nullability and data flow problems |
🔶 Warning | 183 |
Auto-unboxing |
🔶 Warning | 146 |
Unused assignment |
🔶 Warning | 126 |
Non-constant field with upper-case name |
🔶 Warning | 114 |
Constant values |
🔶 Warning | 108 |
Javadoc declaration problems |
🔶 Warning | 103 |
Redundant local variable |
🔶 Warning | 101 |
Lombok @Setter may be used |
🔶 Warning | 89 |
Questionable name |
🔶 Warning | 76 |
Possibly lossy implicit cast in compound assignment |
🔶 Warning | 46 |
Call to 'printStackTrace()' |
🔶 Warning | 36 |
Declaration has problems in Javadoc references |
🔶 Warning | 24 |
Result of method call ignored |
🔶 Warning | 18 |
Deprecated member is still used |
🔶 Warning | 14 |
'size() == 0' can be replaced with 'isEmpty()' |
🔶 Warning | 14 |
Dynamic regular expression can be replaced by compiled 'Pattern' |
🔶 Warning | 13 |
Link specified as plain text |
🔶 Warning | 13 |
Method is identical to its super method |
🔶 Warning | 13 |
Suspicious collection method call |
🔶 Warning | 13 |
'Set' can be replaced with 'EnumSet' |
🔶 Warning | 11 |
Integer division in floating-point context |
🔶 Warning | 9 |
Mismatched query and update of collection |
🔶 Warning | 9 |
'Map' can be replaced with 'EnumMap' |
🔶 Warning | 8 |
String concatenation as argument to 'StringBuilder.append()' call |
🔶 Warning | 8 |
Class is exposed outside of its visibility scope |
🔶 Warning | 7 |
Condition is covered by further condition |
🔶 Warning | 7 |
Octal integer |
🔶 Warning | 7 |
Suspicious variable/parameter name combination |
🔶 Warning | 7 |
Vulnerable imported dependency |
🔶 Warning | 7 |
Functional expression can be folded |
🔶 Warning | 6 |
Copy of existing static method body |
🔶 Warning | 6 |
Unused import |
🔶 Warning | 6 |
Unnecessary unary minus |
🔶 Warning | 6 |
Unstable API Usage |
🔶 Warning | 6 |
Bulk operation can be used instead of iteration |
🔶 Warning | 6 |
Catch block may ignore exception |
🔶 Warning | 5 |
Variable is assigned to itself |
🔶 Warning | 5 |
Stream API call chain can be simplified |
🔶 Warning | 5 |
Dangling Javadoc comment |
🔶 Warning | 4 |
Duplicate condition |
🔶 Warning | 4 |
Inner class may be 'static' |
🔶 Warning | 4 |
Manual min/max calculation |
🔶 Warning | 4 |
AutoCloseable used without 'try'-with-resources |
🔶 Warning | 3 |
Too weak variable type leads to unnecessary cast |
🔶 Warning | 3 |
Mismatched read and write of array |
🔶 Warning | 3 |
Value passed as parameter never read |
🔶 Warning | 3 |
String concatenation in loop |
🔶 Warning | 3 |
Local variable used and declared in different 'switch' branches |
🔶 Warning | 3 |
Unnecessary 'continue' statement |
🔶 Warning | 3 |
'default' not last case in 'switch' |
🔶 Warning | 2 |
Call math rounding with 'int' argument |
🔶 Warning | 2 |
Redundant operation on empty container |
🔶 Warning | 2 |
Redundant usage of unmodifiable collection wrappers |
🔶 Warning | 2 |
Method does not call super method |
🔶 Warning | 2 |
Caught exception is immediately rethrown |
🔶 Warning | 1 |
Redundant 'Collection.addAll()' call |
🔶 Warning | 1 |
'static' method declared 'final' |
🔶 Warning | 1 |
Call to 'toString()' on array |
🔶 Warning | 1 |
Method name same as class name |
🔶 Warning | 1 |
Redundant type cast |
🔶 Warning | 1 |
String comparison using '==', instead of 'equals()' |
🔶 Warning | 1 |
'String.equals()' can be replaced with 'String.isEmpty()' |
🔶 Warning | 1 |
Minimum 'switch' branches |
🔶 Warning | 1 |
Unary plus |
🔶 Warning | 1 |
Commented out code |
◽️ Notice | 1006 |
Duplicated code fragment |
◽️ Notice | 677 |
Non-distinguishable logging calls |
◽️ Notice | 33 |
Non-constant string concatenation as argument to logging call |
◽️ Notice | 17 |
'if' statement with identical branches or common parts |
◽️ Notice | 15 |
Constant values |
◽️ Notice | 14 |
Method can be extracted |
◽️ Notice | 12 |
Multiple occurrences of the same expression |
◽️ Notice | 4 |
Duplicate branches in 'switch' |
◽️ Notice | 3 |
Non-strict inequality '>=' or '<=' can be replaced with '==' |
◽️ Notice | 1 |
☁️ View the detailed Qodana report
📝 Description
This PR contains improved textures for the holographic sights, and also improves the models I try to replicate to real-life models. What differentiates it from other PRs is that the non-visible parts of the models do not have textures, making it only focus on the visible parts.
Improves the model and texture of the holographic (Black color)
Improves the texture and eliminates the z-fighting of the model (Tan color)
🎯 Goals
Perform texture optimization tests with small models
Improve texture quality (A little closer to the original Vic Modern Warfare style)
❌ Non Goals
Breaking holographic sight textures in-game
🚦 Testing
It works correctly...
⏮️ Backwards Compatibility
Fully compatible with older versions
📚 Related Issues & Documents
N/A
🖼️ Screenshots/Recordings
📖 Added to documentation?
😄 [optional] What gif best describes this PR or how it makes you feel?
N/A
By the way, if you have no idea what to do with the mod, let the designers' work revive the mod or let me manage the project for a few days XD.
Now if you don't accept the PR I don't know how I can help you with the mod either.