Damn. Right!

Thanks for reviewing!

Best Regards,
Lars


On Sat, May 18, 2013 at 6:07 PM, Emilio Pozuelo Monfort <pochu27@gmail.com> wrote:
Hi Lars!


On 18/05/13 17:51, Lars Windolf wrote:
+gboolean
+common_str_is_empty (const gchar *s)
+{
+       g_return_val_if_fail (s != NULL, TRUE);
+
+       while (*s != '\0') {
+               if (!g_ascii_isspace (*s))
+                       return FALSE;
+       }
+
+       return TRUE;
+}

Can't this lead to an infinite loop? Looks to me like there should be an 's++;' inside the while loop.

Regards,
Emilio