<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=811085" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/elextr">@elextr</a> well, I rather see that as a plain bug, and I'm not sure we really want bug-for-bug compatibility… and here all that I can see as incompatible behavior is that now plugins would know that a file is <em>not</em> saved when quitting, which is basically giving it more, or at least correct, information.  I fail to see any case that was correct but would be compromised because of getting correct information on this; if anything it would introduce different behavior because the code <em>explicitly wanted to perform different actions depending on the saved state</em> but was lied to, and thus only likely to perform incorrectly in the first place.</p>
<p>And in any case, we broke ABI in this release (<a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/geany/geany/commit/3fa7576e13e129900a6e0acbd5460237f9a1b614/hovercard" href="https://github.com/geany/geany/commit/3fa7576e13e129900a6e0acbd5460237f9a1b614"><tt>3fa7576</tt></a>), would that be enough for you?</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/1857#issuecomment-443617229">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ4duy48Ku5Lm1SB8FEq2_xPMGZkMks5u1NaSgaJpZM4T82nj">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJ2rEiLIb-bEHycY29l_QzidEDaLMks5u1NaSgaJpZM4T82nj.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://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@b4n in #1857: @elextr well, I rather see that as a plain bug, and I'm not sure we really want bug-for-bug compatibility… and here all that I can see as incompatible behavior is that now plugins would know that a file is *not* saved when quitting, which is basically giving it more, or at least correct, information.  I fail to see any case that was correct but would be compromised because of getting correct information on this; if anything it would introduce different behavior because the code *explicitly wanted to perform different actions depending on the saved state* but was lied to, and thus only likely to perform incorrectly in the first place.\r\n\r\nAnd in any case, we broke ABI in this release (3fa7576e13e129900a6e0acbd5460237f9a1b614), would that be enough for you?"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1857#issuecomment-443617229"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/1857#issuecomment-443617229",
"url": "https://github.com/geany/geany/pull/1857#issuecomment-443617229",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [geany/geany] Fix 'changed' flag being altered when all documents are closing. (#1857)",
"sections": [
{
"text": "",
"activityTitle": "**Colomban Wendling**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@b4n",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"geany/geany\",\n\"issueId\": 1857,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close pull request",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"PullRequestClose\",\n\"repositoryFullName\": \"geany/geany\",\n\"pullRequestId\": 1857\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/geany/geany/pull/1857#issuecomment-443617229"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 334719459\n}"
}
],
"themeColor": "26292E"
}
]</script>