Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Primitive wrappers #83

Merged
merged 30 commits into from
Apr 25, 2020
Merged

Primitive wrappers #83

merged 30 commits into from
Apr 25, 2020

Conversation

JarvisCraft
Copy link
Owner

@JarvisCraft JarvisCraft commented Apr 17, 2020 •
edited
Loading

This provides some mutable primitive wrappers:
This also adds padla-tools module with unsafe-methods-access-generator for generating Unsafe accessors
Problems adressing the next PR-work:

  • unsafe-methods-access-generator is not yet satisfying
  • Unsafe-related part is not yet implemented
  • Not all primitives are yet covered (considering all other Atomic* types being related to primitives)

@JarvisCraft
Remove redundant statics from ClassUtil
86393dd
@JarvisCraft
Add PrimitiveWrapper
bda83a2
@JarvisCraft
Add IntWrapper
af51394
@JarvisCraft
Add LongWrapper
72dda2e
@JarvisCraft
Add BooleanUnaryOperator
5237761
@JarvisCraft
Add BooleanFunction
480aeca
@JarvisCraft
Make BooleanUnaryOperator a functional interface extending `Boolean…
e4c0616 
…Function`
@JarvisCraft
Add BooleanBinaryOperator
5f21a85
@JarvisCraft
Rely on default value in IntWrapper#create()
1cf993c
@JarvisCraft
Add tools module
5211976
@JarvisCraft
Implement generator of class for accessing Unsafe methods
57d40a9 
This generator should yet be refactored
@JarvisCraft
Cleanup velocity template
a9bed32
@JarvisCraft
Update template extension
8e1843d
@JarvisCraft
Cleanup velocity template fixing issues in it
fb5f911
@JarvisCraft
Remove redundant statics from UnsafeInternals
4389f6a
@JarvisCraft
Add missing final to UnsafeInternals
af1e20c
@JarvisCraft
Fix imbalanced in BooleanBinaryOperator's docs
96736b6
@JarvisCraft
Add missing @code-tags to docs of BooleanUnaryOperator
14adf08
@JarvisCraft
Add BooleanConsumer
e4a038d
@JarvisCraft
Add package-info.java to ru.progrm_jarvis.javacommons.util.function
f55bf1e
@JarvisCraft
Add BooleanWrapper
eb26c8c
@JarvisCraft JarvisCraft added enhancement New feature or request incubator Possible future feature in development labels Apr 17, 2020
@JarvisCraft JarvisCraft self-assigned this Apr 17, 2020
@pull-assistant
Copy link

pull-assistant bot commented Apr 17, 2020 •
edited
Loading

Score: 0.83

Best reviewed: commit by commit

Optimal code review plan

     Remove redundant statics from ClassUtil

     Add PrimitiveWrapper

     Add IntWrapper

     Add LongWrapper

     Add BooleanUnaryOperator

     Add BooleanFunction

     Make BooleanUnaryOperator a functional interface extending `BooleanF...

     Add BooleanBinaryOperator

     Rely on default value in IntWrapper#create()

     Add tools module

     Implement generator of class for accessing Unsafe methods

     Cleanup velocity template

     Update template extension

     Cleanup velocity template fixing issues in it

     Remove redundant statics from UnsafeInternals

     Add missing final to UnsafeInternals

     Fix imbalanced in BooleanBinaryOperator's docs

     Add missing @code-tags to docs of BooleanUnaryOperator

     Add BooleanConsumer

     Add package-info.java to `ru.progrm_jarvis.javacommons.util.function...

     Add BooleanWrapper

     Merge branch 'development' into primitive-wrappers

     Add Numeric interface

     Integrate primitive wrappers with Numeric interface

     Cleanup primitive wrappers adding getAndSet(..)

     Cleanup primitive wrappers

     Merge branch 'development' into primitive-wrappers

     Resolve compilation issues due to Lombok searching for methods from `P...

     Configure Maven build for tools

     Merge remote-tracking branch 'origin/primitive-wrappers' into primitiv...

Powered by Pull Assistant. Last update 86393dd ... a4faea0. Read the comment docs.

@JarvisCraft JarvisCraft marked this pull request as ready for review April 25, 2020 14:17
@JarvisCraft JarvisCraft mentioned this pull request Apr 25, 2020
Open
@JarvisCraft JarvisCraft merged commit 2e52755 into development Apr 25, 2020
@JarvisCraft JarvisCraft deleted the primitive-wrappers branch April 25, 2020 14:20
@JarvisCraft JarvisCraft linked an issue May 6, 2020 that may be closed by this pull request
Utilize IntWrapper and other primitive wrappers from minecraft-utils #82
Closed
@JarvisCraft JarvisCraft added this to the 1.0.0 milestone Nov 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BDimasik BDimasik Awaiting requested review from BDimasik

Assignees

@JarvisCraft JarvisCraft

Labels
enhancement New feature or request incubator Possible future feature in development
Projects
None yet
Milestone
1.0.0
Development

Successfully merging this pull request may close these issues.

Utilize IntWrapper and other primitive wrappers from minecraft-utils
1 participant
@JarvisCraft