Hi Kristen,
> src/stkutil.c | 167
> +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> src/stkutil.h | 32 +++++++++++
> 2 files changed, 199 insertions(+), 0 deletions(-)
>
> diff --git a/src/stkutil.c b/src/stkutil.c
> index 6f072e7..73449e2 100644
> --- a/src/stkutil.c
> +++ b/src/stkutil.c
> @@ -26,6 +26,7 @@
> #include <string.h>
> #include <stdlib.h>
> #include <stdint.h>
> +#include <stdio.h>
>
> #include <glib.h>
>
> @@ -5819,3 +5820,169 @@ const unsigned char *stk_pdu_from_envelope(const
> struct stk_envelope *envelope,
>
> return pdu;
> }
> +
> +static const char *html_colors[] = {
> + "#000000", /* Black */
> + "#808080", /* Dark Grey */
> + "#C11B17", /* Dark Red */
> + "#FBB117", /* Dark Yellow */
> + "#347235", /* Dark Green */
> + "#307D7E", /* Dark Cyan */
> + "#0000A0", /* Dark Blue */
> + "#C031C7", /* Dark Magenta */
> + "#C0C0C0", /* Grey */
> + "#FFFFFF", /* White */
> + "#FF0000", /* Bright Red */
> + "#FFFF00", /* Bright Yellow */
> + "#00FF00", /* Bright Green */
> + "#00FFFF", /* Bright Cyan */
> + "#0000FF", /* Bright Blue */
> + "#FF00FF", /* Bright Magenta */
> +};
> +
> +static void end_format(GString *string, guint8 code)
> +{
> + g_string_append(string, "</span>");
> +
> + if ((code & STK_TEXT_FORMAT_ALIGN_MASK) != STK_TEXT_FORMAT_NO_ALIGN)
> + g_string_append(string, "</div>");
> +}
> +
> +static void start_format(GString *string, guint8 code, guint8 color)
> +{
> + guint8 align = code & STK_TEXT_FORMAT_ALIGN_MASK;
> + guint8 font = code & STK_TEXT_FORMAT_FONT_MASK;
> + guint8 style = code & STK_TEXT_FORMAT_STYLE_MASK;
> + int fg = color & 0x0f;
> + int bg = (color >> 4) & 0x0f;
> +
> + /* align formatting applies to a block of test */
> + if (align != STK_TEXT_FORMAT_NO_ALIGN)
> + g_string_append(string, "<div style=\"");
> +
> + if (align == STK_TEXT_FORMAT_RIGHT_ALIGN)
> + g_string_append(string, "text-align: right;\">");
> + else if (align == STK_TEXT_FORMAT_CENTER_ALIGN)
> + g_string_append(string, "text-align: center;\">");
> + else if (align == STK_TEXT_FORMAT_LEFT_ALIGN)
> + g_string_append(string, "text-align: left;\">");
using a switch {} statement seems more appropriate here.
> +
> + /* font, style, and color are inline */
> + g_string_append(string, "<span style=\"");
> +
> + if (font == STK_TEXT_FORMAT_FONT_SIZE_LARGE)
> + g_string_append(string, "font-size: big;");
> + else if (font == STK_TEXT_FORMAT_FONT_SIZE_SMALL)
> + g_string_append(string, "font-size: small;");
Same here. I prefer switch {} since it is more readable.
> +
> + if (style == STK_TEXT_FORMAT_STYLE_BOLD)
> + g_string_append(string, "font-weight: bold;");
> + else if (style == STK_TEXT_FORMAT_STYLE_ITALIC)
> + g_string_append(string, "font-style: italic;");
> + else if (style == STK_TEXT_FORMAT_STYLE_UNDERLINED)
> + g_string_append(string, "text-decoration: underline;");
> + else if (style == STK_TEXT_FORMAT_STYLE_STRIKETHROUGH)
> + g_string_append(string, "text-decoration: line-through;");
And also here please.
> +
> + /* add any color */
> + if (fg)
> + g_string_append_printf(string, "color: %s;", html_colors[fg]);
> + if (bg)
> + g_string_append_printf(string, "background-color: %s;",
> + html_colors[bg]);
> + g_string_append(string, "\">");
> +}
> +
> +char *stk_text_to_html(char *text, int text_len,
> + const unsigned char *attrs, int attrs_len)
> +{
> + GString *string = g_string_sized_new(text_len + 1);
> + gint formats[257]; /* maximum number of chars in text + 1 */
> + int pos = 0, attr, i, j, prev_attr;
> + guint8 start, end, code, color, len, align;
> +
> + /* we will need formatting at the position beyond the last char */
> + for (i = 0; i <= text_len; i++)
> + formats[i] = STK_TEXT_FORMAT_INIT;
> +
> + i = 0;
> +
> + while (i < attrs_len) {
> + start = attrs[i++];
> + len = attrs[i++];
> + code = attrs[i++];
> +
> + if (i < attrs_len)
> + color = attrs[i++];
> + else
> + color = 0;
> +
> + if (len == 0)
> + end = text_len;
> + else
> + end = start + len;
> +
> + /* sanity check values */
> + if (start > end || end > text_len)
> + continue;
> +
> + /*
> + * if the alignment is the same as either the default
> + * or the last alignment used, don't set any alignment
> + * value.
> + */
> + if (start == 0)
> + align = STK_DEFAULT_TEXT_ALIGNMENT;
> + else {
> + align = get_align(formats[start - 1]);
> + if (align == STK_TEXT_FORMAT_NO_ALIGN)
> + align = STK_DEFAULT_TEXT_ALIGNMENT;
> + }
> +
> + if ((code & STK_TEXT_FORMAT_ALIGN_MASK) == align)
> + code |= STK_TEXT_FORMAT_NO_ALIGN;
> +
> + attr = make_attr(start, code, color);
> +
> + for (j = start; j < end; j++)
> + formats[j] = attr;
> + }
> +
> + prev_attr = STK_TEXT_FORMAT_INIT;
> +
> + while (pos <= text_len) {
> + attr = formats[pos];
> + if (attr != prev_attr) {
> + if (prev_attr != STK_TEXT_FORMAT_INIT)
> + end_format(string, get_code(attr));
> +
> + if (attr != STK_TEXT_FORMAT_INIT)
> + start_format(string, get_code(attr),
> + get_color(attr));
> +
> + prev_attr = attr;
> + }
> +
> + if (pos == text_len)
> + break;
> +
> + if (text[pos] == '\n')
> + g_string_append(string, "<br/>");
> + else if (text[pos] == '\r') {
> + g_string_append(string, "<br/>");
> + if ((pos + 1 < text_len) && (text[pos + 1] == '\n'))
> + pos++;
> + } else if (text[pos] == '<')
> + g_string_append(string, "<");
> + else if (text[pos] == '>')
> + g_string_append(string, ">");
> + else if (text[pos] == '&')
> + g_string_append(string, "&");
> + else
> + g_string_append_c(string, text[pos]);
Also this one is better done as a switch {} statement. It becomes way
more easier to read that way.
> + pos++;
> + }
> +
> + /* return characters from string. Caller must free char data */
> + return g_string_free(string, FALSE);
> +}
> diff --git a/src/stkutil.h b/src/stkutil.h
> index ca4817e..fbdbc21 100644
> --- a/src/stkutil.h
> +++ b/src/stkutil.h
> @@ -1635,6 +1635,36 @@ struct stk_envelope {
> };
> };
>
> +#define STK_TEXT_FORMAT_ALIGN_MASK 0x03
> +#define STK_TEXT_FORMAT_FONT_MASK 0x0C
> +#define STK_TEXT_FORMAT_STYLE_MASK 0xF0
> +#define STK_DEFAULT_TEXT_ALIGNMENT 0x00
> +#define STK_TEXT_FORMAT_INIT -1
> +
> +#define get_code(attr) \
> + ((attr >> 16) & 0xFF)
> +#define get_align(attr) \
> + ((attr >> 16) & STK_TEXT_FORMAT_ALIGN_MASK)
> +#define get_color(attr) \
> + ((attr >> 24) & 0xFF)
> +#define make_attr(start, code, color) \
> + (start | (code << 16) | (color << 24))
Please put these defines inside the C file right before the function
where you actually use them. Not namespaced defines in a header file are
a bad idea.
Also do we have that many users of these defines that it makes sense to
create defines for it. They all lock pretty simple anyway. And in case
of 1 or 2 users, then sometimes it is better to just put a comment there
and repeat the code. I haven't actually counted.
Regards
Marcel
_______________________________________________
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono
