<blockquote>
<p>Looks unnecessarily complex.</p>
</blockquote>

<p>Believe it or not, the code is actually simplified since it removes an unused function and refactors an existing function to be in terms of another function instead of duplicating code. But yeah, the diff looks complex at a glance due to the way diffs work.</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/1225#issuecomment-246249313">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ4nn_AD5kkTmgNMbl-j0XhWY3F23ks5qpNzZgaJpZM4J6CEe">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJxPnjC5krzZ-AMgtVreaavrKcjyuks5qpNzZgaJpZM4J6CEe.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/1225#issuecomment-246249313"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@codebrainz in #1225: \u003e Looks unnecessarily complex.\r\n\r\nBelieve it or not, the code is actually simplified since it removes an unused function and refactors an existing function to be in terms of another function instead of duplicating code. But yeah, the diff looks complex at a glance due to the way diffs work."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1225#issuecomment-246249313"}}}</script>