Merge pull request #364 from gwenn/non_exhaustive

Make some enums non_exhaustive

src/config.rs

@@ -181,6 +181,7 @@ pub enum HistoryDuplicates {
 }
 
 #[derive(Clone, Copy, Debug, PartialEq, Eq)]
+#[non_exhaustive]
 pub enum CompletionType {
     /// Complete the next full match (like in Vim by default)
     Circular,
@@ -199,13 +200,15 @@ pub enum CompletionType {
 
 /// Style of editing / Standard keymaps
 #[derive(Clone, Copy, Debug, PartialEq, Eq)]
+#[non_exhaustive]
 pub enum EditMode {
     Emacs,
     Vi,
 }
 
 /// Colorization mode
 #[derive(Clone, Copy, Debug, PartialEq, Eq)]
+#[non_exhaustive]
 pub enum ColorMode {
     Enabled,
     Forced,
@@ -215,6 +218,7 @@ pub enum ColorMode {
 /// Should the editor use stdout or stderr
 // TODO console term::TermTarget
 #[derive(Clone, Copy, Debug, PartialEq, Eq)]
+#[non_exhaustive]
 pub enum OutputStreamType {
     Stderr,
     Stdout,

src/error.rs

@@ -10,6 +10,7 @@ use std::io;
 // #[non_exhaustive]
 #[allow(clippy::module_name_repetitions)]
 #[derive(Debug)]
+#[non_exhaustive]
 pub enum ReadlineError {
     /// I/O Error
     Io(io::Error),

src/keymap.rs

@@ -14,8 +14,8 @@ use crate::tty::{RawReader, Term, Terminal};
 pub type RepeatCount = usize;
 
 /// Commands
-// #[non_exhaustive]
 #[derive(Debug, Clone, PartialEq)]
+#[non_exhaustive]
 pub enum Cmd {
     /// abort
     Abort, // Miscellaneous Command
@@ -233,6 +233,7 @@ impl CharSearch {
 
 /// Where to move
 #[derive(Debug, Clone, PartialEq)]
+#[non_exhaustive]
 pub enum Movement {
     WholeLine, // not really a movement
     /// beginning-of-line

src/keys.rs

@@ -1,7 +1,7 @@
 //! Key constants
 
-// #[non_exhaustive]
 #[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
+#[non_exhaustive]
 pub enum KeyPress {
     UnknownEscSeq,
     Backspace, // Ctrl('H')