Skip to content

Conversation

@distay0xGit
Copy link

This PR changes ArrayV to allow operations to be color-coded, with a legend included, if given operations it is able to color-code.

@Gaming32
Copy link
Owner

Seems good, but I do have a few optimizations in mind... (such as not creating an array from a keyset on every frame)

@Gaming32 Gaming32 added enhancement New feature or request new-feature labels Feb 24, 2022
@Gaming32
Copy link
Owner

@distay0xGit Can you fix the checkstyle and build issues?

@Gaming32
Copy link
Owner

This is not a sorts PR. Can you please remove Explicium Sort?

@Gaming32
Copy link
Owner

Git, what are you doing?

We'll see if we can work out a better solution.
Gaming32 and others added 9 commits March 30, 2022 19:59
Git, what the heck?
oops! bad coding practice
AtomicInteger-related type changes I desperately want to avoid a conflict for and removing an interface from ArrayVlist
_why doesn't it want me to change the packages imported for ArrayVList_
the conflict checker is running off black magic and all the spells in it are specifically against pushing Groovy support into this PR
I just hope I can revert this weird hack after this PR is brought up to speed with all the changes in main
@rylid-umil
Copy link

Could the custom color coding be a toggleable option? like to switch between color-coded operations and normal highlighting?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request new-feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants