<p></p>
<p><b>@elextr</b> requested changes on this pull request.</p>

<p>Sorry about the dual reviews, I wanted to see if the first one would attach to the commit (it didn't) but then I couldn't add more to it.</p><hr>

<p>In <a href="https://github.com/geany/geany/pull/2943#discussion_r730193079">src/editor.c</a>:</p>
<pre style='color:#555'>>  
 static void auto_close_chars(ScintillaObject *sci, gint pos, gchar c)
 {
        const gchar *closing_char = NULL;
-       gint end_pos = -1;
-
-       if (utils_isbrace(c, 0))
-               end_pos = sci_find_matching_brace(sci, pos - 1);
</pre>
<p>This was deliberately added in <a class="commit-link" href="https://github.com/geany/geany/commit/ff8664150bab4a9f59120c8d14ca7240ab21a5b6#diff-3fe5ce523684a7fca311fab0d09542552fbe3a28ddaa20ebc5a65c1904bc5e02"><tt>ff86641</tt>#diff-3fe5ce523684a7fca311fab0d09542552fbe3a28ddaa20ebc5a65c1904bc5e02</a> so it is desired by at least some people, and nobody (AFAICT) has wanted it changed since 2008.  So removing it just because one person doesn't like it doesn't seem reasonable.</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/2943#discussion_r730193467">doc/geany.txt</a>:</p>
<pre style='color:#555'>> @@ -2193,6 +2193,10 @@ Geany can automatically insert a closing bracket and quote characters when
 you open them. For instance, you type a ``(`` and Geany will automatically
 insert ``)``. With the following options, you can define for which
 characters this should work.
+For a more flowing writing (not move the typing hand to arrow keys) Geany
+checks if the character written is the closing quote or bracket that was
+previously inserted. In that case only one quote or bracket of the one written
+and the one previously inserted remains.
</pre>
<p>I disagree, it should be less concise and actually explain what happens, so its known that the behaviour is deliberate, especially as pretty much whatever autoclose behaviour is chosen there are edge cases that are annoying.  It really needs to document it properly.  The commit message is more explicit than the manual.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/2943#pullrequestreview-781317499">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ75WWTZM3AG2SMSNV3UHDLORANCNFSM5GBHXS3Q">unsubscribe</a>.<br />Triage notifications on the go with GitHub Mobile for <a href="https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675">iOS</a> or <a href="https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub">Android</a>.
<img src="https://github.com/notifications/beacon/AAIOWJ57BJSV6ZAVPB2IYHDUHDLORA5CNFSM5GBHXS32YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOF2I7K6Y.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2943#pullrequestreview-781317499",
"url": "https://github.com/geany/geany/pull/2943#pullrequestreview-781317499",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>