<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On 2 December 2013 09:45, Matthew Brush <span dir="ltr"><<a href="mailto:mbrush@codebrainz.ca" target="_blank">mbrush@codebrainz.ca</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 13-12-01 01:26 PM, Lex Trotman wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
**gint** utils_strtod()??  Sheesh.<br>
<br>
s/b renamed utils_hexcolor() or similar, thats what it does.<br>
<br>
As for "end", the original strtod() and friends used it to extract several<br>
whitespace separated strings from the one string by just passing "end" to<br>
the next call, but if we don't use it and its wrong anyways we might as<br>
well drop it when we change the name to utils_hexcolor().<br>
<br>
</blockquote>
<br></div>
Agree the name is bad, and also the colour parsing isn't very robust. I meant to improve it when adding the named_colors support but never got around to it.<br>
<br>
Attached patch uses GDK colour parsing instead of own-rolled thing which supports more colour specification formats as well as cleans up some related code while remaining compatible with legacy colour notation and GTK2 and GTK3 builds.<br>

<br>
If nobody objects, I can commit the patch.<br></blockquote><div><br></div><div>Untested, but looks ok, except it has changes not directly related to parsing colour like highlighting.c:698, s/b separate commit IMHO.</div>
<div><br></div><div>Cheers</div><div>Lex</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
Matthew Brush<br>
<br>
<br>_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@lists.geany.org">Devel@lists.geany.org</a><br>
<a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-bin/mailman/listinfo/devel</a><br>
<br></blockquote></div><br></div></div>