[Github-comments] [geany/geany] Fix the current scope shown in the statusbar (#1280)

elextr notifications at xxxxx
Sat Oct 29 10:42:54 UTC 2016


elextr 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);

@b4n, agree that them being const is correct, and assists the compiler (and human) in understanding your intention.  Its not possible to do that in the initialiser if `pos` is non-const, so better out of it.

-- 
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20161029/1c9981dd/attachment.html>


More information about the Github-comments mailing list