Changeset 3184 for libcaca/trunk/caca/codecTweet

Timestamp:

Nov 1, 2008, 7:03:36 AM (12 years ago)

Author:

bsittler

Message:

output valid XHTML (it's valid HTML3 too); note that this drops the COLS=... attribute on the table, but that had previously been calculated incorrectly anyhow so i doubt it mattered

File:

• libcaca/trunk/caca/codec/export.c (modified) (5 diffs)

libcaca/trunk/caca/codec/export.c

@@ -462 +462 @@
     int maxcols;
 
-    /* The HTML table markup: less than 1000 bytes
-     * A line: 10 chars for "<tr></tr>\n"
-     * A glyph: 40 chars for "<td bgcolor=#xxxxxx><font color=#xxxxxx>"
-     *          up to 36 chars for "<b><i><u><blink></blink></u></i></b>"
-     *          up to 48 chars for "&#xxxxxx;", or "&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;" (\t); far less for pure ASCII
-     *          12 chars for "</font></td>" */
-    *bytes = 1000 + cv->height * (10 + cv->width * (40 + 36 + 48 + 12));
-    cur = data = malloc(*bytes);
-
     /* Table */
     maxcols = 0;
@@ -489 +480 @@
     }
     
-    cur += sprintf(cur, "<table border='0' cols='%d' cellpadding='0' cellspacing='0'>\n",
-                        maxcols);
+    /* The HTML table markup: less than 1000 bytes
+     * A line: 10 chars for "<tr></tr>\n"
+     * A glyph: up to 44 chars for "<td bgcolor=\"#xxxxxx\"><font color=\"#xxxxxx\">"
+     *          up to 45 chars for "<tt><b><i><u><blink></blink></u></i></b></tt>"
+     *          up to 9 chars for "&#xxxxxx;" (far less for pure ASCII)
+     *          12 chars for "</font></td>" */
+    *bytes = 1000 + cv->height * (10 + maxcols * (44 + 45 + 9 + 12));
+    cur = data = malloc(*bytes);
+
+    cur += sprintf(cur, "<table border=\"0\" cellpadding=\"0\" cellspacing=\"0\">\n");
 
     for(y = 0; y < cv->height; y++)
@@ -519 +518 @@
 
             if(caca_attr_to_ansi_bg(lineattr[x]) < 0x10)
-                cur += sprintf(cur, " bgcolor=#%.06lx", (unsigned long int)
+                cur += sprintf(cur, " bgcolor=\"#%.06lx\"", (unsigned long int)
                                _caca_attr_to_rgb24bg(lineattr[x]));
 
             if((len + thistab) > 1)
-                cur += sprintf(cur, " colspan=%d", len + thistab);
+                cur += sprintf(cur, " colspan=\"%d\"", len + thistab);
 
             cur += sprintf(cur, ">");
@@ -530 +529 @@
 
             if(needfont)
-                cur += sprintf(cur, "<font color=#%.06lx>", (unsigned long int)
+                cur += sprintf(cur, "<font color=\"#%.06lx\">", (unsigned long int)
                                _caca_attr_to_rgb24fg(lineattr[x]));
 
+            cur += sprintf(cur, "<tt>");
             if(lineattr[x] & CACA_BOLD)
                 cur += sprintf(cur, "<b>");
@@ -582 +582 @@
             if(lineattr[x] & CACA_BOLD)
                 cur += sprintf(cur, "</b>");
+            cur += sprintf(cur, "</tt>");
 
             if(needfont)