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.

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

I have switched usages of Vector to ArrayList. #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Shrulik
Copy link

@Shrulik Shrulik commented Dec 21, 2018

Vector in Java is synchronized for thread safety, and is much slower. I also did some other code clean up where something jumped at me or my IDE.
Here is a reference : https://stackoverflow.com/a/17472102/657657

I read the paper for a course, and skimmed the code and it just jumped at me, so I decided to fix it.
I saw some array work, that could probably benefit from System.arraycopy() if it is in a performance critical area, didn't look enough into it.

Cheers.

…hronized for thread safety, and is much slower. I also did some other code clean up where something jumped at me or my IDE.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant