Cleanup: style, use braces for blenkernel
[blender.git] / source / blender / blenkernel / intern / report.c
index 1319c6d..5db091e 100644 (file)
@@ -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;
 }