Merge pull request #321 from danieleades/clippy/lossless_conversions

use lossless conversions where available

Author: dwrensha

capnp-futures/src/serialize_packed.rs

@@ -135,9 +135,9 @@ impl <R> AsyncRead for PackedRead<R> where R: AsyncRead + Unpin {
                     while ii < outbuf.len() && bitnum < 8
                     {
                         let is_nonzero = (buf[0] & (1u8 << bitnum)) != 0;
-                        outbuf[ii] = buf[*buf_pos] & ((-(is_nonzero as i8)) as u8);
+                        outbuf[ii] = buf[*buf_pos] & ((-i8::from(is_nonzero)) as u8);
                         ii += 1;
-                        *buf_pos += is_nonzero as usize;
+                        *buf_pos += usize::from(is_nonzero);
                         bitnum += 1;
                     }
                     if bitnum == 8 {

capnp/src/private/arena.rs

@@ -103,7 +103,7 @@ impl <S> ReaderArena for ReaderArenaImpl<S> where S: ReaderSegments {
         let (segment_start, segment_len) = self.get_segment(segment_id)?;
         let this_start: usize = segment_start as usize;
         let this_size: usize = segment_len as usize * BYTES_PER_WORD;
-        let offset: i64 = offset_in_words as i64 * BYTES_PER_WORD as i64;
+        let offset: i64 = i64::from(offset_in_words) * BYTES_PER_WORD as i64;
         let start_idx = start as usize;
         if start_idx < this_start || ((start_idx - this_start) as i64 + offset) as usize > this_size {
             Err(Error::failed(String::from("message contained out-of-bounds pointer")))
@@ -219,7 +219,7 @@ impl <A> ReaderArena for BuilderArenaImpl<A> where A: Allocator {
     }
 
     fn check_offset(&self, _segment_id: u32, start: *const u8, offset_in_words: i32) -> Result<*const u8> {
-        unsafe { Ok(start.offset((offset_in_words as i64 * BYTES_PER_WORD as i64) as isize)) }
+        unsafe { Ok(start.offset((i64::from(offset_in_words) * BYTES_PER_WORD as i64) as isize)) }
     }
 
     fn contains_interval(&self, _id: u32, _start: *const u8, _size: usize) -> Result<()> {