diff options
author | Campbell Barton <ideasman42@gmail.com> | 2019-04-22 02:39:35 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2019-04-22 12:48:17 +0300 |
commit | 735515a3f9e4c41738bf714d682b13db64adb638 (patch) | |
tree | 04b551336131bc84ecc664a77ae0cefdded78d0b /source/blender/blenkernel/intern/report.c | |
parent | 14a49950ff11f43b7a5a73da545339b969de97f0 (diff) |
Cleanup: style, use braces for blenkernel
Diffstat (limited to 'source/blender/blenkernel/intern/report.c')
-rw-r--r-- | source/blender/blenkernel/intern/report.c | 51 |
1 files changed, 34 insertions, 17 deletions
diff --git a/source/blender/blenkernel/intern/report.c b/source/blender/blenkernel/intern/report.c index 1319c6de835..5db091e6a43 100644 --- a/source/blender/blenkernel/intern/report.c +++ b/source/blender/blenkernel/intern/report.c @@ -65,8 +65,9 @@ const char *BKE_report_type_str(ReportType type) void BKE_reports_init(ReportList *reports, int flag) { - if (!reports) + if (!reports) { return; + } memset(reports, 0, sizeof(ReportList)); @@ -79,8 +80,9 @@ void BKE_reports_clear(ReportList *reports) { Report *report, *report_next; - if (!reports) + if (!reports) { return; + } report = reports->list.first; @@ -163,8 +165,9 @@ void BKE_reports_prepend(ReportList *reports, const char *_prepend) DynStr *ds; const char *prepend = TIP_(_prepend); - if (!reports) + if (!reports) { return; + } for (report = reports->list.first; report; report = report->next) { ds = BLI_dynstr_new(); @@ -187,8 +190,9 @@ void BKE_reports_prependf(ReportList *reports, const char *_prepend, ...) va_list args; const char *prepend = TIP_(_prepend); - if (!reports) + if (!reports) { return; + } for (report = reports->list.first; report; report = report->next) { ds = BLI_dynstr_new(); @@ -208,32 +212,36 @@ void BKE_reports_prependf(ReportList *reports, const char *_prepend, ...) ReportType BKE_report_print_level(ReportList *reports) { - if (!reports) + if (!reports) { return RPT_ERROR; + } return reports->printlevel; } void BKE_report_print_level_set(ReportList *reports, ReportType level) { - if (!reports) + if (!reports) { return; + } reports->printlevel = level; } ReportType BKE_report_store_level(ReportList *reports) { - if (!reports) + if (!reports) { return RPT_ERROR; + } return reports->storelevel; } void BKE_report_store_level_set(ReportList *reports, ReportType level) { - if (!reports) + if (!reports) { return; + } reports->storelevel = level; } @@ -244,18 +252,23 @@ char *BKE_reports_string(ReportList *reports, ReportType level) DynStr *ds; char *cstring; - if (!reports || !reports->list.first) + if (!reports || !reports->list.first) { return NULL; + } ds = BLI_dynstr_new(); - for (report = reports->list.first; report; report = report->next) - if (report->type >= level) + for (report = reports->list.first; report; report = report->next) { + if (report->type >= level) { BLI_dynstr_appendf(ds, "%s: %s\n", report->typestr, report->message); + } + } - if (BLI_dynstr_get_len(ds)) + if (BLI_dynstr_get_len(ds)) { cstring = BLI_dynstr_get_cstring(ds); - else + } + else { cstring = NULL; + } BLI_dynstr_free(ds); return cstring; @@ -265,8 +278,9 @@ void BKE_reports_print(ReportList *reports, ReportType level) { char *cstring = BKE_reports_string(reports, level); - if (cstring == NULL) + if (cstring == NULL) { return; + } puts(cstring); fflush(stdout); @@ -278,8 +292,9 @@ Report *BKE_reports_last_displayable(ReportList *reports) Report *report; for (report = reports->list.last; report; report = report->prev) { - if (ELEM(report->type, RPT_ERROR, RPT_WARNING, RPT_INFO)) + if (ELEM(report->type, RPT_ERROR, RPT_WARNING, RPT_INFO)) { return report; + } } return NULL; @@ -289,9 +304,11 @@ bool BKE_reports_contain(ReportList *reports, ReportType level) { Report *report; if (reports != NULL) { - for (report = reports->list.first; report; report = report->next) - if (report->type >= level) + for (report = reports->list.first; report; report = report->next) { + if (report->type >= level) { return true; + } + } } return false; } |