From e86ae3f21d330f26582ee623d87f986c0331dd14 Mon Sep 17 00:00:00 2001
From: Paul Masurel <paul.masurel@gmail.com>
Date: Tue, 15 Mar 2016 18:05:15 +0900
Subject: [PATCH] blop

---
 src/core/fastfield.rs | 14 ++++++--------
 src/core/serialize.rs | 10 +++++-----
 2 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/src/core/fastfield.rs b/src/core/fastfield.rs
index df023257a..314bce2a2 100644
--- a/src/core/fastfield.rs
+++ b/src/core/fastfield.rs
@@ -1,11 +1,10 @@
 use std::io::Write;
 use std::io;
 use std::io::Cursor;
-use std::io::Seek;
-use std::io::SeekFrom;
 use core::serialize::BinarySerializable;
 use core::directory::ReadOnlySource;
 use core::schema::DocId;
+use std::ops::Deref;
 
 
 struct IntFastFieldWriter {
@@ -22,7 +21,6 @@ pub fn compute_num_bits(amplitude: u32) -> u8 {
     }
 }
 
-// only works for big-endian
 fn serialize_packed_ints<I: Iterator<Item=u32>>(vals_it: I, num_bits: u8, write: &mut Write) -> io::Result<()> {
     let mut mini_buffer_written = 0;
     let mut mini_buffer = 0u64;
@@ -70,6 +68,7 @@ impl IntFastFieldWriter {
 
 pub struct IntFastFieldReader {
     data: ReadOnlySource,
+    data_ptr: *const u64,
     min_val: u32,
     num_bits: u32,
     mask: u32,
@@ -83,22 +82,22 @@ impl IntFastFieldReader {
         let num_bits = try!(u8::deserialize(&mut cursor));
         let mask = (1 << num_bits) - 1;
         let num_in_pack = 64u32 / (num_bits as u32);
+        let ptr: *const u8 = &(data.deref()[5]);
         Ok(IntFastFieldReader {
             min_val: min_val,
             num_bits: num_bits as u32,
             data: data.slice(5, data.len()),
+            data_ptr: ptr as *const u64,
             mask: mask,
             num_in_pack: num_in_pack,
         })
     }
 
     pub fn get(&self, doc: DocId) -> u32 {
-        let mut cursor = Cursor::new(&*self.data);
         let long_addr = doc / self.num_in_pack;
         let ord_within_long = doc - long_addr * self.num_in_pack;
         let bit_shift = (self.num_bits as u32) * ord_within_long;
-        cursor.seek(SeekFrom::Start((long_addr as u64)   * 8u64)).unwrap();
-        let val_unshifted_unmasked = u64::deserialize(&mut cursor).unwrap();
+        let val_unshifted_unmasked: u64 = unsafe { *self.data_ptr.offset(long_addr as isize) };
         let val_shifted = (val_unshifted_unmasked >> bit_shift) as u32;
         return self.min_val + (val_shifted & self.mask);
     }
@@ -116,6 +115,7 @@ mod tests {
     use rand::Rng;
     use rand::SeedableRng;
     use rand::XorShiftRng;
+    use core::serialize::BinarySerializable;
 
     #[test]
     fn test_compute_num_bits() {
@@ -169,7 +169,6 @@ mod tests {
         }
     }
 
-
     fn generate_permutation() -> Vec<u32> {
         let seed: &[u32; 4] = &[1, 2, 3, 4];
         let mut rng = XorShiftRng::from_seed(*seed);
@@ -200,7 +199,6 @@ mod tests {
         }
     }
 
-
     #[bench]
     fn bench_intfastfield_veclookup(b: &mut Bencher) {
         let permutation = generate_permutation();
diff --git a/src/core/serialize.rs b/src/core/serialize.rs
index 27b0a4ecb..b23c7305a 100644
--- a/src/core/serialize.rs
+++ b/src/core/serialize.rs
@@ -1,5 +1,5 @@
 
-use byteorder::{BigEndian, ReadBytesExt, WriteBytesExt};
+use byteorder::{NativeEndian, ReadBytesExt, WriteBytesExt};
 use std::fmt;
 use std::io::Write;
 use std::io::Read;
@@ -51,13 +51,13 @@ impl<T: BinarySerializable> BinarySerializable for Vec<T> {
 
 impl BinarySerializable for u32 {
     fn serialize(&self, writer: &mut Write) -> io::Result<usize> {
-        writer.write_u32::<BigEndian>(self.clone())
+        writer.write_u32::<NativeEndian>(self.clone())
               .map(|_| 4)
               .map_err(convert_byte_order_error)
     }
 
     fn deserialize(reader: &mut Read) -> io::Result<u32> {
-        reader.read_u32::<BigEndian>()
+        reader.read_u32::<NativeEndian>()
               .map_err(convert_byte_order_error)
     }
 }
@@ -65,12 +65,12 @@ impl BinarySerializable for u32 {
 
 impl BinarySerializable for u64 {
     fn serialize(&self, writer: &mut Write) -> io::Result<usize> {
-        writer.write_u64::<BigEndian>(self.clone())
+        writer.write_u64::<NativeEndian>(self.clone())
               .map(|_| 8)
               .map_err(convert_byte_order_error)
     }
     fn deserialize(reader: &mut Read) -> io::Result<u64> {
-        reader.read_u64::<BigEndian>()
+        reader.read_u64::<NativeEndian>()
               .map_err(convert_byte_order_error)
     }
 }