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

Uses of different Vector implementations across codebase #86

Closed
quaeler opened this issue Oct 13, 2017 · 1 comment · May be fixed by #328
Closed

Uses of different Vector implementations across codebase #86

quaeler opened this issue Oct 13, 2017 · 1 comment · May be fixed by #328
Labels
enhancement Lets change things for the better Tools The command line tools - TLC, SANY, ...

Comments

@quaeler
Copy link
Contributor

quaeler commented Oct 13, 2017

Some of our code uses java.util.Vector<E> and other of our code uses the homegrown version.

We should probably standardize on one.
@quaeler quaeler mentioned this issue Oct 13, 2017
Closed
@lemmy lemmy added enhancement Lets change things for the better Tools The command line tools - TLC, SANY, ... labels Oct 14, 2017
@tautomaton tautomaton mentioned this issue Jun 16, 2019
Open
@lemmy
Copy link
Member

lemmy commented Jul 18, 2019

Reducing the code size is a desirable goal in general. However, a tiny reduction in size does not justifies the risk of introducing subtle bugs. Whenever work is done to add a new feature, one should refactor related code when possible and warranted.

@lemmy lemmy closed this as completed Jul 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Lets change things for the better Tools The command line tools - TLC, SANY, ...
Projects
None yet
Milestone
No milestone
2 participants
@lemmy @quaeler