micro-optimization: fputc (function) -> putc (macro/inline function)

[stagit.git] / src / stagit-index.c
diff --git a/src/stagit-index.c b/src/stagit-index.c

index fa0c21a86cfc1c6a57d849f6087556ead89e0bc9..5454edb0a2fbae91e7cb0aa9944b2c185e5bd2ec 100644 (file)
--- a/src/stagit-index.c
+++ b/src/stagit-index.c
@@ -43,7 +43,7 @@ xmlencode(FILE *fp, const char *s, size_t len)
                 case '\'': fputs("&#39;" , fp); break;
                 case '&':  fputs("&amp;",  fp); break;
                 case '"':  fputs("&quot;", fp); break;
-               default:   fputc(*s, fp);
+               default:   putc(*s, fp);
                 }
         }
  }
@@ -63,36 +63,42 @@ printtimeshort(FILE *fp, const git_time *intime)
  }
  
  void
-writeheader(FILE *fp)
+writeheader(char *path)
  {
+       FILE *fp = fopen(path, "w");
         fputs("<!DOCTYPE html>\n"
-               "<html>\n<head>\n"
+               "<html lang=\"en\">\n<head>\n"
                 "<meta http-equiv=\"Content-Type\" content=\"text/html; charset=UTF-8\" />\n"
+               "<meta name=\"viewport\" content=\"width=device-width, initial-scale=1.0\" />\n"
                 "<title>", fp);
         xmlencode(fp, description, strlen(description));
         fprintf(fp, "</title>\n<link rel=\"icon\" type=\"image/png\" href=\"%sfavicon.png\" />\n", relpath);
-       fprintf(fp, "<link rel=\"stylesheet\" type=\"text/css\" href=\"%sstyle.min.css\" />\n", relpath);
+       fprintf(fp, "<link rel=\"stylesheet\" type=\"text/css\" href=\"%sstyle.css\" />\n", relpath);
         fputs("</head>\n<body>\n", fp);
         fprintf(fp, "<table>\n<tr><td><img src=\"%slogo.png\" alt=\"\" width=\"32\" height=\"32\" /></td>\n"
-               "<td><span class=\"desc\">", relpath);
+               "<td><h1>", relpath);
         xmlencode(fp, description, strlen(description));
-       fputs("</span></td></tr><tr><td></td><td>\n"
+       fputs("</h1></td></tr><tr><td></td><td>\n"
                 "</td></tr>\n</table>\n<hr/>\n<div id=\"content\">\n"
                 "<table id=\"index\"><thead>\n"
                 "<tr><td><b>Name</b></td><td><b>Description</b></td><td><b>Owner</b></td>"
                 "<td><b>Last commit</b></td></tr>"
                 "</thead><tbody>\n", fp);
+       fclose(fp);
  }
  
  void
-writefooter(FILE *fp)
+writefooter(char *path)
  {
+       FILE *fp = fopen(path, "a");
         fputs("</tbody>\n</table>\n</div>\n</body>\n</html>\n", fp);
+       fclose(fp);
  }
  
  int
-writelog(FILE *fp)
+writelog(char *path)
  {
+       FILE *fp = fopen(path, "a");
         git_commit *commit = NULL;
         const git_signature *author;
         git_revwalk *w = NULL;
@@ -136,6 +142,7 @@ writelog(FILE *fp)
  err:
         git_revwalk_free(w);
         free(stripped_name);
+       fclose(fp);
  
         return ret;
  }
@@ -164,7 +171,7 @@ main(int argc, char *argv[])
                 err(1, "pledge");
  #endif
  
-       writeheader(stdout);
+       writeheader("index.html");
  
         for (i = 1; i < argc; i++) {
                 repodir = argv[i];
@@ -210,14 +217,14 @@ main(int argc, char *argv[])
                         owner[strcspn(owner, "\n")] = '\0';
                         fclose(fp);
                 }
-               writelog(stdout);
+               writelog("index.html");
         }
-       writefooter(stdout);
+       writefooter("index.html");
  
         /* copy css */
         char cwd[PATH_MAX];
         strcpy(cwd, getcwd(cwd, sizeof(cwd)));
-       cp("/usr/local/share/stagit/style.min.css", strcat(cwd, "/style.min.css"));
+       cp("/usr/local/share/stagit/style.css", strcat(cwd, "/style.css"));
  
         /* cleanup */
         git_repository_free(repo);