Bug #5565

Replace use of gnu.trove2 with gnu.trove3 where applicable

Added by Tuukka Lehtonen almost 3 years ago. Updated over 2 years ago.

Status:ClosedStart date:2014-12-31
Priority:4Due date:2015-01-16
Assignee:Tuukka Lehtonen% Done:

100%

Category:-Spent time:-
Target version:1.18.1
Release notes:All dependencies on gnu.trove2 replaced with gnu.trove3.
Tags: refactor
Story points-
Velocity based estimate-
ReleaseSimantics 1.18.1Release relationshipAuto

Description

We're currently using both gnu.trove2 and gnu.trove3 for no apparent reason.
The database client even extends gnu.trove2 structures so that might require a bit of testing but otherwise the replacement should just be a matter of reorganizing imports.

org.simantics.db-gnu.trove2-removal.patch Magnifier (10.6 KB) Tuukka Lehtonen, 2014-12-31 04:04 pm


Related issues

Related to Platform - Enhancement #5540: Cleaning up modeling related code Closed 2014-12-10 2015-01-16

Associated revisions

Revision 30745
Added by Tuukka Lehtonen almost 3 years ago

Remove gnu.trove2 dependency from most simantics plug-ins.
Some dependencies are still left in org.simantics.db.{impl,procore} which need to be examined further.

refs #5565

Revision 30747
Added by Tuukka Lehtonen almost 3 years ago

Remove gnu.trove2 dependency from org.simantics.structural2.

refs #5565

Revision 30985
Added by Tuukka Lehtonen over 2 years ago

Got rid of gnu.trove2 in org.simantics.db.procore.

refs #5565

Revision 30986
Added by Tuukka Lehtonen over 2 years ago

Got rid of gnu.trove2 in org.simantics.db.procore.

refs #5565

Revision 30988
Added by Tuukka Lehtonen over 2 years ago

Got rid of gnu.trove2 in org.simantics.db.impl.

refs #5565

Revision 30989
Added by Tuukka Lehtonen over 2 years ago

Merged trunk@30985,30986,30987,30988 to branches/simantics-1.18 on 2015-02-26 for 1.18.1 release.

refs #5565

History

#1 Updated by Tuukka Lehtonen almost 3 years ago

Attached a patch containing my proposal for the removal of the remaining dependencies on thew gnu.trove2 plug-in. I was not 100% sure whether there would be some binary compatibility issues with these so I left them uncommitted for now. Please review.

#2 Updated by Tuukka Lehtonen over 2 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100
  • Release notes set to All dependencies on gnu.trove2 replaced with gnu.trove3.

All gnu.trove2 dependencies should be removed now from both trunk and simantics-1.18 branch.

#3 Updated by Tuukka Lehtonen over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF