<p>No, you should always know what you are setting, randomly overwriting a command that might have been set by a user, or the filetype, or the project is wrong.</p>

<p>You want to get the current command to see what will be executed.  </p>

<p>Perhaps it would have been better to return the source instead of the command, but thats 20/20 hindsight.<br><br>
But it would be a better change.  The plugin can still run rampant, just pass it to the existing setter.  But at least the plugin then has the chance to know what its doing.</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-246256957">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ2EDz5JoXyWWGFi9Txuhtl7xnzZaks5qpOtbgaJpZM4J6CEe">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ9dH_t1WWKQCZ2Zr5dTsAg4bApIQks5qpOtbgaJpZM4J6CEe.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-246256957"></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 #1225: No, you should always know what you are setting, randomly overwriting a command that might have been set by a user, or the filetype, or the project is wrong.\r\n\r\nYou want to get the current command to see what will be executed.  \r\n\r\nPerhaps it would have been better to return the source instead of the command, but thats 20/20 hindsight.  \r\nBut it would be a better change.  The plugin can still run rampant, just pass it to the existing setter.  But at least the plugin then has the chance to know what its doing."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1225#issuecomment-246256957"}}}</script>