fixme/todo cleanup

not much worth doing right now.

proxy_network.c

@@ -336,7 +336,6 @@ void proxy_run_backend_queue(be_head_t *head) {
         if (bec->connecting || bec->validating) {
             P_DEBUG("%s: deferring IO pending connecting (%s:%s)\n", __func__, be->name, be->port);
         } else {
-            // FIXME: doesn't work with tls disabled.
             if (!bec->ssl) {
                 flags = _flush_pending_write(bec);
             } else {

proxy_tls.c

@@ -21,7 +21,6 @@
  *   SSL_connect()) for defense against bugs in our code or OpenSSL.
  */
 
-// TODO: int -> enum?
 int mcp_tls_init(proxy_ctx_t *ctx) {
     if (ctx->tls_ctx) {
         return MCP_TLS_OK;
@@ -205,6 +204,10 @@ int mcp_tls_writev(struct mcp_backendconn_s *be, int iovcnt) {
     proxy_event_thread_t *et = be->event_thread;
     // TODO: move this to event thread init to remove branch and move error
     // handling to startup time.
+    // Actually we won't know if TLS is in use until a backend shows up and
+    // tries to write... so I'm not sure where to move this. TLS compiled in
+    // but not used would waste memory.
+    // Maybe can at least mark it unlikely()?
     if (et->tls_wbuf_size == 0) {
         et->tls_wbuf_size = TLS_WBUF_SIZE;
         et->tls_wbuf = malloc(et->tls_wbuf_size);