This patch looks good to me and applies cleanly to the trunk.  Here's a
synopsis of the code (before the patch) in question:

static int
my_fgets(char *buf, int len, FILE *fp)
{
    for(;;) {
          /* a bunch of code that does not contain break or continue */
          return -2;
    }
}

The patch removes the extraneous for loop and un-indents the inside of
the loop.  However, the patch changes that section of code to use
4-space indents while the rest of the file uses 8-space indents.