X-Git-Url: https://www.fleuret.org/cgi-bin/gitweb/gitweb.cgi?a=blobdiff_plain;f=selector.cc;h=7b51ce30a19de22ce0baa6f2f63ceb38695e85da;hb=043d4e30275d5a1eb42a69d491ab009d01a76685;hp=637ae7432910875c85a93d3feffb1b9f30a77fc0;hpb=4bbb5e96dd0a902c520f811e9b4e11ce2c000f01;p=selector.git

diff --git a/selector.cc b/selector.cc
index 637ae74..7b51ce3 100644
--- a/selector.cc
+++ b/selector.cc
@@ -138,6 +138,7 @@ int *new_hash_table(int hash_table_size) {
 
 int test_and_add(char *new_string, int new_index,
                  char **strings, int *hash_table, int hash_table_size) {
+
   unsigned int code = 0;
 
   // This is my recipe. I checked, it seems to work (as long as
@@ -570,36 +571,46 @@ void read_file(const char *input_filename,
       char *s, *t;
       const char *u;
 
-      s = buffer;
       t = raw_line;
-      while(*t) {
-        u = unctrl(*t++);
-        while(*u) { *s++ = *u++; }
+
+      // Remove the zsh history prefix
+
+      if(zsh_history && *t == ':') {
+        while(*t && *t != ';') s++;
+        if(*t == ';') t++;
       }
-      *s = '\0';
 
-      s = buffer;
+      // Remove the bash history prefix
 
-      if(zsh_history && *s == ':') {
-        while(*s && *s != ';') s++;
-        if(*s == ';') s++;
+      if(bash_history && (*t == ' ' || (*t >= '0' && *t <= '9'))) {
+        while(*t == ' ' || (*t >= '0' && *t <= '9')) t++;
       }
 
-      if(bash_history && (*s == ' ' || (*s >= '0' && *s <= '9'))) {
-        while(*s == ' ' || (*s >= '0' && *s <= '9')) s++;
+      // Copy the string while transforming the ctrl characters into
+      // printable characters
+
+      s = buffer;
+
+      while(*t) {
+        u = unctrl(*t++);
+        while(*u) { *s++ = *u++; }
       }
+      *s = '\0';
+
+      // Check for duplicates with the hash table and insert the line
+      // in the list if necessary
 
       int dup;
 
       if(hash_table) {
-        dup = test_and_add(s, *nb_lines, lines, hash_table, hash_table_size);
+        dup = test_and_add(buffer, *nb_lines, lines, hash_table, hash_table_size);
       } else {
         dup = -1;
       }
 
       if(dup < 0) {
-        lines[*nb_lines] = new char[strlen(s) + 1];
-        strcpy(lines[*nb_lines], s);
+        lines[*nb_lines] = new char[strlen(buffer) + 1];
+        strcpy(lines[*nb_lines], buffer);
       } else {
         // The string was already in there, so we do not allocate a
         // new string but use the pointer to the first occurence of it