awesomewm-mirrors
/
awesome
mirror of https://github.com/awesomeWM/awesome.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

[draw] Remove multiple strlen() calls 

Signed-off-by: Julien Danjou <julien@danjou.info>
This commit is contained in:
Julien Danjou 2008-05-30 12:35:57 +02:00
parent c332971b21
commit 066bfaacc8
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
common/draw.c
View File

@ -1016,12 +1016,13 @@ xcolor_new(xcb_connection_t *conn, int phys_screen, const char *colstr, xcolor_t
xcb_alloc_named_color_reply_t *named_color = NULL; xcb_alloc_named_color_reply_t *named_color = NULL;
unsigned long colnum; unsigned long colnum;
uint16_t red, green, blue; uint16_t red, green, blue;
ssize_t len;
if(!a_strlen(colstr)) if(!(len = a_strlen(colstr)))
return false; return false;
/* The color is given in RGB value */ /* The color is given in RGB value */
if(colstr[0] == '#' && a_strlen(colstr) == 7) if(colstr[0] == '#' && len == 7)
{ {
errno = 0; errno = 0;
colnum = strtoul(&colstr[1], NULL, 16); colnum = strtoul(&colstr[1], NULL, 16);
@ -1057,7 +1058,7 @@ xcolor_new(xcb_connection_t *conn, int phys_screen, const char *colstr, xcolor_t
named_color = xcb_alloc_named_color_reply(conn, named_color = xcb_alloc_named_color_reply(conn,
xcb_alloc_named_color(conn, xcb_alloc_named_color(conn,
s->default_colormap, s->default_colormap,
strlen(colstr), len,
colstr), colstr),
NULL); NULL);