summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
caaf254)
An empty block was used instead of returning from the perl_config_*()
functions in case of an error. I wonder how that has happened... :-/
Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>
char *value = NULL;
if ((0 != ci->children_num) || (1 != ci->values_num)
char *value = NULL;
if ((0 != ci->children_num) || (1 != ci->values_num)
- || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
- }
+ || (OCONFIG_TYPE_STRING != ci->values[0].type))
+ return 1;
value = ci->values[0].value.string;
value = ci->values[0].value.string;
char *value = NULL;
if ((0 != ci->children_num) || (1 != ci->values_num)
char *value = NULL;
if ((0 != ci->children_num) || (1 != ci->values_num)
- || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
- }
+ || (OCONFIG_TYPE_STRING != ci->values[0].type))
+ return 1;
value = ci->values[0].value.string;
value = ci->values[0].value.string;
char *value = NULL;
if ((0 != ci->children_num) || (1 != ci->values_num)
char *value = NULL;
if ((0 != ci->children_num) || (1 != ci->values_num)
- || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
- }
+ || (OCONFIG_TYPE_STRING != ci->values[0].type))
+ return 1;
value = ci->values[0].value.string;
value = ci->values[0].value.string;
char *value = NULL;
if ((0 != ci->children_num) || (1 != ci->values_num)
char *value = NULL;
if ((0 != ci->children_num) || (1 != ci->values_num)
- || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
- }
+ || (OCONFIG_TYPE_STRING != ci->values[0].type))
+ return 1;
value = ci->values[0].value.string;
value = ci->values[0].value.string;