<blockquote>
<p>Also is the previous API deprecated or something? The patch is kind of "why not" but I slightly fail to see "why yes" reasons.</p>
</blockquote>
<p>Please also see the discussion in <a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="299942215" data-permission-text="Issue title is private" data-url="https://github.com/geany/geany-plugins/issues/703" href="https://github.com/geany/geany-plugins/pull/703">#703</a>. It's purely for consistency/cleanup. I wonder if there is a new API why not move to it. Otherwise the old one will/can never be deprecated in the future. Anyway - as written in the discussion I would of course accept if you do not like to merge it.</p>
<blockquote>
<p>Maybe one question - what Geany API does this change require? Is the currently used API version specified in the plugin enough?</p>
</blockquote>
<p>According to <a href="https://www.geany.org/manual/reference/legacy.html" rel="nofollow">https://www.geany.org/manual/reference/legacy.html</a> it's deprecated since Geany 1.26, don't know the API version. But it is NOT obsolete and there is NO plan to remove it. So it will just work fine.</p>
<p>So as writen above I will accept a "no" without any discussion and the change is only because I think it's nicer to not use a deprecated API even if support for it will be ongoing.</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-plugins/pull/708#issuecomment-382040751">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJwHly7GvEOB0jnRtDOZJrTm7nfJdks5tpg3AgaJpZM4SR8uF">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJyibrLLbWcG-2KduKusBe1fc_Myrks5tpg3AgaJpZM4SR8uF.gif" height="1" width="1" alt="" /></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-plugins/pull/708#issuecomment-382040751"></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-plugins","title":"geany/geany-plugins","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-plugins"}},"updates":{"snippets":[{"icon":"PERSON","message":"@LarsGit223 in #708: \u003e Also is the previous API deprecated or something? The patch is kind of \"why not\" but I slightly fail to see \"why yes\" reasons.\r\n\r\nPlease also see the discussion in #703. It's purely for consistency/cleanup. I wonder if there is a new API why not move to it. Otherwise the old one will/can never be deprecated in the future. Anyway - as written in the discussion I would of course accept if you do not like to merge it.\r\n\r\n\u003e Maybe one question - what Geany API does this change require? Is the currently used API version specified in the plugin enough?\r\n\r\nAccording to https://www.geany.org/manual/reference/legacy.html it's deprecated since Geany 1.26, don't know the API version. But it is NOT obsolete and there is NO plan to remove it. So it will just work fine.\r\n\r\nSo as writen above I will accept a \"no\" without any discussion and the change is only because I think it's nicer to not use a deprecated API even if support for it will be ongoing."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany-plugins/pull/708#issuecomment-382040751"}}}</script>