Cleanup: style, use braces for blenkernel
[blender.git] / source / blender / blenkernel / intern / text.c
index 930f9ac..9047aa5 100644 (file)
@@ -209,8 +209,9 @@ void BKE_text_init(Text *ta)
 
   ta->nlines = 1;
   ta->flags = TXT_ISDIRTY | TXT_ISMEM;
 
   ta->nlines = 1;
   ta->flags = TXT_ISDIRTY | TXT_ISMEM;
-  if ((U.flag & USER_TXT_TABSTOSPACES_DISABLE) == 0)
+  if ((U.flag & USER_TXT_TABSTOSPACES_DISABLE) == 0) {
     ta->flags |= TXT_TABSTOSPACES;
     ta->flags |= TXT_TABSTOSPACES;
+  }
 
   BLI_listbase_clear(&ta->lines);
 
 
   BLI_listbase_clear(&ta->lines);
 
@@ -252,8 +253,9 @@ int txt_extended_ascii_as_utf8(char **str)
   int added = 0;
 
   while ((*str)[i]) {
   int added = 0;
 
   while ((*str)[i]) {
-    if ((bad_char = BLI_utf8_invalid_byte(*str + i, length - i)) == -1)
+    if ((bad_char = BLI_utf8_invalid_byte(*str + i, length - i)) == -1) {
       break;
       break;
+    }
 
     added++;
     i += bad_char + 1;
 
     added++;
     i += bad_char + 1;
@@ -321,8 +323,9 @@ static void text_from_buf(Text *text, const unsigned char *buffer, const int len
       tmp->line = (char *)MEM_mallocN(llen + 1, "textline_string");
       tmp->format = NULL;
 
       tmp->line = (char *)MEM_mallocN(llen + 1, "textline_string");
       tmp->format = NULL;
 
-      if (llen)
+      if (llen) {
         memcpy(tmp->line, &buffer[i - llen], llen);
         memcpy(tmp->line, &buffer[i - llen], llen);
+      }
       tmp->line[llen] = 0;
       tmp->len = llen;
 
       tmp->line[llen] = 0;
       tmp->len = llen;
 
@@ -350,8 +353,9 @@ static void text_from_buf(Text *text, const unsigned char *buffer, const int len
     tmp->line = (char *)MEM_mallocN(llen + 1, "textline_string");
     tmp->format = NULL;
 
     tmp->line = (char *)MEM_mallocN(llen + 1, "textline_string");
     tmp->format = NULL;
 
-    if (llen)
+    if (llen) {
       memcpy(tmp->line, &buffer[i - llen], llen);
       memcpy(tmp->line, &buffer[i - llen], llen);
+    }
 
     tmp->line[llen] = 0;
     tmp->len = llen;
 
     tmp->line[llen] = 0;
     tmp->len = llen;
@@ -412,8 +416,9 @@ Text *BKE_text_load_ex(Main *bmain, const char *file, const char *relpath, const
   BLI_stat_t st;
 
   BLI_strncpy(filepath_abs, file, FILE_MAX);
   BLI_stat_t st;
 
   BLI_strncpy(filepath_abs, file, FILE_MAX);
-  if (relpath) /* can be NULL (bg mode) */
+  if (relpath) /* can be NULL (bg mode) */
     BLI_path_abs(filepath_abs, relpath);
     BLI_path_abs(filepath_abs, relpath);
+  }
 
   buffer = BLI_file_read_text_as_mem(filepath_abs, 0, &buffer_len);
   if (buffer == NULL) {
 
   buffer = BLI_file_read_text_as_mem(filepath_abs, 0, &buffer_len);
   if (buffer == NULL) {
@@ -426,8 +431,9 @@ Text *BKE_text_load_ex(Main *bmain, const char *file, const char *relpath, const
   BLI_listbase_clear(&ta->lines);
   ta->curl = ta->sell = NULL;
 
   BLI_listbase_clear(&ta->lines);
   ta->curl = ta->sell = NULL;
 
-  if ((U.flag & USER_TXT_TABSTOSPACES_DISABLE) == 0)
+  if ((U.flag & USER_TXT_TABSTOSPACES_DISABLE) == 0) {
     ta->flags = TXT_TABSTOSPACES;
     ta->flags = TXT_TABSTOSPACES;
+  }
 
   if (is_internal == false) {
     ta->name = MEM_mallocN(strlen(file) + 1, "text_name");
 
   if (is_internal == false) {
     ta->name = MEM_mallocN(strlen(file) + 1, "text_name");
@@ -545,25 +551,30 @@ int BKE_text_file_modified_check(Text *text)
   int result;
   char file[FILE_MAX];
 
   int result;
   char file[FILE_MAX];
 
-  if (!text->name)
+  if (!text->name) {
     return 0;
     return 0;
+  }
 
   BLI_strncpy(file, text->name, FILE_MAX);
   BLI_path_abs(file, BKE_main_blendfile_path_from_global());
 
 
   BLI_strncpy(file, text->name, FILE_MAX);
   BLI_path_abs(file, BKE_main_blendfile_path_from_global());
 
-  if (!BLI_exists(file))
+  if (!BLI_exists(file)) {
     return 2;
     return 2;
+  }
 
   result = BLI_stat(file, &st);
 
 
   result = BLI_stat(file, &st);
 
-  if (result == -1)
+  if (result == -1) {
     return -1;
     return -1;
+  }
 
 
-  if ((st.st_mode & S_IFMT) != S_IFREG)
+  if ((st.st_mode & S_IFMT) != S_IFREG) {
     return -1;
     return -1;
+  }
 
 
-  if (st.st_mtime > text->mtime)
+  if (st.st_mtime > text->mtime) {
     return 1;
     return 1;
+  }
 
   return 0;
 }
 
   return 0;
 }
@@ -574,19 +585,22 @@ void BKE_text_file_modified_ignore(Text *text)
   int result;
   char file[FILE_MAX];
 
   int result;
   char file[FILE_MAX];
 
-  if (!text->name)
+  if (!text->name) {
     return;
     return;
+  }
 
   BLI_strncpy(file, text->name, FILE_MAX);
   BLI_path_abs(file, BKE_main_blendfile_path_from_global());
 
 
   BLI_strncpy(file, text->name, FILE_MAX);
   BLI_path_abs(file, BKE_main_blendfile_path_from_global());
 
-  if (!BLI_exists(file))
+  if (!BLI_exists(file)) {
     return;
     return;
+  }
 
   result = BLI_stat(file, &st);
 
 
   result = BLI_stat(file, &st);
 
-  if (result == -1 || (st.st_mode & S_IFMT) != S_IFREG)
+  if (result == -1 || (st.st_mode & S_IFMT) != S_IFREG) {
     return;
     return;
+  }
 
   text->mtime = st.st_mtime;
 }
 
   text->mtime = st.st_mtime;
 }
@@ -597,10 +611,12 @@ void BKE_text_file_modified_ignore(Text *text)
 
 static void make_new_line(TextLine *line, char *newline)
 {
 
 static void make_new_line(TextLine *line, char *newline)
 {
-  if (line->line)
+  if (line->line) {
     MEM_freeN(line->line);
     MEM_freeN(line->line);
-  if (line->format)
+  }
+  if (line->format) {
     MEM_freeN(line->format);
     MEM_freeN(line->format);
+  }
 
   line->line = newline;
   line->len = strlen(newline);
 
   line->line = newline;
   line->len = strlen(newline);
@@ -611,8 +627,9 @@ static TextLine *txt_new_line(const char *str)
 {
   TextLine *tmp;
 
 {
   TextLine *tmp;
 
-  if (!str)
+  if (!str) {
     str = "";
     str = "";
+  }
 
   tmp = (TextLine *)MEM_mallocN(sizeof(TextLine), "textline");
   tmp->line = MEM_mallocN(strlen(str) + 1, "textline_string");
 
   tmp = (TextLine *)MEM_mallocN(sizeof(TextLine), "textline");
   tmp->line = MEM_mallocN(strlen(str) + 1, "textline_string");
@@ -647,28 +664,35 @@ void txt_clean_text(Text *text)
   TextLine **top, **bot;
 
   if (!text->lines.first) {
   TextLine **top, **bot;
 
   if (!text->lines.first) {
-    if (text->lines.last)
+    if (text->lines.last) {
       text->lines.first = text->lines.last;
       text->lines.first = text->lines.last;
-    else
+    }
+    else {
       text->lines.first = text->lines.last = txt_new_line(NULL);
       text->lines.first = text->lines.last = txt_new_line(NULL);
+    }
   }
 
   }
 
-  if (!text->lines.last)
+  if (!text->lines.last) {
     text->lines.last = text->lines.first;
     text->lines.last = text->lines.first;
+  }
 
   top = (TextLine **)&text->lines.first;
   bot = (TextLine **)&text->lines.last;
 
 
   top = (TextLine **)&text->lines.first;
   bot = (TextLine **)&text->lines.last;
 
-  while ((*top)->prev)
+  while ((*top)->prev) {
     *top = (*top)->prev;
     *top = (*top)->prev;
-  while ((*bot)->next)
+  }
+  while ((*bot)->next) {
     *bot = (*bot)->next;
     *bot = (*bot)->next;
+  }
 
   if (!text->curl) {
 
   if (!text->curl) {
-    if (text->sell)
+    if (text->sell) {
       text->curl = text->sell;
       text->curl = text->sell;
-    else
+    }
+    else {
       text->curl = text->lines.first;
       text->curl = text->lines.first;
+    }
     text->curc = 0;
   }
 
     text->curc = 0;
   }
 
@@ -683,15 +707,18 @@ int txt_get_span(TextLine *from, TextLine *to)
   int ret = 0;
   TextLine *tmp = from;
 
   int ret = 0;
   TextLine *tmp = from;
 
-  if (!to || !from)
+  if (!to || !from) {
     return 0;
     return 0;
-  if (from == to)
+  }
+  if (from == to) {
     return 0;
     return 0;
+  }
 
   /* Look forwards */
   while (tmp) {
 
   /* Look forwards */
   while (tmp) {
-    if (tmp == to)
+    if (tmp == to) {
       return ret;
       return ret;
+    }
     ret++;
     tmp = tmp->next;
   }
     ret++;
     tmp = tmp->next;
   }
@@ -701,13 +728,15 @@ int txt_get_span(TextLine *from, TextLine *to)
     tmp = from;
     ret = 0;
     while (tmp) {
     tmp = from;
     ret = 0;
     while (tmp) {
-      if (tmp == to)
+      if (tmp == to) {
         break;
         break;
+      }
       ret--;
       tmp = tmp->prev;
     }
       ret--;
       tmp = tmp->prev;
     }
-    if (!tmp)
+    if (!tmp) {
       ret = 0;
       ret = 0;
+    }
   }
 
   return ret;
   }
 
   return ret;
@@ -717,8 +746,9 @@ static void txt_make_dirty(Text *text)
 {
   text->flags |= TXT_ISDIRTY;
 #ifdef WITH_PYTHON
 {
   text->flags |= TXT_ISDIRTY;
 #ifdef WITH_PYTHON
-  if (text->compiled)
+  if (text->compiled) {
     BPY_text_free_code(text);
     BPY_text_free_code(text);
+  }
 #endif
 }
 
 #endif
 }
 
@@ -787,8 +817,9 @@ int txt_utf8_column_to_offset(const char *str, int column)
   int offset = 0, pos = 0, col;
   while (*(str + offset) && pos < column) {
     col = BLI_str_utf8_char_width_safe(str + offset);
   int offset = 0, pos = 0, col;
   while (*(str + offset) && pos < column) {
     col = BLI_str_utf8_char_width_safe(str + offset);
-    if (pos + col > column)
+    if (pos + col > column) {
       break;
       break;
+    }
     offset += BLI_str_utf8_size_safe(str + offset);
     pos += col;
   }
     offset += BLI_str_utf8_size_safe(str + offset);
     pos += col;
   }
@@ -800,14 +831,16 @@ void txt_move_up(Text *text, const bool sel)
   TextLine **linep;
   int *charp;
 
   TextLine **linep;
   int *charp;
 
-  if (sel)
+  if (sel) {
     txt_curs_sel(text, &linep, &charp);
     txt_curs_sel(text, &linep, &charp);
+  }
   else {
     txt_pop_first(text);
     txt_curs_cur(text, &linep, &charp);
   }
   else {
     txt_pop_first(text);
     txt_curs_cur(text, &linep, &charp);
   }
-  if (!*linep)
+  if (!*linep) {
     return;
     return;
+  }
 
   if ((*linep)->prev) {
     int column = txt_utf8_offset_to_column((*linep)->line, *charp);
 
   if ((*linep)->prev) {
     int column = txt_utf8_offset_to_column((*linep)->line, *charp);
@@ -818,8 +851,9 @@ void txt_move_up(Text *text, const bool sel)
     txt_move_bol(text, sel);
   }
 
     txt_move_bol(text, sel);
   }
 
-  if (!sel)
+  if (!sel) {
     txt_pop_sel(text);
     txt_pop_sel(text);
+  }
 }
 
 void txt_move_down(Text *text, const bool sel)
 }
 
 void txt_move_down(Text *text, const bool sel)
@@ -827,14 +861,16 @@ void txt_move_down(Text *text, const bool sel)
   TextLine **linep;
   int *charp;
 
   TextLine **linep;
   int *charp;
 
-  if (sel)
+  if (sel) {
     txt_curs_sel(text, &linep, &charp);
     txt_curs_sel(text, &linep, &charp);
+  }
   else {
     txt_pop_last(text);
     txt_curs_cur(text, &linep, &charp);
   }
   else {
     txt_pop_last(text);
     txt_curs_cur(text, &linep, &charp);
   }
-  if (!*linep)
+  if (!*linep) {
     return;
     return;
+  }
 
   if ((*linep)->next) {
     int column = txt_utf8_offset_to_column((*linep)->line, *charp);
 
   if ((*linep)->next) {
     int column = txt_utf8_offset_to_column((*linep)->line, *charp);
@@ -845,8 +881,9 @@ void txt_move_down(Text *text, const bool sel)
     txt_move_eol(text, sel);
   }
 
     txt_move_eol(text, sel);
   }
 
-  if (!sel)
+  if (!sel) {
     txt_pop_sel(text);
     txt_pop_sel(text);
+  }
 }
 
 int txt_calc_tab_left(TextLine *tl, int ch)
 }
 
 int txt_calc_tab_left(TextLine *tl, int ch)
@@ -855,15 +892,17 @@ int txt_calc_tab_left(TextLine *tl, int ch)
 
   int tabsize = (ch < TXT_TABSIZE) ? ch : TXT_TABSIZE;
 
 
   int tabsize = (ch < TXT_TABSIZE) ? ch : TXT_TABSIZE;
 
-  for (int i = 0; i < ch; i++)
+  for (int i = 0; i < ch; i++) {
     if (tl->line[i] != ' ') {
       tabsize = 0;
       break;
     }
     if (tl->line[i] != ' ') {
       tabsize = 0;
       break;
     }
+  }
 
   /* if in the middle of the space-tab */
 
   /* if in the middle of the space-tab */
-  if (tabsize && ch % TXT_TABSIZE != 0)
+  if (tabsize && ch % TXT_TABSIZE != 0) {
     tabsize = (ch % TXT_TABSIZE);
     tabsize = (ch % TXT_TABSIZE);
+  }
   return tabsize;
 }
 
   return tabsize;
 }
 
@@ -895,14 +934,16 @@ void txt_move_left(Text *text, const bool sel)
   int *charp;
   int tabsize = 0;
 
   int *charp;
   int tabsize = 0;
 
-  if (sel)
+  if (sel) {
     txt_curs_sel(text, &linep, &charp);
     txt_curs_sel(text, &linep, &charp);
+  }
   else {
     txt_pop_first(text);
     txt_curs_cur(text, &linep, &charp);
   }
   else {
     txt_pop_first(text);
     txt_curs_cur(text, &linep, &charp);
   }
-  if (!*linep)
+  if (!*linep) {
     return;
     return;
+  }
 
   if (*charp == 0) {
     if ((*linep)->prev) {
 
   if (*charp == 0) {
     if ((*linep)->prev) {
@@ -926,8 +967,9 @@ void txt_move_left(Text *text, const bool sel)
     }
   }
 
     }
   }
 
-  if (!sel)
+  if (!sel) {
     txt_pop_sel(text);
     txt_pop_sel(text);
+  }
 }
 
 void txt_move_right(Text *text, const bool sel)
 }
 
 void txt_move_right(Text *text, const bool sel)
@@ -935,14 +977,16 @@ void txt_move_right(Text *text, const bool sel)
   TextLine **linep;
   int *charp;
 
   TextLine **linep;
   int *charp;
 
-  if (sel)
+  if (sel) {
     txt_curs_sel(text, &linep, &charp);
     txt_curs_sel(text, &linep, &charp);
+  }
   else {
     txt_pop_last(text);
     txt_curs_cur(text, &linep, &charp);
   }
   else {
     txt_pop_last(text);
     txt_curs_cur(text, &linep, &charp);
   }
-  if (!*linep)
+  if (!*linep) {
     return;
     return;
+  }
 
   if (*charp == (*linep)->len) {
     if ((*linep)->next) {
 
   if (*charp == (*linep)->len) {
     if ((*linep)->next) {
@@ -967,8 +1011,9 @@ void txt_move_right(Text *text, const bool sel)
     }
   }
 
     }
   }
 
-  if (!sel)
+  if (!sel) {
     txt_pop_sel(text);
     txt_pop_sel(text);
+  }
 }
 
 void txt_jump_left(Text *text, const bool sel, const bool use_init_step)
 }
 
 void txt_jump_left(Text *text, const bool sel, const bool use_init_step)
@@ -976,20 +1021,23 @@ void txt_jump_left(Text *text, const bool sel, const bool use_init_step)
   TextLine **linep;
   int *charp;
 
   TextLine **linep;
   int *charp;
 
-  if (sel)
+  if (sel) {
     txt_curs_sel(text, &linep, &charp);
     txt_curs_sel(text, &linep, &charp);
+  }
   else {
     txt_pop_first(text);
     txt_curs_cur(text, &linep, &charp);
   }
   else {
     txt_pop_first(text);
     txt_curs_cur(text, &linep, &charp);
   }
-  if (!*linep)
+  if (!*linep) {
     return;
     return;
+  }
 
   BLI_str_cursor_step_utf8(
       (*linep)->line, (*linep)->len, charp, STRCUR_DIR_PREV, STRCUR_JUMP_DELIM, use_init_step);
 
 
   BLI_str_cursor_step_utf8(
       (*linep)->line, (*linep)->len, charp, STRCUR_DIR_PREV, STRCUR_JUMP_DELIM, use_init_step);
 
-  if (!sel)
+  if (!sel) {
     txt_pop_sel(text);
     txt_pop_sel(text);
+  }
 }
 
 void txt_jump_right(Text *text, const bool sel, const bool use_init_step)
 }
 
 void txt_jump_right(Text *text, const bool sel, const bool use_init_step)
@@ -997,20 +1045,23 @@ void txt_jump_right(Text *text, const bool sel, const bool use_init_step)
   TextLine **linep;
   int *charp;
 
   TextLine **linep;
   int *charp;
 
-  if (sel)
+  if (sel) {
     txt_curs_sel(text, &linep, &charp);
     txt_curs_sel(text, &linep, &charp);
+  }
   else {
     txt_pop_last(text);
     txt_curs_cur(text, &linep, &charp);
   }
   else {
     txt_pop_last(text);
     txt_curs_cur(text, &linep, &charp);
   }
-  if (!*linep)
+  if (!*linep) {
     return;
     return;
+  }
 
   BLI_str_cursor_step_utf8(
       (*linep)->line, (*linep)->len, charp, STRCUR_DIR_NEXT, STRCUR_JUMP_DELIM, use_init_step);
 
 
   BLI_str_cursor_step_utf8(
       (*linep)->line, (*linep)->len, charp, STRCUR_DIR_NEXT, STRCUR_JUMP_DELIM, use_init_step);
 
-  if (!sel)
+  if (!sel) {
     txt_pop_sel(text);
     txt_pop_sel(text);
+  }
 }
 
 void txt_move_bol(Text *text, const bool sel)
 }
 
 void txt_move_bol(Text *text, const bool sel)
@@ -1018,17 +1069,21 @@ void txt_move_bol(Text *text, const bool sel)
   TextLine **linep;
   int *charp;
 
   TextLine **linep;
   int *charp;
 
-  if (sel)
+  if (sel) {
     txt_curs_sel(text, &linep, &charp);
     txt_curs_sel(text, &linep, &charp);
-  else
+  }
+  else {
     txt_curs_cur(text, &linep, &charp);
     txt_curs_cur(text, &linep, &charp);
-  if (!*linep)
+  }
+  if (!*linep) {
     return;
     return;
+  }
 
   *charp = 0;
 
 
   *charp = 0;
 
-  if (!sel)
+  if (!sel) {
     txt_pop_sel(text);
     txt_pop_sel(text);
+  }
 }
 
 void txt_move_eol(Text *text, const bool sel)
 }
 
 void txt_move_eol(Text *text, const bool sel)
@@ -1036,17 +1091,21 @@ void txt_move_eol(Text *text, const bool sel)
   TextLine **linep;
   int *charp;
 
   TextLine **linep;
   int *charp;
 
-  if (sel)
+  if (sel) {
     txt_curs_sel(text, &linep, &charp);
     txt_curs_sel(text, &linep, &charp);
-  else
+  }
+  else {
     txt_curs_cur(text, &linep, &charp);
     txt_curs_cur(text, &linep, &charp);
-  if (!*linep)
+  }
+  if (!*linep) {
     return;
     return;
+  }
 
   *charp = (*linep)->len;
 
 
   *charp = (*linep)->len;
 
-  if (!sel)
+  if (!sel) {
     txt_pop_sel(text);
     txt_pop_sel(text);
+  }
 }
 
 void txt_move_bof(Text *text, const bool sel)
 }
 
 void txt_move_bof(Text *text, const bool sel)
@@ -1054,18 +1113,22 @@ void txt_move_bof(Text *text, const bool sel)
   TextLine **linep;
   int *charp;
 
   TextLine **linep;
   int *charp;
 
-  if (sel)
+  if (sel) {
     txt_curs_sel(text, &linep, &charp);
     txt_curs_sel(text, &linep, &charp);
-  else
+  }
+  else {
     txt_curs_cur(text, &linep, &charp);
     txt_curs_cur(text, &linep, &charp);
-  if (!*linep)
+  }
+  if (!*linep) {
     return;
     return;
+  }
 
   *linep = text->lines.first;
   *charp = 0;
 
 
   *linep = text->lines.first;
   *charp = 0;
 
-  if (!sel)
+  if (!sel) {
     txt_pop_sel(text);
     txt_pop_sel(text);
+  }
 }
 
 void txt_move_eof(Text *text, const bool sel)
 }
 
 void txt_move_eof(Text *text, const bool sel)
@@ -1073,18 +1136,22 @@ void txt_move_eof(Text *text, const bool sel)
   TextLine **linep;
   int *charp;
 
   TextLine **linep;
   int *charp;
 
-  if (sel)
+  if (sel) {
     txt_curs_sel(text, &linep, &charp);
     txt_curs_sel(text, &linep, &charp);
-  else
+  }
+  else {
     txt_curs_cur(text, &linep, &charp);
     txt_curs_cur(text, &linep, &charp);
-  if (!*linep)
+  }
+  if (!*linep) {
     return;
     return;
+  }
 
   *linep = text->lines.last;
   *charp = (*linep)->len;
 
 
   *linep = text->lines.last;
   *charp = (*linep)->len;
 
-  if (!sel)
+  if (!sel) {
     txt_pop_sel(text);
     txt_pop_sel(text);
+  }
 }
 
 void txt_move_toline(Text *text, unsigned int line, const bool sel)
 }
 
 void txt_move_toline(Text *text, unsigned int line, const bool sel)
@@ -1099,26 +1166,33 @@ void txt_move_to(Text *text, unsigned int line, unsigned int ch, const bool sel)
   int *charp;
   unsigned int i;
 
   int *charp;
   unsigned int i;
 
-  if (sel)
+  if (sel) {
     txt_curs_sel(text, &linep, &charp);
     txt_curs_sel(text, &linep, &charp);
-  else
+  }
+  else {
     txt_curs_cur(text, &linep, &charp);
     txt_curs_cur(text, &linep, &charp);
-  if (!*linep)
+  }
+  if (!*linep) {
     return;
     return;
+  }
 
   *linep = text->lines.first;
   for (i = 0; i < line; i++) {
 
   *linep = text->lines.first;
   for (i = 0; i < line; i++) {
-    if ((*linep)->next)
+    if ((*linep)->next) {
       *linep = (*linep)->next;
       *linep = (*linep)->next;
-    else
+    }
+    else {
       break;
       break;
+    }
   }
   }
-  if (ch > (unsigned int)((*linep)->len))
+  if (ch > (unsigned int)((*linep)->len)) {
     ch = (unsigned int)((*linep)->len);
     ch = (unsigned int)((*linep)->len);
+  }
   *charp = ch;
 
   *charp = ch;
 
-  if (!sel)
+  if (!sel) {
     txt_pop_sel(text);
     txt_pop_sel(text);
+  }
 }
 
 /****************************/
 }
 
 /****************************/
@@ -1167,10 +1241,12 @@ void txt_pop_sel(Text *text)
 
 void txt_order_cursors(Text *text, const bool reverse)
 {
 
 void txt_order_cursors(Text *text, const bool reverse)
 {
-  if (!text->curl)
+  if (!text->curl) {
     return;
     return;
-  if (!text->sell)
+  }
+  if (!text->sell) {
     return;
     return;
+  }
 
   /* Flip so text->curl is before/after text->sell */
   if (reverse == false) {
 
   /* Flip so text->curl is before/after text->sell */
   if (reverse == false) {
@@ -1197,13 +1273,16 @@ static void txt_delete_sel(Text *text, TextUndoBuf *utxt)
   TextLine *tmpl;
   char *buf;
 
   TextLine *tmpl;
   char *buf;
 
-  if (!text->curl)
+  if (!text->curl) {
     return;
     return;
-  if (!text->sell)
+  }
+  if (!text->sell) {
     return;
     return;
+  }
 
 
-  if (!txt_has_sel(text))
+  if (!txt_has_sel(text)) {
     return;
     return;
+  }
 
   txt_order_cursors(text, false);
 
 
   txt_order_cursors(text, false);
 
@@ -1224,8 +1303,9 @@ static void txt_delete_sel(Text *text, TextUndoBuf *utxt)
   tmpl = text->sell;
   while (tmpl != text->curl) {
     tmpl = tmpl->prev;
   tmpl = text->sell;
   while (tmpl != text->curl) {
     tmpl = tmpl->prev;
-    if (!tmpl)
+    if (!tmpl) {
       break;
       break;
+    }
 
     txt_delete_line(text, tmpl->next);
   }
 
     txt_delete_line(text, tmpl->next);
   }
@@ -1258,8 +1338,9 @@ void txt_sel_clear(Text *text)
 
 void txt_sel_line(Text *text)
 {
 
 void txt_sel_line(Text *text)
 {
-  if (!text->curl)
+  if (!text->curl) {
     return;
     return;
+  }
 
   text->curc = 0;
   text->sell = text->curl;
 
   text->curc = 0;
   text->sell = text->curl;
@@ -1277,12 +1358,15 @@ char *txt_to_buf(Text *text)
   int charf, charl;
   char *buf;
 
   int charf, charl;
   char *buf;
 
-  if (!text->curl)
+  if (!text->curl) {
     return NULL;
     return NULL;
-  if (!text->sell)
+  }
+  if (!text->sell) {
     return NULL;
     return NULL;
-  if (!text->lines.first)
+  }
+  if (!text->lines.first) {
     return NULL;
     return NULL;
+  }
 
   linef = text->lines.first;
   charf = 0;
 
   linef = text->lines.first;
   charf = 0;
@@ -1341,32 +1425,40 @@ int txt_find_string(Text *text, const char *findstr, int wrap, int match_case)
   TextLine *tl, *startl;
   const char *s = NULL;
 
   TextLine *tl, *startl;
   const char *s = NULL;
 
-  if (!text->curl || !text->sell)
+  if (!text->curl || !text->sell) {
     return 0;
     return 0;
+  }
 
   txt_order_cursors(text, false);
 
   tl = startl = text->sell;
 
 
   txt_order_cursors(text, false);
 
   tl = startl = text->sell;
 
-  if (match_case)
+  if (match_case) {
     s = strstr(&tl->line[text->selc], findstr);
     s = strstr(&tl->line[text->selc], findstr);
-  else
+  }
+  else {
     s = BLI_strcasestr(&tl->line[text->selc], findstr);
     s = BLI_strcasestr(&tl->line[text->selc], findstr);
+  }
   while (!s) {
     tl = tl->next;
     if (!tl) {
   while (!s) {
     tl = tl->next;
     if (!tl) {
-      if (wrap)
+      if (wrap) {
         tl = text->lines.first;
         tl = text->lines.first;
-      else
+      }
+      else {
         break;
         break;
+      }
     }
 
     }
 
-    if (match_case)
+    if (match_case) {
       s = strstr(tl->line, findstr);
       s = strstr(tl->line, findstr);
-    else
+    }
+    else {
       s = BLI_strcasestr(tl->line, findstr);
       s = BLI_strcasestr(tl->line, findstr);
-    if (tl == startl)
+    }
+    if (tl == startl) {
       break;
       break;
+    }
   }
 
   if (s) {
   }
 
   if (s) {
@@ -1376,8 +1468,9 @@ int txt_find_string(Text *text, const char *findstr, int wrap, int match_case)
     txt_move_to(text, newl, newc + strlen(findstr), 1);
     return 1;
   }
     txt_move_to(text, newl, newc + strlen(findstr), 1);
     return 1;
   }
-  else
+  else {
     return 0;
     return 0;
+  }
 }
 
 char *txt_sel_to_buf(Text *text)
 }
 
 char *txt_sel_to_buf(Text *text)
@@ -1387,10 +1480,12 @@ char *txt_sel_to_buf(Text *text)
   TextLine *tmp, *linef, *linel;
   int charf, charl;
 
   TextLine *tmp, *linef, *linel;
   int charf, charl;
 
-  if (!text->curl)
+  if (!text->curl) {
     return NULL;
     return NULL;
-  if (!text->sell)
+  }
+  if (!text->sell) {
     return NULL;
     return NULL;
+  }
 
   if (text->curl == text->sell) {
     linef = linel = text->curl;
 
   if (text->curl == text->sell) {
     linef = linel = text->curl;
@@ -1470,8 +1565,9 @@ void txt_insert_buf(Text *text, TextUndoBuf *utxt, const char *in_buffer)
   TextLine *add;
   char *buffer;
 
   TextLine *add;
   char *buffer;
 
-  if (!in_buffer)
+  if (!in_buffer) {
     return;
     return;
+  }
 
   txt_delete_sel(text, utxt);
 
 
   txt_delete_sel(text, utxt);
 
@@ -2267,8 +2363,9 @@ void txt_do_undo(Text *text, TextUndoBuf *utxt)
         prev_flags = text->flags;
         text->flags &= ~TXT_TABSTOSPACES;
 
         prev_flags = text->flags;
         text->flags &= ~TXT_TABSTOSPACES;
 
-        for (i = 0; i < linep; i++)
+        for (i = 0; i < linep; i++) {
           txt_move_right(text, 1);
           txt_move_right(text, 1);
+        }
 
         text->flags = prev_flags;
       }
 
         text->flags = prev_flags;
       }
@@ -2559,27 +2656,31 @@ void txt_split_curline(Text *text, TextUndoBuf *utxt)
   TextLine *ins;
   char *left, *right;
 
   TextLine *ins;
   char *left, *right;
 
-  if (!text->curl)
+  if (!text->curl) {
     return;
     return;
+  }
 
   txt_delete_sel(text, utxt);
 
 
   txt_delete_sel(text, utxt);
 
-  if (!undoing)
+  if (!undoing) {
     txt_undo_add_charop(text, utxt, UNDO_INSERT_1, '\n');
     txt_undo_add_charop(text, utxt, UNDO_INSERT_1, '\n');
+  }
 
   /* Make the two half strings */
 
   left = MEM_mallocN(text->curc + 1, "textline_string");
 
   /* Make the two half strings */
 
   left = MEM_mallocN(text->curc + 1, "textline_string");
-  if (text->curc)
+  if (text->curc) {
     memcpy(left, text->curl->line, text->curc);
     memcpy(left, text->curl->line, text->curc);
+  }
   left[text->curc] = 0;
 
   right = MEM_mallocN(text->curl->len - text->curc + 1, "textline_string");
   memcpy(right, text->curl->line + text->curc, text->curl->len - text->curc + 1);
 
   MEM_freeN(text->curl->line);
   left[text->curc] = 0;
 
   right = MEM_mallocN(text->curl->len - text->curc + 1, "textline_string");
   memcpy(right, text->curl->line + text->curc, text->curl->len - text->curc + 1);
 
   MEM_freeN(text->curl->line);
-  if (text->curl->format)
+  if (text->curl->format) {
     MEM_freeN(text->curl->format);
     MEM_freeN(text->curl->format);
+  }
 
   /* Make the new TextLine */
 
 
   /* Make the new TextLine */
 
@@ -2604,15 +2705,18 @@ void txt_split_curline(Text *text, TextUndoBuf *utxt)
 
 static void txt_delete_line(Text *text, TextLine *line)
 {
 
 static void txt_delete_line(Text *text, TextLine *line)
 {
-  if (!text->curl)
+  if (!text->curl) {
     return;
     return;
+  }
 
   BLI_remlink(&text->lines, line);
 
 
   BLI_remlink(&text->lines, line);
 
-  if (line->line)
+  if (line->line) {
     MEM_freeN(line->line);
     MEM_freeN(line->line);
-  if (line->format)
+  }
+  if (line->format) {
     MEM_freeN(line->format);
     MEM_freeN(line->format);
+  }
 
   MEM_freeN(line);
 
 
   MEM_freeN(line);
 
@@ -2624,8 +2728,9 @@ static void txt_combine_lines(Text *text, TextLine *linea, TextLine *lineb)
 {
   char *tmp, *s;
 
 {
   char *tmp, *s;
 
-  if (!linea || !lineb)
+  if (!linea || !lineb) {
     return;
     return;
+  }
 
   tmp = MEM_mallocN(linea->len + lineb->len + 1, "textline_string");
 
 
   tmp = MEM_mallocN(linea->len + lineb->len + 1, "textline_string");
 
@@ -2646,8 +2751,9 @@ void txt_duplicate_line(Text *text, TextUndoBuf *utxt)
 {
   TextLine *textline;
 
 {
   TextLine *textline;
 
-  if (!text->curl)
+  if (!text->curl) {
     return;
     return;
+  }
 
   if (text->curl == text->sell) {
     textline = txt_new_line(text->curl->line);
 
   if (text->curl == text->sell) {
     textline = txt_new_line(text->curl->line);
@@ -2656,8 +2762,9 @@ void txt_duplicate_line(Text *text, TextUndoBuf *utxt)
     txt_make_dirty(text);
     txt_clean_text(text);
 
     txt_make_dirty(text);
     txt_clean_text(text);
 
-    if (!undoing)
+    if (!undoing) {
       txt_undo_add_op(text, utxt, UNDO_DUPLICATE);
       txt_undo_add_op(text, utxt, UNDO_DUPLICATE);
+    }
   }
 }
 
   }
 }
 
@@ -2665,8 +2772,9 @@ void txt_delete_char(Text *text, TextUndoBuf *utxt)
 {
   unsigned int c = '\n';
 
 {
   unsigned int c = '\n';
 
-  if (!text->curl)
+  if (!text->curl) {
     return;
     return;
+  }
 
   if (txt_has_sel(text)) { /* deleting a selection */
     txt_delete_sel(text, utxt);
 
   if (txt_has_sel(text)) { /* deleting a selection */
     txt_delete_sel(text, utxt);
@@ -2678,8 +2786,9 @@ void txt_delete_char(Text *text, TextUndoBuf *utxt)
       txt_combine_lines(text, text->curl, text->curl->next);
       txt_pop_sel(text);
     }
       txt_combine_lines(text, text->curl, text->curl->next);
       txt_pop_sel(text);
     }
-    else
+    else {
       return;
       return;
+    }
   }
   else { /* Just deleting a char */
     size_t c_len = 0;
   }
   else { /* Just deleting a char */
     size_t c_len = 0;
@@ -2697,8 +2806,9 @@ void txt_delete_char(Text *text, TextUndoBuf *utxt)
   txt_make_dirty(text);
   txt_clean_text(text);
 
   txt_make_dirty(text);
   txt_clean_text(text);
 
-  if (!undoing)
+  if (!undoing) {
     txt_undo_add_charop(text, utxt, UNDO_DEL_1, c);
     txt_undo_add_charop(text, utxt, UNDO_DEL_1, c);
+  }
 }
 
 void txt_delete_word(Text *text, TextUndoBuf *utxt)
 }
 
 void txt_delete_word(Text *text, TextUndoBuf *utxt)
@@ -2712,8 +2822,9 @@ void txt_backspace_char(Text *text, TextUndoBuf *utxt)
 {
   unsigned int c = '\n';
 
 {
   unsigned int c = '\n';
 
-  if (!text->curl)
+  if (!text->curl) {
     return;
     return;
+  }
 
   if (txt_has_sel(text)) { /* deleting a selection */
     txt_delete_sel(text, utxt);
 
   if (txt_has_sel(text)) { /* deleting a selection */
     txt_delete_sel(text, utxt);
@@ -2721,8 +2832,9 @@ void txt_backspace_char(Text *text, TextUndoBuf *utxt)
     return;
   }
   else if (text->curc == 0) { /* Appending two lines */
     return;
   }
   else if (text->curc == 0) { /* Appending two lines */
-    if (!text->curl->prev)
+    if (!text->curl->prev) {
       return;
       return;
+    }
 
     text->curl = text->curl->prev;
     text->curc = text->curl->len;
 
     text->curl = text->curl->prev;
     text->curc = text->curl->len;
@@ -2749,8 +2861,9 @@ void txt_backspace_char(Text *text, TextUndoBuf *utxt)
   txt_make_dirty(text);
   txt_clean_text(text);
 
   txt_make_dirty(text);
   txt_clean_text(text);
 
-  if (!undoing)
+  if (!undoing) {
     txt_undo_add_charop(text, utxt, UNDO_BS_1, c);
     txt_undo_add_charop(text, utxt, UNDO_BS_1, c);
+  }
 }
 
 void txt_backspace_word(Text *text, TextUndoBuf *utxt)
 }
 
 void txt_backspace_word(Text *text, TextUndoBuf *utxt)
@@ -2780,8 +2893,9 @@ static bool txt_add_char_intern(Text *text, TextUndoBuf *utxt, unsigned int add,
   char *tmp, ch[BLI_UTF8_MAX];
   size_t add_len;
 
   char *tmp, ch[BLI_UTF8_MAX];
   size_t add_len;
 
-  if (!text->curl)
+  if (!text->curl) {
     return 0;
     return 0;
+  }
 
   if (add == '\n') {
     txt_split_curline(text, utxt);
 
   if (add == '\n') {
     txt_split_curline(text, utxt);
@@ -2796,8 +2910,9 @@ static bool txt_add_char_intern(Text *text, TextUndoBuf *utxt, unsigned int add,
 
   txt_delete_sel(text, utxt);
 
 
   txt_delete_sel(text, utxt);
 
-  if (!undoing)
+  if (!undoing) {
     txt_undo_add_charop(text, utxt, UNDO_INSERT_1, add);
     txt_undo_add_charop(text, utxt, UNDO_INSERT_1, add);
+  }
 
   add_len = BLI_str_utf8_from_unicode(add, ch);
 
 
   add_len = BLI_str_utf8_from_unicode(add, ch);
 
@@ -2842,8 +2957,9 @@ bool txt_replace_char(Text *text, TextUndoBuf *utxt, unsigned int add)
   size_t del_size = 0, add_size;
   char ch[BLI_UTF8_MAX];
 
   size_t del_size = 0, add_size;
   char ch[BLI_UTF8_MAX];
 
-  if (!text->curl)
+  if (!text->curl) {
     return false;
     return false;
+  }
 
   /* If text is selected or we're at the end of the line just use txt_add_char */
   if (text->curc == text->curl->len || txt_has_sel(text) || add == '\n') {
 
   /* If text is selected or we're at the end of the line just use txt_add_char */
   if (text->curc == text->curl->len || txt_has_sel(text) || add == '\n') {
@@ -2913,13 +3029,15 @@ static void txt_select_prefix(Text *text, const char *add)
       tmp = MEM_mallocN(text->curl->len + indentlen + 1, "textline_string");
 
       text->curc = 0;
       tmp = MEM_mallocN(text->curl->len + indentlen + 1, "textline_string");
 
       text->curc = 0;
-      if (text->curc)
+      if (text->curc) {
         memcpy(tmp, text->curl->line, text->curc); /* XXX never true, check prev line */
         memcpy(tmp, text->curl->line, text->curc); /* XXX never true, check prev line */
+      }
       memcpy(tmp + text->curc, add, indentlen);
 
       len = text->curl->len - text->curc;
       memcpy(tmp + text->curc, add, indentlen);
 
       len = text->curl->len - text->curc;
-      if (len > 0)
+      if (len > 0) {
         memcpy(tmp + text->curc + indentlen, text->curl->line + text->curc, len);
         memcpy(tmp + text->curc + indentlen, text->curl->line + text->curc, len);
+      }
       tmp[text->curl->len + indentlen] = 0;
 
       make_new_line(text->curl, tmp);
       tmp[text->curl->len + indentlen] = 0;
 
       make_new_line(text->curl, tmp);
@@ -2939,10 +3057,12 @@ static void txt_select_prefix(Text *text, const char *add)
       num++;
     }
   }
       num++;
     }
   }
-  if (!curc_old)
+  if (!curc_old) {
     text->curc = 0;
     text->curc = 0;
-  else
+  }
+  else {
     text->curc = curc_old + indentlen;
     text->curc = curc_old + indentlen;
+  }
 
   while (num > 0) {
     text->curl = text->curl->prev;
 
   while (num > 0) {
     text->curl = text->curl->prev;
@@ -2983,8 +3103,9 @@ static void txt_select_unprefix(Text *text,
   while (true) {
     bool changed = false;
     if (STREQLEN(text->curl->line, remove, indentlen)) {
   while (true) {
     bool changed = false;
     if (STREQLEN(text->curl->line, remove, indentlen)) {
-      if (num == 0)
+      if (num == 0) {
         unindented_first = true;
         unindented_first = true;
+      }
       text->curl->len -= indentlen;
       memmove(text->curl->line, text->curl->line + indentlen, text->curl->len + 1);
       changed = true;
       text->curl->len -= indentlen;
       memmove(text->curl->line, text->curl->line + indentlen, text->curl->len + 1);
       changed = true;
@@ -3004,8 +3125,9 @@ static void txt_select_unprefix(Text *text,
     txt_clean_text(text);
 
     if (text->curl == text->sell) {
     txt_clean_text(text);
 
     if (text->curl == text->sell) {
-      if (changed)
+      if (changed) {
         text->selc = MAX2(text->selc - indentlen, 0);
         text->selc = MAX2(text->selc - indentlen, 0);
+      }
       break;
     }
     else {
       break;
     }
     else {
@@ -3014,8 +3136,9 @@ static void txt_select_unprefix(Text *text,
     }
   }
 
     }
   }
 
-  if (unindented_first)
+  if (unindented_first) {
     text->curc = MAX2(text->curc - indentlen, 0);
     text->curc = MAX2(text->curc - indentlen, 0);
+  }
 
   while (num > 0) {
     text->curl = text->curl->prev;
 
   while (num > 0) {
     text->curl = text->curl->prev;
@@ -3099,15 +3222,17 @@ void txt_move_lines(struct Text *text, TextUndoBuf *utxt, const int direction)
 
   BLI_assert(ELEM(direction, TXT_MOVE_LINE_UP, TXT_MOVE_LINE_DOWN));
 
 
   BLI_assert(ELEM(direction, TXT_MOVE_LINE_UP, TXT_MOVE_LINE_DOWN));
 
-  if (!text->curl || !text->sell)
+  if (!text->curl || !text->sell) {
     return;
     return;
+  }
 
   txt_order_cursors(text, false);
 
   line_other = (direction == TXT_MOVE_LINE_DOWN) ? text->sell->next : text->curl->prev;
 
 
   txt_order_cursors(text, false);
 
   line_other = (direction == TXT_MOVE_LINE_DOWN) ? text->sell->next : text->curl->prev;
 
-  if (!line_other)
+  if (!line_other) {
     return;
     return;
+  }
 
   BLI_remlink(&text->lines, line_other);
 
 
   BLI_remlink(&text->lines, line_other);
 
@@ -3136,8 +3261,9 @@ int txt_setcurr_tab_spaces(Text *text, int space)
   const char indent = (text->flags & TXT_TABSTOSPACES) ? ' ' : '\t';
   static const char *back_words[] = {"return", "break", "continue", "pass", "yield", NULL};
 
   const char indent = (text->flags & TXT_TABSTOSPACES) ? ' ' : '\t';
   static const char *back_words[] = {"return", "break", "continue", "pass", "yield", NULL};
 
-  if (!text->curl)
+  if (!text->curl) {
     return 0;
     return 0;
+  }
 
   while (text->curl->line[i] == indent) {
     //we only count those tabs/spaces that are before any text or before the curs;
 
   while (text->curl->line[i] == indent) {
     //we only count those tabs/spaces that are before any text or before the curs;
@@ -3195,10 +3321,12 @@ int text_check_bracket(const char ch)
   char close[] = ")]}";
 
   for (a = 0; a < (sizeof(opens) - 1); a++) {
   char close[] = ")]}";
 
   for (a = 0; a < (sizeof(opens) - 1); a++) {
-    if (ch == opens[a])
+    if (ch == opens[a]) {
       return a + 1;
       return a + 1;
-    else if (ch == close[a])
+    }
+    else if (ch == close[a]) {
       return -(a + 1);
       return -(a + 1);
+    }
   }
   return 0;
 }
   }
   return 0;
 }
@@ -3210,50 +3338,64 @@ bool text_check_delim(const char ch)
   char delims[] = "():\"\' ~!%^&*-+=[]{};/<>|.#\t,@";
 
   for (a = 0; a < (sizeof(delims) - 1); a++) {
   char delims[] = "():\"\' ~!%^&*-+=[]{};/<>|.#\t,@";
 
   for (a = 0; a < (sizeof(delims) - 1); a++) {
-    if (ch == delims[a])
+    if (ch == delims[a]) {
       return true;
       return true;
+    }
   }
   return false;
 }
 
 bool text_check_digit(const char ch)
 {
   }
   return false;
 }
 
 bool text_check_digit(const char ch)
 {
-  if (ch < '0')
+  if (ch < '0') {
     return false;
     return false;
-  if (ch <= '9')
+  }
+  if (ch <= '9') {
     return true;
     return true;
+  }
   return false;
 }
 
 bool text_check_identifier(const char ch)
 {
   return false;
 }
 
 bool text_check_identifier(const char ch)
 {
-  if (ch < '0')
+  if (ch < '0') {
     return false;
     return false;
-  if (ch <= '9')
+  }
+  if (ch <= '9') {
     return true;
     return true;
-  if (ch < 'A')
+  }
+  if (ch < 'A') {
     return false;
     return false;
-  if (ch <= 'Z' || ch == '_')
+  }
+  if (ch <= 'Z' || ch == '_') {
     return true;
     return true;
-  if (ch < 'a')
+  }
+  if (ch < 'a') {
     return false;
     return false;
-  if (ch <= 'z')
+  }
+  if (ch <= 'z') {
     return true;
     return true;
+  }
   return false;
 }
 
 bool text_check_identifier_nodigit(const char ch)
 {
   return false;
 }
 
 bool text_check_identifier_nodigit(const char ch)
 {
-  if (ch <= '9')
+  if (ch <= '9') {
     return false;
     return false;
-  if (ch < 'A')
+  }
+  if (ch < 'A') {
     return false;
     return false;
-  if (ch <= 'Z' || ch == '_')
+  }
+  if (ch <= 'Z' || ch == '_') {
     return true;
     return true;
-  if (ch < 'a')
+  }
+  if (ch < 'a') {
     return false;
     return false;
-  if (ch <= 'z')
+  }
+  if (ch <= 'z') {
     return true;
     return true;
+  }
   return false;
 }
 
   return false;
 }
 
@@ -3271,8 +3413,9 @@ int text_check_identifier_nodigit_unicode(const unsigned int ch)
 
 bool text_check_whitespace(const char ch)
 {
 
 bool text_check_whitespace(const char ch)
 {
-  if (ch == ' ' || ch == '\t' || ch == '\r' || ch == '\n')
+  if (ch == ' ' || ch == '\t' || ch == '\r' || ch == '\n') {
     return true;
     return true;
+  }
   return false;
 }
 
   return false;
 }