<p></p>
<blockquote>
<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 with keyfile.c is fairly self-contained and in a good state.</p>
</blockquote>
<p>That's OK but I wouldn't really want to merge the one without the other (not into master, at least). Not sure if github can handle inter-dependent PRs. Maybe have to merge this to a staging branch so that your other PR can use that as a base.</p>
<blockquote>
<p>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>
</blockquote>
<p>Maybe. For the most part it's a matter of taste I guess. But, keep in mind that projects can be stored anywhere, e.g. also in a $HOME/projects that is just a pool of *.geany files with no relation to the actual project base directory. I might not want the session files in that pool directory. On the other hand, when it's in the project (to be checked into VCS), then you probably want it to be a dot-file so it's hidden (and add it to .gitignore), but then it's easy to forget to delete them. Many question marks around that, for me it would be simpler and natural to store machine-specific meta-stuff like this centralized location within $HOME/.config/geany. I would prefer a single key-file with the project name or some other uniq identifier as keys. Perhaps with a timestamp so we can do GC on the file.</p>
<p>But it's up to 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/2776#issuecomment-813689322">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ33GBWPB35OAZADWPTTHI3NPANCNFSM42KJV3OQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAIOWJ4XBMPVAZXECIM2C4DTHI3NPA5CNFSM42KJV3O2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGB76T2Q.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-813689322",
"url": "https://github.com/geany/geany/pull/2776#issuecomment-813689322",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>