more error handling for checkwps

Change-Id: I03055d045c0a8e0e63e17b290cc71c54a8dc3634
This commit is contained in:
Jonathan Gordon 2013-02-27 21:15:57 +11:00
parent d76dca165b
commit 2febee5265

View file

@ -152,8 +152,11 @@ static struct skin_element* skin_parse_viewport(const char** document)
else if (callback) else if (callback)
{ {
if (callback(retval, callback_data) == CALLBACK_ERROR) if (callback(retval, callback_data) == CALLBACK_ERROR)
{
skin_error(GOT_CALLBACK_ERROR, cursor);
return NULL; return NULL;
} }
}
#endif #endif
if (check_viewport(cursor)) if (check_viewport(cursor))
@ -329,6 +332,7 @@ static struct skin_element* skin_parse_line_optional(const char** document,
switch (callback(retval, callback_data)) switch (callback(retval, callback_data))
{ {
case CALLBACK_ERROR: case CALLBACK_ERROR:
skin_error(GOT_CALLBACK_ERROR, cursor);
return NULL; return NULL;
default: default:
break; break;
@ -475,8 +479,11 @@ static struct skin_element* skin_parse_sublines_optional(const char** document,
if (callback) if (callback)
{ {
if (callback(retval, callback_data) == CALLBACK_ERROR) if (callback(retval, callback_data) == CALLBACK_ERROR)
{
skin_error(GOT_CALLBACK_ERROR, *document);
return NULL; return NULL;
} }
}
#endif #endif
*document = cursor; *document = cursor;
retval->children = skin_buffer_to_offset(children); retval->children = skin_buffer_to_offset(children);
@ -896,8 +903,11 @@ static int skin_parse_text(struct skin_element* element, const char** document,
if (callback) if (callback)
{ {
if (callback(element, callback_data) == CALLBACK_ERROR) if (callback(element, callback_data) == CALLBACK_ERROR)
{
skin_error(GOT_CALLBACK_ERROR, *document);
return 0; return 0;
} }
}
#endif #endif
*document = cursor; *document = cursor;