http_get: comment about snprintf() being a hot spot

cmogstored is pretty fast, but it could be faster.

1 files changed, 8 insertions, 1 deletions

diff --git a/http_get.c b/http_get.c
index 2204508..a104b47 100644
--- a/http_get.c
+++ b/http_get.c
@@ -36,7 +36,14 @@ static ssize_t linux_sendfile(int sockfd, int filefd, off_t *off, size_t count)
 
 #define ERR416 "416 Requested Range Not Satisfiable"
 
-/* TODO: refactor this */
+/*
+ * TODO: refactor this
+ *
+ * snprintf() usage here is a hot spot in profiling.  Perhaps one day,
+ * link-time optimization will be able to work on *printf() functions
+ * so we won't hurt code maintainability by optimizing away snprintf()
+ * ourselves.  This function is ugly enough already
+ */
 static off_t http_get_resp_hdr(struct mog_http *http, struct stat *sb)
 {
         char *modified;