<div><div dir="auto">{}; means “default value of the return type”, so I’m retirning a QUrl() where it’s supposed to be a QUrl, and a QString where it’s supposed to be a string.</div></div><div dir="auto"><br></div><div dir="auto">This is more future proof as if we change the type of the function we don’t need to change that return.</div><div dir="auto"><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Em qua, 10 de abr de 2019 às 19:21, Nathaniel Graham <<a href="mailto:noreply@phabricator.kde.org">noreply@phabricator.kde.org</a>> escreveu:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><table><tbody><tr><td>ngraham added inline comments.
</td><td><a style="text-decoration:none;padding:4px 8px;margin:0 8px 8px;float:right;color:#464c5c;font-weight:bold;border-radius:3px;background-color:#f7f7f9;background-image:linear-gradient(to bottom,#fff,#f1f0f1);display:inline-block;border:1px solid rgba(71,87,120,.2)" href="https://phabricator.kde.org/D20438" target="_blank">View Revision</a></td></tr></tbody></table><br><div><strong>INLINE COMMENTS</strong><div><div style="margin:6px 0 12px 0"><div style="border:1px solid #c7ccd9;border-radius:3px"><div style="padding:0;background:#f7f7f7;border-color:#e3e4e8;border-style:solid;border-width:0 0 1px 0;margin:0"><div style="color:#74777d;background:#eff2f4;padding:6px 8px;overflow:hidden"><a style="float:right;text-decoration:none" href="https://phabricator.kde.org/D20438#inline-114927" target="_blank">View Inline</a><span style="color:#4b4d51;font-weight:bold">BookmarkHandler.cpp:105</span></div>
<div style="font:11px/15px "Menlo","Consolas","Monaco",monospace;white-space:pre-wrap;clear:both;padding:4px 0;margin:0"><div style="padding:0 8px;margin:0 4px">    <span class="m_8834534437185089022p">}</span>
</div><div style="padding:0 8px;margin:0 4px;background:rgba(151,234,151,.6)">    <span style="color:#aa4000">return</span> <span class="m_8834534437185089022p">{};</span>
</div><div style="padding:0 8px;margin:0 4px"><span class="m_8834534437185089022p">}</span>
</div></div></div>
<div style="margin:8px 0;padding:0 12px"><p style="padding:0;margin:8px">Why not an empty <tt style="background:#ebebeb;font-size:13px">QUrl()</tt>?</p></div></div><br><div style="border:1px solid #c7ccd9;border-radius:3px"><div style="padding:0;background:#f7f7f7;border-color:#e3e4e8;border-style:solid;border-width:0 0 1px 0;margin:0"><div style="color:#74777d;background:#eff2f4;padding:6px 8px;overflow:hidden"><a style="float:right;text-decoration:none" href="https://phabricator.kde.org/D20438#inline-114928" target="_blank">View Inline</a><span style="color:#4b4d51;font-weight:bold">BookmarkHandler.cpp:145</span></div>
<div style="font:11px/15px "Menlo","Consolas","Monaco",monospace;white-space:pre-wrap;clear:both;padding:4px 0;margin:0"><div style="padding:0 8px;margin:0 4px">    <span class="m_8834534437185089022p">}</span>
</div><div style="padding:0 8px;margin:0 4px;background:rgba(151,234,151,.6)">    <span style="color:#aa4000">return</span> <span class="m_8834534437185089022p">{};</span>
</div><div style="padding:0 8px;margin:0 4px"><span class="m_8834534437185089022p">}</span>
</div></div></div>
<div style="margin:8px 0;padding:0 12px"><p style="padding:0;margin:8px">Why not an empty <tt style="background:#ebebeb;font-size:13px">QString()</tt>?</p></div></div></div></div></div><br><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20438" target="_blank">https://phabricator.kde.org/D20438</a></div></div><br><div><strong>To: </strong>tcanabrava, Konsole, hindenburg<br><strong>Cc: </strong>ngraham, konsole-devel, gennad, thsurrel, maximilianocuria, hindenburg<br></div></blockquote></div></div>