@techee commented on this pull request.


> +             gint pos_start = sci_get_position_from_line(sci, line);
+               gint pos_end = sci_get_position_from_line(sci, line + 1);
+               gint pos;
+
+               for (pos = pos_start; pos < pos_end; pos++)
+               {
+                       gchar c = sci_get_char_at(sci, pos);
+                       if (c == '[')
+                               break;
+                       if (!isspace(c))
+                       {
+                               sci_insert_text(sci, pos_start, "#");
+                               break;
+                       }
+               }

Yeah, something like that, but again, your code doesn't detect empty lines 
where we shouldn't insert `#` which is the purpose of the `isspace()` test in 
my code. I also think we don't really have to check for the indented `[` or `#` 
since these aren't valid conf files.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3413#discussion_r1133337131
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3413/review/1336098...@github.com>

Reply via email to