eclipse-ocl / org.eclipse.ocl

Eclipse Public License 2.0
0 stars 0 forks source link

[project] Replace StringBuffer by StringBuilder #812

Closed eclipse-ocl-bot closed 7 hours ago

eclipse-ocl-bot commented 7 hours ago

| --- | --- | | Bugzilla Link | 364806 | | Status | CLOSED FIXED | | Importance | P3 normal | | Reported | Nov 25, 2011 05:25 EDT | | Modified | May 20, 2013 11:36 EDT | | Version | 4.0.0 | | Depends on | 367705 | | Reporter | Ed Willink |

Description

StringBuffer is synchronized, StringBuilder is not, therefore replace all usage of StringBuffer by StringBuilder unless there is a specific need for synchronization.

eclipse-ocl-bot commented 7 hours ago

By Ed Willink on Jan 02, 2012 09:18

Numerous local changes in examples/tests committed to bug 318092 branch, whih will be merged to master once I can build on Hudson again.

Additional changes to core/impact plugins await review in (the last commit to) bug/364806. A couple of minor APIs use StringBuffer. These cannot be changed, but could be duplicated; I don't think the duplication is justified.

Further changes in auto-generated code await a template change for bug 367705.

eclipse-ocl-bot commented 7 hours ago

By Axel Uhl on Jan 05, 2012 09:32

The branch for this bug mixes changes regarding the obvious StringBuffer-->StringBuilder with changes regarding the way how delegates are resolved and how their URIs are being handled. I don't see the connection with the StringBuilder changes and suggest to move the delegate stuff to a separate bug / branch and keep this branch's changes limited to the subject of this bug.

eclipse-ocl-bot commented 7 hours ago

By Axel Uhl on Jan 05, 2012 09:42

Sorry, my bad; had'n merged the latest origin/master into my local master... Now reviewing the right diff. All I see in the core bundles are StringBuffer-->StringBuilder diffs.

+1 for merge to master.

eclipse-ocl-bot commented 7 hours ago

By Ed Willink on Jan 05, 2012 16:38

Pushed to master for M5.

eclipse-ocl-bot commented 7 hours ago

By Ed Willink on May 20, 2013 11:36

CLOSED after a year in the RESOLVED state.