From 8cf26abf24f923e2ff19794e56f95a1d7ceb89e4 Mon Sep 17 00:00:00 2001 From: Scott Wolchok Date: Fri, 18 Apr 2014 13:41:14 -0700 Subject: [PATCH 1/2] Use isnan(3) to check for NaN --- src/Layout.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Layout.c b/src/Layout.c index 5f496c97..82a9060c 100644 --- a/src/Layout.c +++ b/src/Layout.c @@ -125,7 +125,7 @@ void print_number_0(const char *str, float number) { } } void print_number_nan(const char *str, float number) { - if (number == number) { + if (!isnan(number)) { printf("%s: %g, ", str, number); } } @@ -240,7 +240,7 @@ int dim[2] = { bool isUndefined(float value) { - return value != value; // NaN check + return isnan(value); } float getMargin(css_node_t *node, int location) { From 54985160873fd5d5af2f64b347e4fff864c48e91 Mon Sep 17 00:00:00 2001 From: Scott Wolchok Date: Fri, 18 Apr 2014 13:45:57 -0700 Subject: [PATCH 2/2] fix memory leak found by valgrind --- src/Layout.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/Layout.c b/src/Layout.c index 82a9060c..f9e71102 100644 --- a/src/Layout.c +++ b/src/Layout.c @@ -107,6 +107,7 @@ void free_css_node(css_node_t *node) { for (int i = 0; i < node->children_count; ++i) { free_css_node(node->children[i]); } + free(node->children); free(node); }