<p><a href="https://github.com/codebrainz" class="user-mention">@codebrainz</a></p>
<blockquote>
<p>This macro should probably be used in this PR, else navqueue won't work for files that aren't saved yet (or is this an existing bug?).</p>
</blockquote>
<p>This is existing behavior. The navqueue uses the filename as a unique identifier of a document. (It doesn’t really need them right now, because it evicts a document from the navqueue once it is closed — so could use a <code>GeanyDocument *</code> — but the comments there mention fixing that as a to-do.)</p>
<blockquote>
<p>If you're unlucky, it will do anything else. What happens is completely undefined behaviour.</p>
</blockquote>
<p>Fair enough. Coming from other languages I’m not really used to the concept of UB yet. (Does it <em>realistically</em> ever not crash though?..)</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/1537#issuecomment-315542239">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ1JXPflX4ceeswsOSnwi6TTRc9TVks5sON0OgaJpZM4OR0zM">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ7AOxwNkW-YG07Pxy8otokDEzTYvks5sON0OgaJpZM4OR0zM.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/1537#issuecomment-315542239"></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":"@vfaronov in #1537: @codebrainz \r\n\r\n\u003e This macro should probably be used in this PR, else navqueue won't work for files that aren't saved yet (or is this an existing bug?).\r\n\r\nThis is existing behavior. The navqueue uses the filename as a unique identifier of a document. (It doesn’t really need them right now, because it evicts a document from the navqueue once it is closed — so could use a `GeanyDocument *` — but the comments there mention fixing that as a to-do.)\r\n\r\n\u003e If you're unlucky, it will do anything else. What happens is completely undefined behaviour.\r\n\r\nFair enough. Coming from other languages I’m not really used to the concept of UB yet. (Does it *realistically* ever not crash though?..)"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1537#issuecomment-315542239"}}}</script>