diff --git a/src/main/java/net/openhft/chronicle/map/SerializationBuilder.java b/src/main/java/net/openhft/chronicle/map/SerializationBuilder.java index be6eaa22f..15ea99836 100644 --- a/src/main/java/net/openhft/chronicle/map/SerializationBuilder.java +++ b/src/main/java/net/openhft/chronicle/map/SerializationBuilder.java @@ -92,6 +92,8 @@ final class SerializationBuilder implements Cloneable { agileMarshaller((AgileBytesMarshaller) VoidMarshaller.INSTANCE, factory); } else if (eClass == Long.class) { agileMarshaller((AgileBytesMarshaller) LongMarshaller.INSTANCE, factory); + } else if (eClass == Double.class) { + agileMarshaller((AgileBytesMarshaller) DoubleMarshaller.INSTANCE, factory); } else if (eClass == byte[].class) { reader((BytesReader) ByteArrayMarshaller.INSTANCE, factory); interop((BytesInterop) ByteArrayMarshaller.INSTANCE); @@ -113,8 +115,6 @@ final class SerializationBuilder implements Cloneable { return new ExternalizableMarshaller(classForMarshaller); if (eClass == Integer.class) return (BytesMarshaller) IntegerMarshaller.INSTANCE; - if (eClass == Double.class) - return (BytesMarshaller) DoubleMarshaller.INSTANCE; return null; } @@ -351,28 +351,6 @@ final class SerializationBuilder implements Cloneable { } } - private enum DoubleMarshaller implements BytesMarshaller { - INSTANCE; - - @Override - public void write(Bytes bytes, Double v) { - bytes.writeDouble(v); - } - - @Nullable - @Override - public Double read(Bytes bytes) { - return bytes.readDouble(); - } - - @Nullable - @Override - public Double read(Bytes bytes, @Nullable Double v) { - return bytes.readDouble(); - } - } - - private static class SerializableMarshaller implements BytesMarshaller { private static final long serialVersionUID = 0L; diff --git a/src/main/java/net/openhft/chronicle/map/serialization/impl/DoubleMarshaller.java b/src/main/java/net/openhft/chronicle/map/serialization/impl/DoubleMarshaller.java new file mode 100644 index 000000000..ea3080b98 --- /dev/null +++ b/src/main/java/net/openhft/chronicle/map/serialization/impl/DoubleMarshaller.java @@ -0,0 +1,72 @@ +/* + * Copyright 2014 Higher Frequency Trading http://www.higherfrequencytrading.com + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package net.openhft.chronicle.map.serialization.impl; + +import net.openhft.chronicle.map.serialization.AgileBytesMarshaller; +import net.openhft.chronicle.map.serialization.Hasher; +import net.openhft.lang.io.Bytes; + +import static java.lang.Double.doubleToLongBits; + +public enum DoubleMarshaller implements AgileBytesMarshaller { + INSTANCE; + + @Override + public long size(Double e) { + return 8L; + } + + @Override + public int sizeEncodingSize(long size) { + return 0; + } + + @Override + public void writeSize(Bytes bytes, long size) { + // do nothing + } + + @Override + public boolean startsWith(Bytes bytes, Double e) { + return doubleToLongBits(e) == bytes.readLong(0); + } + + @Override + public long hash(Double e) { + return Hasher.hash(doubleToLongBits(e)); + } + + @Override + public void write(Bytes bytes, Double e) { + bytes.writeLong(doubleToLongBits(e)); + } + + @Override + public long readSize(Bytes bytes) { + return 8L; + } + + @Override + public Double read(Bytes bytes, long size) { + return Double.longBitsToDouble(bytes.readLong()); + } + + @Override + public Double read(Bytes bytes, long size, Double e) { + return read(bytes, size); + } +}