From: Kevin Day <Kevin@kevux.org>
Date: Thu, 16 Jan 2025 05:11:56 +0000 (-0600)
Subject: Progress: Resume work on completing the remove program.
X-Git-Url: https://git.kevux.org/?a=commitdiff_plain;h=f1a6d49f1a072de8239125588e41e4302b370414;p=kevux-tools

Progress: Resume work on completing the remove program.

This may be simpler to complete than TacocaT.
Make a few improvements while reviewing the state of the code and trying to remember where to resume development.

I should try to wrap up the simulate function entirely as the starting point.
---

diff --git a/sources/c/program/kevux/tools/remove/main/common/define.h b/sources/c/program/kevux/tools/remove/main/common/define.h
index 0d0446f..36317ba 100644
--- a/sources/c/program/kevux/tools/remove/main/common/define.h
+++ b/sources/c/program/kevux/tools/remove/main/common/define.h
@@ -197,7 +197,7 @@ extern "C" {
  *   - changed:                Remove by changed datetime.
  *   - different:              Remove by user different from caller.
  *   - directory:              Remove by file type: directory.
- *   - empty_all:              All empty flag bits are combined.
+ *   - empty_all:              Helper flag representing all empty flag bits.
  *   - empty_only:             Remove empty directories.
  *   - empty_only_fail:        Fail on empty directories.
  *   - empty_not:              Remove not empty directories.
@@ -210,7 +210,7 @@ extern "C" {
  *   - link:                   Remove by file type: link.
  *   - mode:                   Remove by mode.
  *   - option_used:            This gets set when when certain options are specified to toggle the default match detection boolean during removal of each file.
- *   - prompt_all:             All prompt flag bits are combined.
+ *   - prompt_all:             Helper flag representing all prompt flag bits.
  *   - prompt_each:            Operate in interactive mode, prompting for every file.
  *   - prompt_follow:          Operate in interactive mode: prompting for every link that is being followed.
  *   - prompt_never:           Do not operate in interactive mode.
diff --git a/sources/c/program/kevux/tools/remove/main/operate.c b/sources/c/program/kevux/tools/remove/main/operate.c
index 2f93405..c6c7fd5 100644
--- a/sources/c/program/kevux/tools/remove/main/operate.c
+++ b/sources/c/program/kevux/tools/remove/main/operate.c
@@ -386,11 +386,12 @@ extern "C" {
 
     if (flag & kt_remove_flag_file_operate_directory_d) {
       kt_remove_print_simulate_operate_boolean(&main->program.output, kt_remove_recurse_s, flag & kt_remove_flag_file_operate_recurse_d);
-    }
 
-    if (flag & kt_remove_flag_file_operate_directory_d) {
       if (main->setting.flag & kt_remove_main_flag_empty_all_d) {
         // @todo handle simulate for this.
+        // 1. print the particular empty flags set.
+        // 2. determine if, when a directory, is empty or not.
+        // 3. print the empty or not state.
       }
 
       // Recurse effectively forces tree.
diff --git a/sources/c/program/kevux/tools/remove/main/operate.h b/sources/c/program/kevux/tools/remove/main/operate.h
index afd0624..4f8c1d4 100644
--- a/sources/c/program/kevux/tools/remove/main/operate.h
+++ b/sources/c/program/kevux/tools/remove/main/operate.h
@@ -145,7 +145,7 @@ extern "C" {
  * Perform simulation of the file operation.
  *
  * The simulation process is also intended to be used to determine what to do with the actual file.
- * A flag is populated based on the results of the simulation analysis.
+ * Therefore this should be called even when not simulating.
  *
  * @param main
  *   The main program and settings data.
@@ -162,7 +162,7 @@ extern "C" {
  *   The path to the file to operate on.
  *
  * @return
- *   The flags determined by the simulation.
+ *   The resulting flags determined by the simulation analysis.
  *
  * @see f_file_link_read()
  * @see f_file_remove()