<blockquote>
<p>Sigh, this setting is an example where the settings and the session get mixed, this looks right for the setting, but now the session status won't be saved at all.</p>

<p>That is a change from current behaviour. </p>
</blockquote>

<p>Did you test it? It should not change any behaviour with respect to the "session" (project). If it does in practice, then it's an unintended regression. The desired behaviour is identical to current, except that new instances will pickup the change now that existing instances don't wait until exit to save their config.</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/1257#issuecomment-251881028">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJwAmbAUGBli304n2BWhXEE7GhvVHks5qxJoLgaJpZM4KPeqz">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ4JoCDWloNyPbapKoYZEnt5L3ocrks5qxJoLgaJpZM4KPeqz.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/1257#issuecomment-251881028"></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 #1257: \u003e Sigh, this setting is an example where the settings and the session get mixed, this looks right for the setting, but now the session status won't be saved at all.\r\n\u003e\r\n\u003e That is a change from current behaviour. \r\n\r\nDid you test it? It should not change any behaviour with respect to the \"session\" (project). If it does in practice, then it's an unintended regression. The desired behaviour is identical to current, except that new instances will pickup the change now that existing instances don't wait until exit to save their config."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1257#issuecomment-251881028"}}}</script>