Optimizations and code deduplication

Cargo.toml

@@ -3,43 +3,46 @@ name = "whenever"
 description = "Whenever Automation Tool"
 readme = "README.md"
 license = "LGPL-2.1-or-later"
-version = "1.3.1"
+version = "1.3.2"
 authors = ["Francesco Garosi <francesco.garosi@gmail.com>"]
 repository = "https://github.com/almostearthling/whenever/"
 edition = "2024"
 
+
 [dependencies]
-time = "0.3"
+async-std = "1.12"
+async-trait = "0.1"
+bstr = "1.12"
+cfgmap = { version = "0.4", features = ["from_toml"] }
 chrono = "0.4"
-rand = "0.10"
-regex = "1.7"
-log = "0.4"
-flexi_logger = "0.31"
-nu-ansi-term = "0.50"
+clap = { version = "4.1", features = ["derive"] }
 clokwerk = "0.4"
+ctrlc = { version = "3.5", features = ["termination"] }
+flexi_logger = "0.31"
+futures = "0.3"
+itertools = "0.15"
+lazy_static = "1.5"
 listenfd = "1.0"
-toml = "1.0"
+log = "0.4"
+minreq = { version = "3.0", features = ["https-native-tls"], optional = true }
+nu-ansi-term = "0.50"
+notify = "8.1"
+parking_lot = "0.12"
+rand = "0.10"
+regex = "1.7"
 serde = "1.0"
-lazy_static = "1.4"
-subprocess = "1.1"
-ctrlc = { version = "3.5", features = ["termination"] }
-cfgmap = { version = "0.4", features = ["from_toml", "from_json"] }
-clap = { version = "4.1", features = ["derive"] }
-mlua = { version = "0.11", features = ["lua54", "vendored"] }
-unique_id = "0.1"
 serde_json = "1.0"
 single-instance = "0.3"
-notify = "8.1"
-async-std = "1.12"
-zbus = { version = "5.9", optional = true }
-whoami = "2.1"
-itertools = "0.15"
-futures = "0.3"
-async-trait = "0.1"
+subprocess = "1.1"
 system-idle-time = "1.0"
-parking_lot = "0.12"
-bstr = "1.12"
-minreq = { version = "3.0", features = ["https-native-tls"], optional = true }
+time = "0.3"
+toml = "1.0"
+unique_id = "0.1"
+whoami = "2.1"
+zbus = { version = "5.17", optional = true }
+
+# the "lua54" feature can be modified to use a different Lua version
+mlua = { version = "0.12", features = ["lua54", "vendored"] }
 
 [target.'cfg(windows)'.dependencies]
 wmi = { version = "0.18", optional = true }
@@ -65,9 +68,10 @@ winresource = "0.1"
 # - cargo build --release --features windows_std    (Windows)
 #
 # as the default configuration does not include any option.
+
 [features]
 default = []
-# default = ["wmi", "dbus", "lua_sync", "lua_httpreq"]
+# default = ["check_std"]
 dbus = ["dep:zbus"]
 wmi = ["dep:wmi"]
 lua_sync = []
@@ -80,10 +84,10 @@ check_std = ["wmi", "dbus", "lua_sync", "lua_httpreq"]
 
 # make the executable memory footprint as small as possible for release
 [profile.release]
-strip = true        # strip executable
-opt-level = "z"     # optimize for size
+strip = true    # strip executable
+opt-level = "z" # optimize for size
 lto = true
-panic = 'abort'     # remove garbage strings from executable
+panic = 'abort' # remove garbage strings from executable
 
 
 # end

docs/65.lua.rst

@@ -11,6 +11,11 @@ is running, it only needs to be initialized and provided the script that, in tur
 loaded at startup. Tasks (or checks) that would last even seconds, can be performed in a small
 fraction of the time.
 
+.. note::
+  The default *Lua* interpreter is *Lua v5.4*: this is intentional, as this version can be
+  considered stable and well supported. However the interpreter version can be changed at build
+  time by modifying the ``"lua54"`` entry in *Cargo.toml*.
+
 The embedded interpreter makes the standard *Lua* library available for scripts, which includes:
 
 * string manipulation
@@ -306,7 +311,7 @@ script from within **whenever**: for more complex needs, external commands can b
 anyway.
 
 .. note::
-  The *http* module is only available when the ``lua_httpreq`` feature is enabled; however, 
+  The *http* module is only available when the ``lua_httpreq`` feature is enabled; however,
   both the standard configurations and the provided binaries come with the feature enabled.
 
 

docs/conf.py

@@ -9,7 +9,7 @@
 project = "whenever"
 author = "Francesco Garosi"
 copyright = "2023-%Y, Francesco Garosi"
-release = "1.3.1"
+release = "1.3.2"
 
 html_logo = "graphics/metronome.png"
 

src/cfghelp.rs

@@ -8,7 +8,7 @@
 use cfgmap::CfgMap;
 use regex::Regex;
 
-use crate::common::wres::{Error, Kind, Result};
+use crate::utility::result::{Error, Kind, Result};
 use crate::constants::*;
 
 /// use this to specify that a configuration element is mandatory