<p></p>
<p>I agree that separating the project file a natural next step, but I would have to spend some time getting familiar with <code>project.c</code>. I'm back at work tomorrow after the long Easter weekend, so it might take a while to come up with a proof of concept. I would lean towards keeping the project session files next to the project files, but I don't think this has a huge impact on the implementation (just change how the path to the file is built, right?).</p>
<p>If it's OK with you, I would like to move that to a separate issue and PR. I think what we have here my changes to <code>keyfile.c</code> is fairly self-contained and in a good state.</p>
<p>I am removing "WIP:" and I believe there is one outstanding point to be addressed: should geany clear the old session data from <code>geany.conf</code> + introduce a preference to enable this new behaviour, or should it just leave it? <code>dotfiles</code> users are presumably already used to dealing with exactly this themselves. A "clear history" button (as suggested in <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="295502941" data-permission-text="Title is private" data-url="https://github.com/geany/geany/issues/1763" data-hovercard-type="issue" data-hovercard-url="/geany/geany/issues/1763/hovercard?comment_id=364176848&comment_type=issue_comment" href="https://github.com/geany/geany/issues/1763#issuecomment-364176848">#1763 (comment)</a>) which checks <code>geany.conf</code> and (deletes?) <code>session.conf</code> would take care of this, but I see that as a separate feature for another PR.</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/2776#issuecomment-813650659">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJZQEEJN6RJGM7B2XP3THIRCNANCNFSM42KJV3OQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAIOWJYPGHRQR6SQD6CVO6LTHIRCNA5CNFSM42KJV3O2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGB7VFYY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2776#issuecomment-813650659",
"url": "https://github.com/geany/geany/pull/2776#issuecomment-813650659",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>