<p>In <a href="https://github.com/geany/geany/pull/774#discussion_r45967569">src/encodings.c</a>:</p>
<pre style='color:#555'>>  
> -                  while (order < group_size)   /* the biggest group has 13 elements */
> +          /** TODO can it be optimized? ATM 882 runs at line "if (encoding->group ...)" */
</pre>
<p>Is this comment still relevant, and if so which "if" does it refer to?  AFAICT there is no <code>if(encoding->group</code> line in the following loop.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/geany/geany/pull/774/files#r45967569">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ0N0eQsHQ567_kJj03Z4yfetk2Ttks5pJuBXgaJpZM4Gpnxa.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/774/files#r45967569"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>