From 1467dec14710b4939a2191fea5f75c872a25f2ed Mon Sep 17 00:00:00 2001
From: Daniel Hwang <danielleehwang@gmail.com>
Date: Mon, 21 Sep 2015 21:06:42 -0700
Subject: [PATCH] tool: remove redundant libcurl check

The easysrc generation is run only when --libcurl is initialized.

Ref: https://github.com/bagder/curl/issues/429

Closes #448
---
 src/tool_easysrc.c | 82 ++++++++++++++++++++++------------------------
 1 file changed, 40 insertions(+), 42 deletions(-)

diff --git a/src/tool_easysrc.c b/src/tool_easysrc.c
index 0482ef6ae..59e471dc5 100644
--- a/src/tool_easysrc.c
+++ b/src/tool_easysrc.c
@@ -172,55 +172,53 @@ void dumpeasysrc(struct GlobalConfig *config)
   struct curl_slist *ptr;
   char *o = config->libcurl;
 
-  if(o) {
-    FILE *out;
-    bool fopened = FALSE;
-    if(strcmp(o, "-")) {
-      out = fopen(o, FOPEN_WRITETEXT);
-      fopened = TRUE;
-    }
-    else
-      out = stdout;
-    if(!out)
-      warnf(config, "Failed to open %s to write libcurl code!\n", o);
-    else {
-      int i;
-      const char *c;
+  FILE *out;
+  bool fopened = FALSE;
+  if(strcmp(o, "-")) {
+    out = fopen(o, FOPEN_WRITETEXT);
+    fopened = TRUE;
+  }
+  else
+    out = stdout;
+  if(!out)
+    warnf(config, "Failed to open %s to write libcurl code!\n", o);
+  else {
+    int i;
+    const char *c;
 
-      for(i=0; ((c = srchead[i]) != NULL); i++)
-        fprintf(out, "%s\n", c);
+    for(i=0; ((c = srchead[i]) != NULL); i++)
+      fprintf(out, "%s\n", c);
 
-      /* Declare variables used for complex setopt values */
-      for(ptr=easysrc_decl; ptr; ptr = ptr->next)
-        fprintf(out, "  %s\n", ptr->data);
-
-      /* Set up complex values for setopt calls */
-      if(easysrc_data) {
-        fprintf(out, "\n");
-
-        for(ptr=easysrc_data; ptr; ptr = ptr->next)
-          fprintf(out, "  %s\n", ptr->data);
-      }
+    /* Declare variables used for complex setopt values */
+    for(ptr=easysrc_decl; ptr; ptr = ptr->next)
+      fprintf(out, "  %s\n", ptr->data);
 
+    /* Set up complex values for setopt calls */
+    if(easysrc_data) {
       fprintf(out, "\n");
-      for(ptr=easysrc_code; ptr; ptr = ptr->next) {
-        if(ptr->data[0]) {
-          fprintf(out, "  %s\n", ptr->data);
-        }
-        else {
-          fprintf(out, "\n");
-        }
-      }
 
-      for(ptr=easysrc_clean; ptr; ptr = ptr->next)
+      for(ptr=easysrc_data; ptr; ptr = ptr->next)
         fprintf(out, "  %s\n", ptr->data);
-
-      for(i=0; ((c = srcend[i]) != NULL); i++)
-        fprintf(out, "%s\n", c);
-
-      if(fopened)
-        fclose(out);
     }
+
+    fprintf(out, "\n");
+    for(ptr=easysrc_code; ptr; ptr = ptr->next) {
+      if(ptr->data[0]) {
+        fprintf(out, "  %s\n", ptr->data);
+      }
+      else {
+        fprintf(out, "\n");
+      }
+    }
+
+    for(ptr=easysrc_clean; ptr; ptr = ptr->next)
+      fprintf(out, "  %s\n", ptr->data);
+
+    for(i=0; ((c = srcend[i]) != NULL); i++)
+      fprintf(out, "%s\n", c);
+
+    if(fopened)
+      fclose(out);
   }
 
   easysrc_free();