[patch #6735] Fexecute: Remove the last iteration fo for(), over empty string

[Stepan Kasal]

URL:
  <http://savannah.gnu.org/patch/?6735>

                 Summary: Fexecute: Remove the last iteration fo for(), over
empty string
                 Project: grep
            Submitted by: kasal
            Submitted on: Thu 29 Jan 2009 03:57:07 PM CET
                Category: None
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any

    _______________________________________________________

Details:

In Fexecute, kwsexec is called in a for loop, where the start position
iterates from the beginning to the end.
In the last iteration, the start position is the end of the string, so
kwsexec is called on an empty string. I do not see any reason why this is done
and I think that fgrep would still work correctly if this last iteration were
removed.

AFAIK, the current code also works correctly, but I believe the patch make
the code more elegant and more understandable.

This is what the patch does.  OK to commit?

More info:
This patch was needed in Fedora's patched version of grep to fix the
following bug:
https://bugzilla.redhat.com/show_bug.cgi?id=479152
The bug does not reproduce with vanilla GNU grep, but IMHO it would be a good
idea to commit this patch anyway.



    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Thu 29 Jan 2009 03:57:08 PM CET  Name: grep-Fexecute-for.patch  Size:
624B   By: kasal

<http://savannah.gnu.org/patch/download.php?file_id=17350>

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?6735>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/