* find.c (search_content): Redo recent change without infinite loop ;-),

egrep_path hardcoded to "egrep" as it is done in locate_egrep ()

(locate_egrep): commented out

(rotating_dash) [!HAVE_X]: moved into do_search to prevent warning

src/ChangeLog

@@ -1,3 +1,12 @@
+2001-04-27  Andrew V. Samoilov  <sav@bcs.zp.ua>
+
+	* find.c (search_content): Redo recent change without infinite loop ;-),
+	egrep_path hardcoded to "egrep" as it is done in locate_egrep ()  
+
+	(locate_egrep): commented out
+
+	(rotating_dash) [!HAVE_X]: moved into do_search to prevent warning
+
 2001-04-26  Pavel Roskin  <proski@gnu.org>
 
 	* find.c (search_content): Undo a recent change that caused an
@@ -8,7 +17,7 @@
 	* view.c (get_line_at): previous newline is right condition 
 	for regexp '^' pattern also, fixed
 
-	(search) [HAVE_GNOME]: gnome_message_box_new don't expand printf
+	(search) [HAVE_GNOME]: gnome_message_box_new doesn't expand printf
 	pattern, so g_strdup_printf used
 
 	* find.c (find_parameters): don't use stat structure if mc_stat failed

src/find.c

@@ -111,9 +111,6 @@ static WLabel *status_label;	/* Finished, Searching etc. */
 static WListbox *find_list;	/* Listbox with the file list */
 #endif
 
-/* For nice updating */
-static char *rotating_dash = "|/-\\";
-
 /* This keeps track of the directory stack */
 typedef struct dir_stack {
     char *name;
@@ -388,12 +385,12 @@ find_add_match (Dlg_head *h, char *dir, char *file)
 #endif	    
 }
 
+#if 0
 static char *
 locate_egrep (void)
 {
 /* commented out because mc runs egrep via execvp() which searches PATH
     itself. It is likely that egrep in PATH is better than in /bin */
-#if 0
     char *paths [] = {
 	"/bin/egrep",
 	"/usr/bin/egrep",
@@ -408,9 +405,9 @@ locate_egrep (void)
 	if (stat (*p, &s) == 0)
 	    return *p;
     }
-#endif
     return "egrep";
 }
+#endif
 
 /* 
  * search_content:
@@ -428,7 +425,7 @@ search_content (Dlg_head *h, char *directory, char *filename)
     char c;
     int i;
     pid_t pid;
-    static char *egrep_path;
+    char *egrep_path = "egrep";
     char *egrep_opts = case_sensitive ? "-n" : "-in";
 
     fname = concat_dir_and_file (directory, filename);
@@ -444,9 +441,6 @@ search_content (Dlg_head *h, char *directory, char *filename)
     if (file_fd == -1)
 	return;
 
-    if (!egrep_path)
-	egrep_path = locate_egrep ();
-
 #ifndef GREP_STDIN
     pipe = mc_doublepopen (file_fd, -1, &pid, egrep_path, egrep_path, egrep_opts, content_pattern, NULL);
 #else /* GREP_STDIN */
@@ -468,10 +462,7 @@ search_content (Dlg_head *h, char *directory, char *filename)
     enable_interrupt_key ();
     got_interrupt ();
 
-    while (1){
+    while ((i = read (pipe, &c, 1)) == 1){
-	i = read (pipe, &c, 1);
-	if (i == 0)
-	    break;
 
 	if (c == '\n'){
 	    p = buffer;
@@ -564,7 +555,7 @@ do_search (struct Dlg_head *h)
 	    } 
 	    
 	    strcpy (directory, tmp);
-	   g_free (tmp);
+	    g_free (tmp);
 
 	    if (verbose){
 		    char buffer [BUF_SMALL];
@@ -614,9 +605,12 @@ do_search (struct Dlg_head *h)
 
     /* Displays the nice dot */
     count++;
-    if (!(count & 31)){
-	if (verbose){
 #ifndef HAVE_X
+    if (!(count & 31)){
+	/* For nice updating */
+	char *rotating_dash = "|/-\\";
+
+	if (verbose){
 	    pos = (pos + 1) % 4;
 	    attrset (NORMALC);
 	    dlg_move (h, FIND2_Y-6, FIND2_X - 4);
@@ -625,9 +619,6 @@ do_search (struct Dlg_head *h)
 	}
     } else
 	goto do_search_begin;
-#else
-        }
-    }
 #endif
     x_flush_events ();
     return 1;
@@ -895,7 +886,7 @@ get_list_info (char **file, char **dir)
 	gtk_clist_get_text (GTK_CLIST (g_clist), current_row, 0, file);
 	*dir = gtk_clist_get_row_data (GTK_CLIST (g_clist), current_row);
 }
-#else
+#else	/* HAVE_GNOME */
 
 static void
 get_list_info (char **file, char **dir)
@@ -1091,7 +1082,7 @@ kill_gui (void)
     set_idle_proc (find_dlg, 0);
     destroy_dlg (find_dlg);
 }
-#endif
+#endif	/* HAVE_GNOME */
 
 static int
 find_file (char *start_dir, char *pattern, char *content, char **dirname,  char **filename)