Skip to content

Conversation

grigoni
Copy link
Contributor

@grigoni grigoni commented Jul 8, 2025

about StringBuffer usage, this PR splits changes per type and module

use StringBuilder instead of StringBuffer: this avoids unnecessary sync
use StringBuilder best practice when concatenating
avoid concatenation in logging
use java.nio.charset.StandardCharsets for getBytes
use isEmpty() instead of ...length() > 0
use "0123456789".repeat(Math.max(0, loopSize)); instead of a loop with a SB
some typos fixing
see https://issues.apache.org/jira/browse/AMQ-9732

mattrpav and others added 15 commits July 3, 2025 07:22
use StringBuilder in method createCompositeDestination instead of basic String concatenation +=
use StringBuilder instead of StringBuffer in loadJmsMessage method
apply best practices in string collation
avoid StringBuffer and use java 11 String#repeat
avoid StringBuffer and use java 11 String#repeat
rewrite some logging to use params instead of concatenation
use StandardCharsets.UTF_8/US_ASCII which is more robust and does not throw UnsupportedEncodingException
remove StringBuffer where no loop is involved since java compiler will optimize better
rewrite some logging to use params instead of concatenation
@grigoni grigoni changed the title Feature/cleanup part2 stringbuffer splitted Stringbuffer cleanup part2 (splitted commits) Jul 8, 2025
giana added 13 commits July 9, 2025 23:00
use StringBuilder in method createCompositeDestination instead of basic String concatenation +=
use StringBuilder instead of StringBuffer in loadJmsMessage method
apply best practices in string collation
avoid StringBuffer and use java 11 String#repeat
avoid StringBuffer and use java 11 String#repeat
rewrite some logging to use params instead of concatenation
use StandardCharsets.UTF_8/US_ASCII which is more robust and does not throw UnsupportedEncodingException
remove StringBuffer where no loop is involved since java compiler will optimize better
rewrite some logging to use params instead of concatenation
jbonofre and others added 28 commits September 25, 2025 07:56
Bumps [org.springframework:spring-core](https://github.com/spring-projects/spring-framework) from 6.2.10 to 6.2.11.
- [Release notes](https://github.com/spring-projects/spring-framework/releases)
- [Commits](spring-projects/spring-framework@v6.2.10...v6.2.11)

---
updated-dependencies:
- dependency-name: org.springframework:spring-core
  dependency-version: 6.2.11
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…aged, and temporary queues and topics (apache#1288)

* Add attributes to display total number of queues and topics

* Addressed comments and add metrics for temporary queues and topics

* Use assertEquals and make the test more accurate

* renamed NonSuppreseed to Managed

* fixed logic for counting ManagedTopic and ManagedQueue
use StringBuilder in method createCompositeDestination instead of basic String concatenation +=
use StringBuilder instead of StringBuffer in loadJmsMessage method
apply best practices in string collation
avoid StringBuffer and use java 11 String#repeat
avoid StringBuffer and use java 11 String#repeat
rewrite some logging to use params instead of concatenation
use StandardCharsets.UTF_8/US_ASCII which is more robust and does not throw UnsupportedEncodingException
remove StringBuffer where no loop is involved since java compiler will optimize better
rewrite some logging to use params instead of concatenation
…fer_splitted' into feature/cleanup_part2_stringbuffer_splitted
@grigoni grigoni closed this Oct 5, 2025
@grigoni grigoni deleted the feature/cleanup_part2_stringbuffer_splitted branch October 5, 2025 16:34
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.