Multiple bug fixes for use with less and more

ios_error.h

@@ -31,6 +31,7 @@ extern "C" {
 // these functions are defined differently in C++. The #define approach breaks things.
 #ifndef __cplusplus
   #define getwchar() fgetwc(thread_stdin)
+  #define putwchar(a) fputwc(a, thread_stdout)
   // iswprint depends on the given locale, and setlocale() fails on iOS:
   #define iswprint(a) 1
   #define write ios_write

ios_system.m

@@ -185,11 +185,24 @@ static void cleanup_function(void* parameters) {
     // This function is called when pthread_exit() or ios_kill() is called
     functionParameters *p = (functionParameters *) parameters;
     char* commandName = p->argv[0];
-    if ((strcmp(commandName, "less") == 0) || (strcmp(commandName, "more") == 0)) {
-        if ((currentSession->current_command_root_thread != 0) && (currentSession->current_command_root_thread != pthread_self())) {
-            // Command was "root_command | sthg | less". We need to kill root command:
-            pthread_kill(currentSession->current_command_root_thread, SIGINT);
-            while (fgetc(thread_stdin) != EOF) { } // flush input, otherwise previous command gets blocked.
+    // commandName can be NULL, I don't know exactly under which circumstances.
+    if (commandName != NULL) {
+        if ((strcmp(commandName, "less") == 0) || (strcmp(commandName, "more") == 0)) {
+            if ((currentSession->current_command_root_thread != 0) && (currentSession->current_command_root_thread != pthread_self())) {
+                // Something started less. Was that a pipe?
+                NSLog(@"less quitting. currentsession->stdin: %d thread_stdin: %d\n", fileno(currentSession->stdin), fileno(thread_stdin));
+                if (fileno(currentSession->stdin) != fileno(thread_stdin)) {
+                    // Command was "root_command | sthg | less". We need to kill the root command:
+                    pthread_kill(currentSession->current_command_root_thread, SIGINT);
+                    // ... and keep listening for input, otherwise the command piping into us will get blocked:
+                    char c;
+                    // Stop when rootthread is 0? will it work?
+                    // fflush(thread_stdin)?
+                    while (((c = fgetc(thread_stdin)) != EOF) && (currentSession->current_command_root_thread != 0)) {
+                        // NSLog(@"Root thread: %x received: %c\n", currentSession->current_command_root_thread, c);
+                    }
+                }
+            }
         }
     }
     if ((!joinMainThread) && p->isPipeOut) {
@@ -207,16 +220,18 @@ static void cleanup_function(void* parameters) {
     fflush(thread_stdout);
     fflush(thread_stderr);
     // release parameters:
-    NSLog(@"Terminating command: %s thread_id %x stdin %d stdout %d stderr %d isPipeOut %d", commandName, pthread_self(), fileno(p->stdin), fileno(p->stdout), fileno(p->stderr), p->isPipeOut);
-    // Specific to run multiple python3 interpreters:
-    if ((strncmp(commandName, "python", 6) == 0) && (strlen(commandName) == strlen("python") + 1)) {
-        // It's one of the multiple python3 interpreters
-        char commandNumber = commandName[6];
-        if (commandNumber == '3') PythonIsRunning[0] = false;
-        else {
-            commandNumber -= 'A' - 1;
-            if ((commandNumber > 0) && (commandNumber < MaxPythonInterpreters))
-                PythonIsRunning[commandNumber] = false;
+    if (commandName != NULL) {
+        // NSLog(@"Terminating command: %s thread_id %x stdin %d stdout %d stderr %d isPipeOut %d", commandName, pthread_self(), fileno(p->stdin), fileno(p->stdout), fileno(p->stderr), p->isPipeOut);
+        // Specific to run multiple python3 interpreters:
+        if ((strncmp(commandName, "python", 6) == 0) && (strlen(commandName) == strlen("python") + 1)) {
+            // It's one of the multiple python3 interpreters
+            char commandNumber = commandName[6];
+            if (commandNumber == '3') PythonIsRunning[0] = false;
+            else {
+                commandNumber -= 'A' - 1;
+                if ((commandNumber > 0) && (commandNumber < MaxPythonInterpreters))
+                    PythonIsRunning[commandNumber] = false;
+            }
         }
     }
     bool isSh = strcmp(p->argv[0], "sh") == 0;
@@ -278,7 +293,7 @@ void crash_handler(int sig) {
     functionParameters *p = (functionParameters *) parameters;
     ios_storeThreadId(pthread_self());
     // NSLog(@"Storing thread_id: %x isPipeOut: %x isPipeErr: %x stdin %d stdout %d stderr %d command= %s\n", pthread_self(), p->isPipeOut, p->isPipeErr, fileno(p->stdin), fileno(p->stdout), fileno(p->stderr), p->argv[0]);
-    NSLog(@"Starting command: %s thread_id %x", p->argv[0], pthread_self());
+    // NSLog(@"Starting command: %s thread_id %x", p->argv[0], pthread_self());
     // re-initialize for getopt:
     // TODO: move to __thread variable for optind too
     optind = 1;
@@ -353,7 +368,7 @@ void initializeEnvironment() {
     setenv("TMPDIR", NSTemporaryDirectory().UTF8String, 0); // tmp directory
     setenv("CLICOLOR", "1", 1);
     setenv("LSCOLORS", "ExFxBxDxCxegedabagacad", 0); // colors for ls on black background
-    
+
     // We can't write in $HOME so we need to set the position of config files:
     setenv("SSH_HOME", docsPath.UTF8String, 0);  // SSH keys in ~/Documents/.ssh/ or [Cloud Drive]/.ssh
     setenv("DIG_HOME", docsPath.UTF8String, 0);  // .digrc is in ~/Documents/.digrc or [Cloud Drive]/.digrc
@@ -1954,7 +1969,7 @@ int ios_system(const char* inputCmd) {
                     currentSession->isMainThread = true;
                 }
             } else {
-                NSLog(@"Starting command %s, global_errno= %d\n", command, currentSession->global_errno);
+                // NSLog(@"Starting command %s, global_errno= %d\n", command, currentSession->global_errno);
                 // Don't send signal if not in main thread. Also, don't join threads.
                 volatile pthread_t _tid_local = NULL;
                 pthread_create(&_tid_local, NULL, run_function, params);
@@ -1992,7 +2007,7 @@ int ios_system(const char* inputCmd) {
         free(dontExpand);
         free(params);
     }
-    NSLog(@"returning from ios_system, global_errno= %d\n", currentSession->global_errno);
+    // NSLog(@"returning from ios_system, global_errno= %d\n", currentSession->global_errno);
     free(originalCommand); // releases cmd, which was a strdup of inputCommand
     fflush(thread_stdin);
     fflush(thread_stdout);