Fixed warnings (#3345)

substrate/core/cli/src/execution_strategy.rs

@@ -0,0 +1,35 @@
+// Copyright 2018-2019 Parity Technologies (UK) Ltd.
+// This file is part of Substrate.
+
+// Substrate is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+
+// Substrate is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+
+// You should have received a copy of the GNU General Public License
+// along with Substrate.  If not, see <http://www.gnu.org/licenses/>.
+
+#![allow(missing_docs)]
+
+use structopt::clap::{arg_enum, _clap_count_exprs};
+
+arg_enum! {
+	/// How to execute blocks
+	#[derive(Debug, Clone, Copy)]
+	pub enum ExecutionStrategy {
+		// Execute with native build (if available, WebAssembly otherwise).
+		Native,
+		// Only execute with the WebAssembly build.
+		Wasm,
+		// Execute with both native (where available) and WebAssembly builds.
+		Both,
+		// Execute with the native build if possible; if it fails, then execute with WebAssembly.
+		NativeElseWasm,
+	}
+}
+

substrate/core/cli/src/lib.rs

@@ -22,6 +22,7 @@
 #[macro_use]
 mod traits;
 mod params;
+mod execution_strategy;
 pub mod error;
 pub mod informant;
 

substrate/core/cli/src/params.rs

@@ -20,6 +20,8 @@ use std::path::PathBuf;
 use structopt::{StructOpt, clap::{arg_enum, _clap_count_exprs, App, AppSettings, SubCommand, Arg}};
 use client;
 
+pub use crate::execution_strategy::ExecutionStrategy;
+
 /// Auxiliary macro to implement `GetLogFilter` for all types that have the `shared_params` field.
 macro_rules! impl_get_log_filter {
 	( $type:ident ) => {
@@ -31,22 +33,6 @@ macro_rules! impl_get_log_filter {
 	}
 }
 
-arg_enum! {
-	/// How to execute blocks
-	#[allow(missing_docs)]
-	#[derive(Debug, Clone, Copy)]
-	pub enum ExecutionStrategy {
-		// Execute with native build (if available, WebAssembly otherwise).
-		Native,
-		// Only execute with the WebAssembly build.
-		Wasm,
-		// Execute with both native (where available) and WebAssembly builds.
-		Both,
-		// Execute with the native build if possible; if it fails, then execute with WebAssembly.
-		NativeElseWasm,
-	}
-}
-
 impl Into<client::ExecutionStrategy> for ExecutionStrategy {
 	fn into(self) -> client::ExecutionStrategy {
 		match self {