<p>Oh my, all those people who have two Geanys using the same config are gonna have many more races <g-emoji class="g-emoji" alias="grin" fallback-src="https://github.githubassets.com/images/icons/emoji/unicode/1f601.png">😁</g-emoji></p>
<p>And an occasional failure when both try to write at the same time (on those file systems where such access is exclusive).</p>
<blockquote>
<p>The only thing I don't like, which could be improved later, is that it's still pretty arbitrary where the settings are being saved.</p>
</blockquote>
<p>If by "where" you mean "when" then I thought settings are now pretty much always saved when they are changed, its only the session that saved at the end, and thats what this addresses.  Of course because of the way g_key_files work the whole lot has to be saved.</p>
<p>But as <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=181177" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/codebrainz">@codebrainz</a> points out there are potential problems with slow or flakey drives maybe there needs to be a setting to disable it instead of forcing it on everyone.</p>
<p>Of course since it only writes to <code>geany_debug()</code> on error nobody will notice its failing I guess.</p>
<p>Agree best to commit immediately after next release.</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/2114#issuecomment-477818868">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJysJR2udU-5gbGnrp8l0gDAA8Nszks5vbV19gaJpZM4cQ_4i">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJ_cx5EI7aZW4CP_KkUMBU5weWZfVks5vbV19gaJpZM4cQ_4i.gif" height="1" width="1" alt="" /></p>
<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://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@elextr in #2114: Oh my, all those people who have two Geanys using the same config are gonna have many more races :grin:\r\n\r\nAnd an occasional failure when both try to write at the same time (on those file systems where such access is exclusive).\r\n\r\n\u003e The only thing I don't like, which could be improved later, is that it's still pretty arbitrary where the settings are being saved.\r\n\r\nIf by \"where\" you mean \"when\" then I thought settings are now pretty much always saved when they are changed, its only the session that saved at the end, and thats what this addresses.  Of course because of the way g_key_files work the whole lot has to be saved.\r\n\r\nBut as @codebrainz points out there are potential problems with slow or flakey drives maybe there needs to be a setting to disable it instead of forcing it on everyone.\r\n\r\nOf course since it only writes to `geany_debug()` on error nobody will notice its failing I guess.\r\n\r\nAgree best to commit immediately after next release.\r\n\r\n"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/2114#issuecomment-477818868"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2114#issuecomment-477818868",
"url": "https://github.com/geany/geany/pull/2114#issuecomment-477818868",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>