<p><a href="https://github.com/kugel-" class="user-mention">@kugel-</a> to be clear, nothing I said is a general criticism of the purpose your PR or the implementation (apart from the specific comments you have already fixed).</p>
<p>I just found it difficult to understand it, not aided by lower case macros that "capture" following syntax.  Also as you pointed out above, not only is it lower case, it is an unprefixed Geany local macro.  So I was not confident I understood the implementation, and couldn't justify spending more time on it.  Thats why I approved my review, so someone else could review it and not need to override my review.</p>
<p>Yes I believe macros like foreach_strv and foreach_document etc should be removed, but thats not part of this PR, will open another issue for discussions about that.</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/1445#issuecomment-289007035">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ9DyH3soYhOV7Wf-QRpk6QDCJNLwks5ro7MpgaJpZM4MnYrc">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ_aJpYLuJjPgzCpY4p3m6N0dVlCJks5ro7MpgaJpZM4MnYrc.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/1445#issuecomment-289007035"></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":"@elextr in #1445: @kugel- to be clear, nothing I said is a general criticism of the purpose your PR or the implementation (apart from the specific comments you have already fixed).  \r\n\r\nI just found it difficult to understand it, not aided by lower case macros that \"capture\" following syntax.  Also as you pointed out above, not only is it lower case, it is an unprefixed Geany local macro.  So I was not confident I understood the implementation, and couldn't justify spending more time on it.  Thats why I approved my review, so someone else could review it and not need to override my review.\r\n\r\nYes I believe macros like foreach_strv and foreach_document etc should be removed, but thats not part of this PR, will open another issue for discussions about that."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1445#issuecomment-289007035"}}}</script>