From c15efc0f40aea427e15c81acefc27d4a3a0a61c6 Mon Sep 17 00:00:00 2001 From: leventov Date: Sun, 12 Oct 2014 16:10:17 +0400 Subject: [PATCH] Removed ChronicleSetBuilder.alignment() config because it is pointless for this abstraction --- .../chronicle/set/ChronicleSetBuilder.java | 19 ++----------------- 1 file changed, 2 insertions(+), 17 deletions(-) diff --git a/src/main/java/net/openhft/chronicle/set/ChronicleSetBuilder.java b/src/main/java/net/openhft/chronicle/set/ChronicleSetBuilder.java index c71134b82..6b37d2fb7 100644 --- a/src/main/java/net/openhft/chronicle/set/ChronicleSetBuilder.java +++ b/src/main/java/net/openhft/chronicle/set/ChronicleSetBuilder.java @@ -37,7 +37,8 @@ public class ChronicleSetBuilder implements Cloneable { private ChronicleMapBuilder chronicleMapBuilder; ChronicleSetBuilder(Class keyClass) { - chronicleMapBuilder = ChronicleMapBuilder.of(keyClass, Void.class); + chronicleMapBuilder = ChronicleMapBuilder.of(keyClass, Void.class) + .entryAndValueAlignment(Alignment.NO_ALIGNMENT); } public static ChronicleSetBuilder of(Class keyClass) { @@ -127,22 +128,6 @@ public class ChronicleSetBuilder implements Cloneable { return this; } - - /** - * Configures alignment strategy of address in memory of elements - * - *

Useful when elements are updated intensively, particularly fields with volatile access, because it - * doesn't work well if the value elements cache lines. Also, on some (nowadays rare) architectures any - * misaligned memory access is more expensive than aligned. - * - * @param alignment the new alignment of the seta constructed by this builder - * @return this {@code ChronicleSetBuilder} back - */ - public ChronicleSetBuilder alignment(Alignment alignment) { - chronicleMapBuilder.entryAndValueAlignment(alignment); - return this; - } - public ChronicleSetBuilder elements(long entries) { chronicleMapBuilder.entries(entries); return this;