From 725c79b9d0504d9d1c2a3e2a149ebb7e441c78e4 Mon Sep 17 00:00:00 2001
From: PartialVolume <22084881+PartialVolume@users.noreply.github.com>
Date: Tue, 17 Oct 2023 09:46:11 +0100
Subject: [PATCH] Change_HPA_message_to_stop_flashing_&_move_to_end_of_line

The HPA message alternating with the drive model serial
number made reading the serial number quite a challenge
and made the display far too messy looking when displaying
20+ drives or even 10+ drives with differing HPA statuses.

Removed, alternating appearance.
Reduced size of message.
Placed after temperature, before drive model/serial in display.
---
 src/device.c |  1 +
 src/gui.c    | 64 ++++++++++++++++++++--------------------------------
 2 files changed, 25 insertions(+), 40 deletions(-)

diff --git a/src/device.c b/src/device.c
index ed1ead9f..121b879e 100644
--- a/src/device.c
+++ b/src/device.c
@@ -291,6 +291,7 @@ int check_device( nwipe_context_t*** c, PedDevice* dev, int dcount )
     }
 
     /* Initialise the variables that toggle the [size][temp c] with [HPA status]
+     * Not currently used, but may be used in the future or for other purposes
      */
     next_device->HPA_toggle_time = time( NULL );
     next_device->HPA_display_toggle_state = 0;
diff --git a/src/gui.c b/src/gui.c
index 311e80bd..67d1b18e 100644
--- a/src/gui.c
+++ b/src/gui.c
@@ -832,51 +832,35 @@ void nwipe_gui_select( int count, nwipe_context_t** c )
                 /* print the temperature */
                 wprintw_temperature( c[i + offset] );
 
-                /* Toggle the drive/serial with [HDA/DCO Status]
-                 */
-                switch( c[i + offset]->HPA_display_toggle_state )
+                switch( c[i + offset]->HPA_status )
                 {
-                    case 0:
-                        /* print the drive model and serial number */
-                        wprintw( main_window, " %s/%s", c[i + offset]->device_model, c[i + offset]->device_serial_no );
+                    case HPA_ENABLED:
+                        wprintw( main_window, " " );
+                        wattron( main_window, COLOR_PAIR( 9 ) );
+                        wprintw( main_window, "[HS? YES!]" );
+                        wattroff( main_window, COLOR_PAIR( 9 ) );
                         break;
 
-                    case 1:
-                        switch( c[i + offset]->HPA_status )
-                        {
-                            case HPA_ENABLED:
-                                wprintw( main_window, " " );
-                                wattron( main_window, COLOR_PAIR( 9 ) );
-                                wprintw( main_window, " HPA/DCO Hidden sectors detected !!" );
-                                wattroff( main_window, COLOR_PAIR( 9 ) );
-                                break;
-
-                            case HPA_DISABLED:
-                                wprintw( main_window, " " );
-                                wprintw( main_window, "HPA/DCO No hidden sectors detected " );
-                                break;
-
-                            case HPA_UNKNOWN:
-                                wprintw( main_window, " " );
-                                wattron( main_window, COLOR_PAIR( 9 ) );
-                                wprintw( main_window, " HPA/DCO hidden sectors indeterminate " );
-                                wattroff( main_window, COLOR_PAIR( 9 ) );
-                                break;
-
-                            case HPA_NOT_APPLICABLE:
-                                /* print the drive model and serial number */
-                                wprintw( main_window,
-                                         " %s/%s",
-                                         c[i + offset]->device_model,
-                                         c[i + offset]->device_serial_no );
-                                break;
-
-                            default:
-
-                                break;
-                        }
+                    case HPA_DISABLED:
+                        wprintw( main_window, " " );
+                        wprintw( main_window, "[HS? NO ]" );
+                        break;
+
+                    case HPA_UNKNOWN:
+                        wprintw( main_window, " " );
+                        wprintw( main_window, "[HS? ???]" );
+                        break;
+
+                    default:
+
+                        wprintw( main_window, " " );
+                        wprintw( main_window, "[HS? N/A]" );
+                        break;
                 }
 
+                /* print the drive model and serial number */
+                wprintw( main_window, " %s/%s", c[i + offset]->device_model, c[i + offset]->device_serial_no );
+
                 if( c[i + offset]->HPA_toggle_time + 1 < time( NULL ) )
                 {
                     switch( c[i + offset]->HPA_display_toggle_state )