<blockquote>
<p>This provides 'save before run' feature for the edited file if the context string contains %f, %d etc. placeholders.</p>
</blockquote>
<p>Thats sensible for %f, the command likely wants to access it, but it need a confirmatory dialog, not to be forced, some users don't want to accidentally write over their files and not all commands will read the file just because it gets the name.  Thats why builds only save the current file and even thats got an option to disable it.  Not so sure its at all sensible for %d or any other substitutions, they are probably not accessing the file.</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/1841#issuecomment-384509625">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ8d6_CPxH_LrniDtxcHniSdLdSDIks5tsU0_gaJpZM4Thz7Y">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJ_nuBdNCJfFyY9CL0mDAtla2MqRXks5tsU0_gaJpZM4Thz7Y.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/pull/1841#issuecomment-384509625"></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 #1841: \u003e This provides 'save before run' feature for the edited file if the context string contains %f, %d etc. placeholders.\r\n\r\nThats sensible for %f, the command likely wants to access it, but it need a confirmatory dialog, not to be forced, some users don't want to accidentally write over their files and not all commands will read the file just because it gets the name.  Thats why builds only save the current file and even thats got an option to disable it.  Not so sure its at all sensible for %d or any other substitutions, they are probably not accessing the file."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1841#issuecomment-384509625"}}}</script>