Skip to content

Commit 54883cd

Browse files
committed
Fixing silly oversight - commented out benchmarks to run one in isolation earlier
1 parent 372dff0 commit 54883cd

File tree

1 file changed

+69
-69
lines changed

1 file changed

+69
-69
lines changed

internal-api/src/jmh/java/datadog/trace/util/ListIterationBenchmark.java

Lines changed: 69 additions & 69 deletions
Original file line numberDiff line numberDiff line change
@@ -335,75 +335,75 @@ public enum ListSpec {
335335

336336
@Param ListSpec listSpec;
337337

338-
// @Benchmark
339-
// public void forEach_inline() {
340-
// this.listSpec.list.forEach(Element::manipulate_inline);
341-
// }
342-
//
343-
// @Benchmark
344-
// public void forEach_dont_inline() {
345-
// this.listSpec.list.forEach(Element::manipulate_dont_inline);
346-
// }
347-
//
348-
// @Benchmark
349-
// public void enhancedFor_inline() {
350-
// // Enhanced for-loop is just syntax sugar for an Iterator
351-
// for ( Element e : this.listSpec.list ) {
352-
// e.manipulate_inline();
353-
// }
354-
// }
355-
//
356-
// @Benchmark
357-
// public void enhancedFor_dont_inline() {
358-
// // Enhanced for-loop is just syntax sugar for an Iterator
359-
// for ( Element e : this.listSpec.list ) {
360-
// e.manipulate_dont_inline();
361-
// }
362-
// }
363-
//
364-
// @Benchmark
365-
// public void iterator_inline() {
366-
// for ( Iterator<Element> iter = this.listSpec.list.iterator(); iter.hasNext(); ) {
367-
// iter.next().manipulate_inline();
368-
// }
369-
// }
370-
//
371-
// @Benchmark
372-
// public void iterator_dont_inline() {
373-
// for ( Iterator<Element> iter = this.listSpec.list.iterator(); iter.hasNext(); ) {
374-
// iter.next().manipulate_dont_inline();
375-
// }
376-
// }
377-
//
378-
//
379-
// @Benchmark
380-
// public void cstyleFor_inline() {
381-
// for ( int i = 0; i < this.listSpec.list.size(); ++i ) {
382-
// this.listSpec.list.get(i).manipulate_inline();
383-
// }
384-
// }
385-
//
386-
// @Benchmark
387-
// public void cstyleFor_dont_inline() {
388-
// for ( int i = 0; i < this.listSpec.list.size(); ++i ) {
389-
// this.listSpec.list.get(i).manipulate_dont_inline();
390-
// }
391-
// }
392-
//
393-
// @Benchmark
394-
// public void streams_inline() {
395-
// this.listSpec.list.stream().forEach(Element::manipulate_inline);
396-
// }
397-
//
398-
// @Benchmark
399-
// public void streams_dont_inline() {
400-
// this.listSpec.list.stream().forEach(Element::manipulate_dont_inline);
401-
// }
402-
//
403-
// @Benchmark
404-
// public void parallelStreams_inline() {
405-
// listSpec.list.parallelStream().forEach(Element::manipulate_dont_inline);
406-
// }
338+
@Benchmark
339+
public void forEach_inline() {
340+
this.listSpec.list.forEach(Element::manipulate_inline);
341+
}
342+
343+
@Benchmark
344+
public void forEach_dont_inline() {
345+
this.listSpec.list.forEach(Element::manipulate_dont_inline);
346+
}
347+
348+
@Benchmark
349+
public void enhancedFor_inline() {
350+
// Enhanced for-loop is just syntax sugar for an Iterator
351+
for ( Element e : this.listSpec.list ) {
352+
e.manipulate_inline();
353+
}
354+
}
355+
356+
@Benchmark
357+
public void enhancedFor_dont_inline() {
358+
// Enhanced for-loop is just syntax sugar for an Iterator
359+
for ( Element e : this.listSpec.list ) {
360+
e.manipulate_dont_inline();
361+
}
362+
}
363+
364+
@Benchmark
365+
public void iterator_inline() {
366+
for ( Iterator<Element> iter = this.listSpec.list.iterator(); iter.hasNext(); ) {
367+
iter.next().manipulate_inline();
368+
}
369+
}
370+
371+
@Benchmark
372+
public void iterator_dont_inline() {
373+
for ( Iterator<Element> iter = this.listSpec.list.iterator(); iter.hasNext(); ) {
374+
iter.next().manipulate_dont_inline();
375+
}
376+
}
377+
378+
379+
@Benchmark
380+
public void cstyleFor_inline() {
381+
for ( int i = 0; i < this.listSpec.list.size(); ++i ) {
382+
this.listSpec.list.get(i).manipulate_inline();
383+
}
384+
}
385+
386+
@Benchmark
387+
public void cstyleFor_dont_inline() {
388+
for ( int i = 0; i < this.listSpec.list.size(); ++i ) {
389+
this.listSpec.list.get(i).manipulate_dont_inline();
390+
}
391+
}
392+
393+
@Benchmark
394+
public void streams_inline() {
395+
this.listSpec.list.stream().forEach(Element::manipulate_inline);
396+
}
397+
398+
@Benchmark
399+
public void streams_dont_inline() {
400+
this.listSpec.list.stream().forEach(Element::manipulate_dont_inline);
401+
}
402+
403+
@Benchmark
404+
public void parallelStreams_inline() {
405+
listSpec.list.parallelStream().forEach(Element::manipulate_dont_inline);
406+
}
407407

408408
@Benchmark
409409
public void parallelStreams_dont_inline() {

0 commit comments

Comments
 (0)