summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndreas Gruenbacher <agruen@suse.de>2010-09-15 11:38:21 +0200
committerAndreas Gruenbacher <agruen@suse.de>2010-09-17 01:57:22 +0200
commit93129f45a1fc1f824b652f50898fb57440449f73 (patch)
treee148ddc78738247a3380a3366067c54c878b278c
parent76649aba9e1a6f191684480994114a57a5ddee0c (diff)
downloadpatch-93129f45a1fc1f824b652f50898fb57440449f73.tar.gz
Replace unnecessary local variable
* src/patch.c (abort_hunk_unified): Replace unnecessary local variable.
-rw-r--r--ChangeLog2
-rw-r--r--src/patch.c25
2 files changed, 14 insertions, 13 deletions
diff --git a/ChangeLog b/ChangeLog
index 6221c53..a03861d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -15,6 +15,8 @@
* src/pch.c (open_patch_file): Compute TMPPATNAME with make_tempfile()
if needed instead of precomputing it in src/patch.c (main).
+ * src/patch.c (abort_hunk_unified): Replace unnecessary local variable.
+
2010-07-27 Tim Waugh <twaugh@redhat.com>
* src/patch.c: Stops "patch --get 1" from segfaulting.
diff --git a/src/patch.c b/src/patch.c
index 189a5a7..05890d3 100644
--- a/src/patch.c
+++ b/src/patch.c
@@ -1116,7 +1116,6 @@ print_header_line (FILE *fp, const char *tag, bool reverse)
static void
abort_hunk_unified (bool header, bool reverse)
{
- FILE *fp = rejfp;
lin old = 1;
lin lastline = pch_ptrn_lines ();
lin new = lastline + 1;
@@ -1124,17 +1123,17 @@ abort_hunk_unified (bool header, bool reverse)
if (header)
{
if (pch_name (INDEX))
- fprintf(fp, "Index: %s\n", pch_name (INDEX));
+ fprintf(rejfp, "Index: %s\n", pch_name (INDEX));
print_header_line (rejfp, "---", reverse);
print_header_line (rejfp, "+++", ! reverse);
}
/* Add out_offset to guess the same as the previous successful hunk. */
- fprintf (fp, "@@ -");
- print_unidiff_range (fp, pch_first () + out_offset, lastline);
- fprintf (fp, " +");
- print_unidiff_range (fp, pch_newfirst () + out_offset, pch_repl_lines ());
- fprintf (fp, " @@\n");
+ fprintf (rejfp, "@@ -");
+ print_unidiff_range (rejfp, pch_first () + out_offset, lastline);
+ fprintf (rejfp, " +");
+ print_unidiff_range (rejfp, pch_newfirst () + out_offset, pch_repl_lines ());
+ fprintf (rejfp, " @@\n");
while (pch_char (new) == '=' || pch_char (new) == '\n')
new++;
@@ -1146,13 +1145,13 @@ abort_hunk_unified (bool header, bool reverse)
{
for (; pch_char (old) == '-'; old++)
{
- fputc ('-', fp);
- pch_write_line (old, fp);
+ fputc ('-', rejfp);
+ pch_write_line (old, rejfp);
}
for (; pch_char (new) == '+'; new++)
{
- fputc ('+', fp);
- pch_write_line (new, fp);
+ fputc ('+', rejfp);
+ pch_write_line (new, rejfp);
}
if (old > lastline)
@@ -1161,8 +1160,8 @@ abort_hunk_unified (bool header, bool reverse)
if (pch_char (new) != pch_char (old))
mangled_patch (old, new);
- fputc (' ', fp);
- pch_write_line (old, fp);
+ fputc (' ', rejfp);
+ pch_write_line (old, rejfp);
}
if (pch_char (new) != '^')
mangled_patch (old, new);