code clean part 2

Author: dinaraparanid

Cargo.lock

@@ -1,12 +1,12 @@
 [package]
 name = "booklibrs"
-version = "1.0.14"
+version = "1.0.15"
 authors = ["Paranid5 <dinaraparanid@gmail.com>"]
 edition = "2018"
 license = "MIT/Apache-2.0"
 description = "Book libarary project including giveaway operations and gui interface."
 homepage = "https://github.com/dinaraparanid/Library_rs"
-documentation = "https://docs.rs/binartree/1.0.14/booklibrs/"
+documentation = "https://docs.rs/binartree/1.0.15/booklibrs/"
 
 [dependencies]
 chrono = "0.4.19"

Cargo.toml

@@ -194,6 +194,7 @@ pub fn remove_book(
 /// If you have mistakes in input,
 /// program will let you know
 
+#[allow(dead_code)]
 #[deprecated(
     note = "Used before to add new TheBook. Complains if TheBook with same params exists. Use add_books() instead"
 )]

src/actions/book/add_rem/full.rs

@@ -257,6 +257,7 @@ pub(crate) fn remove_book_simple(
 /// from known the book
 
 #[inline]
+#[allow(dead_code)]
 pub(crate) fn remove_book_simple2(
     index: usize,
     s_index: usize,

src/actions/book/add_rem/simple.rs

@@ -531,6 +531,17 @@ pub fn the_book_info_simple(
         MessageTheBook::ChangePages,
     );
 
+    menu.add_emit(
+        match lang {
+            Lang::English => "&Change/Customize book genres\t",
+            Lang::Russian => "&Изменить/Изменить жанры книги\t",
+        },
+        Shortcut::empty(),
+        MenuFlag::Normal,
+        s,
+        MessageTheBook::CustomizeBookGenre,
+    );
+
     menu.add_emit(
         match lang {
             Lang::English => "&Remove/Remove all books\t",

src/actions/book/info/simple.rs

@@ -2,19 +2,14 @@ extern crate fltk;
 
 use crate::{
     actions::{giveaway::simple::*, read::utils::check_reader},
-    books::{book::Book, book_sys::BookSystem, date::Date, genres::Genres},
+    books::{book_sys::BookSystem, genres::Genres},
     change::{input3::Input3, Inputable},
     reading::read_base::ReaderBase,
     restore::caretaker::Caretaker,
     Lang,
 };
 
-use fltk::{
-    app::{channel, App},
-    dialog::alert,
-    input::*,
-    prelude::*,
-};
+use fltk::{app::App, dialog::alert, input::*};
 
 /// Function that changes
 /// return date for the book

src/actions/giveaway/full.rs

@@ -17,7 +17,6 @@ use fltk::{
     dialog::alert,
     input::{Input, IntInput},
     prelude::*,
-    tree::TreeItem,
 };
 
 use chrono::Local;
@@ -280,6 +279,7 @@ pub(crate) fn give_book_known_reader(
 ///
 /// Gives book to known reader (user input version)
 
+#[allow(dead_code)]
 #[deprecated(note = "Used before. Requires input. Consider using give_book_known_reader() instead")]
 fn give_book_known_reader_input(
     rind: usize,
@@ -710,6 +710,7 @@ pub(crate) fn get_book_known_reader(
 ///
 /// Function that gets book from known reader
 
+#[allow(dead_code)]
 #[deprecated(
     note = "Used before. Requires user input. Consider using get_book_known_reader() instead"
 )]

src/actions/giveaway/simple.rs

@@ -1,4 +1,3 @@
-extern crate chrono;
 extern crate fltk;
 extern crate fltk_calendar;
 
@@ -14,8 +13,6 @@ use crate::{
     Lang,
 };
 
-use chrono::Datelike;
-
 /// Function that adds reader.
 /// If you have mistakes in input,
 /// program will let you know

src/actions/read/add_rem/full.rs

@@ -1,34 +1,17 @@
 extern crate fltk;
 
-use fltk::{
-    app,
-    app::App,
-    dialog::alert,
-    draw,
-    frame::Frame,
-    group::VGrid,
-    input::{Input, IntInput},
-    menu::{MenuBar, MenuFlag},
-    prelude::*,
-    table::Table,
-    window::SingleWindow,
-};
+use fltk::{app, app::App, input::Input, prelude::*};
 
 use crate::{
-    actions::{
-        book::info::simple::book_info_simple,
-        giveaway::simple::{get_book_known_reader, give_book_known_reader},
-        read::{change::*, info::simple::reader_info_simple, utils::check_reader},
-        tables::*,
-    },
-    books::{book_sys::BookSystem, date::Date, genres::Genres},
+    actions::read::{info::simple::reader_info_simple, utils::check_reader},
+    books::{book_sys::BookSystem, genres::Genres},
     change::{input3::Input3, Inputable},
     reading::read_base::ReaderBase,
     restore::caretaker::Caretaker,
     Lang,
 };
 
-use std::{cell::RefCell, cmp::max, rc::Rc};
+use std::{cell::RefCell, rc::Rc};
 
 /// Function that gives info about reader.
 /// If you have mistakes in input,

src/actions/read/info/full.rs

@@ -132,6 +132,7 @@ impl Genres {
     /// Deletes all genres from current genres keeper
 
     #[inline]
+    #[allow(dead_code)]
     pub(crate) fn clear(&mut self) -> &mut Self {
         self.genres.clear();
         self

src/books/genres.rs

@@ -9,6 +9,8 @@ pub mod change;
 pub mod reading;
 pub mod restore;
 
+#[allow(dead_code)]
+#[allow(unused_imports)]
 #[deprecated(note = "It has been a lot of changes since I need it,\
      since everything works correct, I have no need for tests")]
 mod tests;

src/lib.rs

@@ -138,6 +138,7 @@ impl Reader {
     /// else none
 
     #[inline]
+    #[allow(dead_code)]
     pub(crate) fn find_book_first(&self, book: &Rc<RefCell<Book>>) -> Option<usize> {
         self.books
             .iter()
@@ -149,6 +150,7 @@ impl Reader {
     /// else none
 
     #[inline]
+    #[allow(dead_code)]
     pub(crate) fn find_book_last(&self, book: &Rc<RefCell<Book>>) -> Option<usize> {
         self.books
             .iter()

src/reading/reader.rs

@@ -1,2 +1,4 @@
+#[allow(dead_code)]
 mod book_test;
+#[allow(dead_code)]
 mod reader_test;