Cleanup: Clang-tidy, readability-else-after-return

This commit is contained in:
Ankit Meel 2020-11-07 21:21:02 +05:30
parent 4429b4b77e
commit 7be47dadea
2 changed files with 20 additions and 42 deletions

View File

@ -2611,9 +2611,7 @@ int RNA_property_int_get(PointerRNA *ptr, PropertyRNA *prop)
if (iprop->get_ex) {
return iprop->get_ex(ptr, prop);
}
else {
return iprop->defaultvalue;
}
return iprop->defaultvalue;
}
void RNA_property_int_set(PointerRNA *ptr, PropertyRNA *prop, int value)
@ -2947,12 +2945,10 @@ float RNA_property_float_get(PointerRNA *ptr, PropertyRNA *prop)
if (fprop->get) {
return fprop->get(ptr);
}
else if (fprop->get_ex) {
if (fprop->get_ex) {
return fprop->get_ex(ptr, prop);
}
else {
return fprop->defaultvalue;
}
return fprop->defaultvalue;
}
void RNA_property_float_set(PointerRNA *ptr, PropertyRNA *prop, float value)
@ -3386,12 +3382,10 @@ int RNA_property_string_length(PointerRNA *ptr, PropertyRNA *prop)
if (sprop->length) {
return sprop->length(ptr);
}
else if (sprop->length_ex) {
if (sprop->length_ex) {
return sprop->length_ex(ptr, prop);
}
else {
return strlen(sprop->defaultvalue);
}
return strlen(sprop->defaultvalue);
}
void RNA_property_string_set(PointerRNA *ptr, PropertyRNA *prop, const char *value)
@ -3552,9 +3546,7 @@ int RNA_property_enum_get(PointerRNA *ptr, PropertyRNA *prop)
if (eprop->get_ex) {
return eprop->get_ex(ptr, prop);
}
else {
return eprop->defaultvalue;
}
return eprop->defaultvalue;
}
void RNA_property_enum_set(PointerRNA *ptr, PropertyRNA *prop, int value)
@ -3668,15 +3660,13 @@ PointerRNA RNA_property_pointer_get(PointerRNA *ptr, PropertyRNA *prop)
if (pprop->get) {
return pprop->get(ptr);
}
else if (prop->flag & PROP_IDPROPERTY) {
if (prop->flag & PROP_IDPROPERTY) {
/* XXX temporary hack to add it automatically, reading should
* never do any write ops, to ensure thread safety etc .. */
RNA_property_pointer_add(ptr, prop);
return RNA_property_pointer_get(ptr, prop);
}
else {
return PointerRNA_NULL;
}
return PointerRNA_NULL;
}
void RNA_property_pointer_set(PointerRNA *ptr,

View File

@ -124,10 +124,8 @@ static bool parse_int_relative(const char *str,
*r_err_msg = msg;
return false;
}
else {
*r_value = (int)value;
return true;
}
*r_value = (int)value;
return true;
}
static const char *parse_int_range_sep_search(const char *str, const char *str_end_test)
@ -230,10 +228,8 @@ static bool parse_int_strict_range(const char *str,
*r_err_msg = msg;
return false;
}
else {
*r_value = (int)value;
return true;
}
*r_value = (int)value;
return true;
}
static bool parse_int(const char *str,
@ -1589,10 +1585,8 @@ static int arg_handle_render_frame(int argc, const char **argv, void *data)
printf("\nError: frame number must follow '%s'.\n", arg_id);
return 0;
}
else {
printf("\nError: no blend loaded. cannot use '%s'.\n", arg_id);
return 0;
}
printf("\nError: no blend loaded. cannot use '%s'.\n", arg_id);
return 0;
}
static const char arg_handle_render_animation_doc[] =
@ -1674,10 +1668,8 @@ static int arg_handle_frame_start_set(int argc, const char **argv, void *data)
printf("\nError: frame number must follow '%s'.\n", arg_id);
return 0;
}
else {
printf("\nError: no blend loaded. cannot use '%s'.\n", arg_id);
return 0;
}
printf("\nError: no blend loaded. cannot use '%s'.\n", arg_id);
return 0;
}
static const char arg_handle_frame_end_set_doc[] =
@ -1709,10 +1701,8 @@ static int arg_handle_frame_end_set(int argc, const char **argv, void *data)
printf("\nError: frame number must follow '%s'.\n", arg_id);
return 0;
}
else {
printf("\nError: no blend loaded. cannot use '%s'.\n", arg_id);
return 0;
}
printf("\nError: no blend loaded. cannot use '%s'.\n", arg_id);
return 0;
}
static const char arg_handle_frame_skip_set_doc[] =
@ -1737,10 +1727,8 @@ static int arg_handle_frame_skip_set(int argc, const char **argv, void *data)
printf("\nError: number of frames to step must follow '%s'.\n", arg_id);
return 0;
}
else {
printf("\nError: no blend loaded. cannot use '%s'.\n", arg_id);
return 0;
}
printf("\nError: no blend loaded. cannot use '%s'.\n", arg_id);
return 0;
}
static const char arg_handle_python_file_run_doc[] =