Skip to content

java: use triple-quote strings instead of concat#7481

Merged
a10y merged 1 commit intodevelopfrom
aduffy/tripquote
Apr 16, 2026
Merged

java: use triple-quote strings instead of concat#7481
a10y merged 1 commit intodevelopfrom
aduffy/tripquote

Conversation

@a10y
Copy link
Copy Markdown
Contributor

@a10y a10y commented Apr 16, 2026

A little easier to read/edit

Signed-off-by: Andrew Duffy <andrew@a10y.dev>
@a10y a10y requested a review from robert3005 April 16, 2026 14:39
@a10y a10y enabled auto-merge (squash) April 16, 2026 14:39
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Apr 16, 2026

Merging this PR will improve performance by 20.02%

⚡ 9 improved benchmarks
✅ 1154 untouched benchmarks
⏩ 1457 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation take_map[(0.1, 0.5)] 1,154.5 µs 980.3 µs +17.77%
Simulation take_map[(0.1, 1.0)] 2 ms 1.6 ms +20.02%
Simulation patched_take_10k_first_chunk_only 302 µs 271.8 µs +11.14%
Simulation patched_take_10k_contiguous_not_patches 258.4 µs 228.1 µs +13.28%
Simulation patched_take_10k_dispersed 316 µs 285.8 µs +10.58%
Simulation patched_take_10k_contiguous_patches 258.1 µs 227.7 µs +13.32%
Simulation patched_take_10k_random 270.3 µs 240 µs +12.64%
Simulation take_10k_first_chunk_only 270.6 µs 225.7 µs +19.89%
Simulation take_10k_dispersed 284.4 µs 239.5 µs +18.76%

Comparing aduffy/tripquote (e382268) with develop (8b09d0c)

Open in CodSpeed

Footnotes

  1. 1457 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@robert3005 robert3005 added the changelog/skip Do not list PR in the changelog label Apr 16, 2026
@a10y a10y merged commit 63e5aee into develop Apr 16, 2026
65 of 67 checks passed
@a10y a10y deleted the aduffy/tripquote branch April 16, 2026 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/skip Do not list PR in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants