codebrainz commented on this pull request.


> @@ -2289,6 +2290,32 @@ static gint get_fold_header_after(ScintillaObject 
> *sci, gint line)
 }
 
 
+/* returns the line after following all brace match for @brace on @line */
+static gint resolve_matching_braces(ScintillaObject *sci, gint line, gint 
brace)
+{
+       gint pos = sci_get_position_from_line(sci, line);
+       gint line_end = sci_get_line_end_position(sci, line);
+       gint lexer = sci_get_lexer(sci);

Following usually [best 
practices](https://www.securecoding.cert.org/confluence/display/c/DCL19-C.+Minimize+the+scope+of+variables+and+functions)
 they all should be really. It's not a common style in Geany yet because we 
only recently started using C99, but IMO we should follow this style wherever 
possible.

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

Reply via email to