Hex in struct Colorpixel.hex to contiguous memory

No reason for hex not to be of a constant size, and
no reason to introduce a cache miss by allocating it
separated from the rest of the structure.

diff --git a/libi3/get_colorpixel.c b/libi3/get_colorpixel.c
index a0a9e3453ef2676f327f6eedfae3024362473636..218f82e6f5c25ce1eac70d2c1b5d43f4c427d612 100644 (file)
--- a/libi3/get_colorpixel.c
+++ b/libi3/get_colorpixel.c
@@ -13,7 +13,7 @@
 #include "libi3.h"
 
 struct Colorpixel {
-    char *hex;
+    char hex[8];
     uint32_t pixel;
 
     SLIST_ENTRY(Colorpixel)
@@ -74,7 +74,7 @@ uint32_t get_colorpixel(const char *hex) {
 
     /* Store the result in the cache */
     struct Colorpixel *cache_pixel = scalloc(1, sizeof(struct Colorpixel));
-    cache_pixel->hex = sstrdup(hex);
+    strncpy(cache_pixel->hex, hex, 8);
     cache_pixel->pixel = pixel;
 
     SLIST_INSERT_HEAD(&(colorpixels), cache_pixel, colorpixels);