This is an automated email from the ASF dual-hosted git repository.

uwe pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new b095994  ARROW-2408: [Rust] Remove build warnings
b095994 is described below

commit b095994d299ad66186068b04b8ff1e406c1d1a23
Author: Maximilian Roos <m...@maxroos.com>
AuthorDate: Mon Apr 9 11:04:54 2018 +0200

    ARROW-2408: [Rust] Remove build warnings
    
    ...and test for warnings in CI
    
    Author: Maximilian Roos <m...@maxroos.com>
    
    Closes #1847 from maxim-lian/rust-warnings and squashes the following 
commits:
    
    01e5582 <Maximilian Roos> remove build warnings
---
 ci/travis_script_rust.sh            | 4 ++++
 rust/examples/array_from_builder.rs | 1 -
 rust/src/buffer.rs                  | 1 -
 rust/src/builder.rs                 | 2 --
 rust/src/datatypes.rs               | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/ci/travis_script_rust.sh b/ci/travis_script_rust.sh
index 4405d03..6a8ecf0 100755
--- a/ci/travis_script_rust.sh
+++ b/ci/travis_script_rust.sh
@@ -23,8 +23,12 @@ RUST_DIR=${TRAVIS_BUILD_DIR}/rust
 
 pushd $RUST_DIR
 
+# raises on any formatting errors
 rustup component add rustfmt-preview
 cargo fmt --all -- --write-mode=diff
+# raises on any warnings
+cargo rustc -- -D warnings 
+
 cargo build
 cargo test
 
diff --git a/rust/examples/array_from_builder.rs 
b/rust/examples/array_from_builder.rs
index 3a273a6..ea1ecec 100644
--- a/rust/examples/array_from_builder.rs
+++ b/rust/examples/array_from_builder.rs
@@ -18,7 +18,6 @@
 extern crate arrow;
 
 use arrow::array::*;
-use arrow::buffer::*;
 use arrow::builder::*;
 
 fn main() {
diff --git a/rust/src/buffer.rs b/rust/src/buffer.rs
index ab90a5b..1f2ec6c 100644
--- a/rust/src/buffer.rs
+++ b/rust/src/buffer.rs
@@ -18,7 +18,6 @@
 use bytes::Bytes;
 use libc;
 use std::mem;
-use std::ptr;
 use std::slice;
 
 use super::memory::*;
diff --git a/rust/src/builder.rs b/rust/src/builder.rs
index 1cc0240..c8ba274 100644
--- a/rust/src/builder.rs
+++ b/rust/src/builder.rs
@@ -15,11 +15,9 @@
 // specific language governing permissions and limitations
 // under the License.
 
-use bytes::Bytes;
 use libc;
 use std::mem;
 use std::ptr;
-use std::slice;
 
 use super::buffer::*;
 use super::memory::*;
diff --git a/rust/src/datatypes.rs b/rust/src/datatypes.rs
index 85278f7..ac2c2c6 100644
--- a/rust/src/datatypes.rs
+++ b/rust/src/datatypes.rs
@@ -16,7 +16,6 @@
 // under the License.
 
 use super::error::ArrowError;
-use serde_json;
 use serde_json::Value;
 use std::fmt;
 
@@ -241,6 +240,7 @@ impl fmt::Display for Schema {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use serde_json;
 
     #[test]
     fn create_struct_type() {

-- 
To stop receiving notification emails like this one, please contact
u...@apache.org.

Reply via email to