<p><a href="https://github.com/codebrainz" class="user-mention">@codebrainz</a> the file_name can be null for unnamed docs, or am I wrong ?</p>
<p>An assert carries the intent to say "if this is null, then there is a programming error elsewhere" while an if (null) says null is expectable (which isn't the case here).</p>
<p>For API functions we usually use g_return_if_fail which logs an error does not abort the program, to save the user from poorly written plugins. But the function isnt part of the API so IMO an assert is perfectly appropriate.</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-315511861">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ38ppTIrl3uzkbYbO-qE3NjYQXaLks5sOFIhgaJpZM4OR0zM">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJx4AsS0zqPojVTyxiAne_chDLjxIks5sOFIhgaJpZM4OR0zM.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-315511861"></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":"@kugel- in #1537: @codebrainz the file_name can be null for unnamed docs, or am I wrong ?\r\n\r\nAn assert carries the intent to say \"if this is null, then there is a programming error elsewhere\" while an if (null) says null is expectable (which isn't the case here).\r\n\r\nFor API functions we usually use g_return_if_fail which logs an error does not abort the program, to save the user from poorly written plugins. But the function isnt part of the API so IMO an assert is perfectly appropriate."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1537#issuecomment-315511861"}}}</script>