<br><br><div class="gmail_quote">On Wed, May 26, 2010 at 3:13 PM, Sebastian Trüg <span dir="ltr">&lt;<a href="mailto:trueg@kde.org">trueg@kde.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On 05/26/2010 10:08 AM, Vishesh Handa wrote:<br>
&gt;     &gt; 3. When checking if the the m_kickoffUri is a nie:url for a resource.<br>
&gt;     &gt; The nieUrl is assigned to be equal to the uri. This however gets fixed<br>
&gt;     &gt; in the load(). And nieUrl isn&#39;t used anywhere, so it doesn&#39;t<br>
&gt;     matter that<br>
&gt;     &gt; much.<br>
&gt;<br>
&gt;     I suppose you mean this part:<br>
&gt;<br>
&gt;     if( it.next() ) {<br>
&gt;          QUrl uri = it[&quot;r&quot;].uri();<br>
&gt;          if( uri.isEmpty() ) {<br>
&gt;             m_uri = m_kickoffUri;<br>
&gt;          }<br>
&gt;          else {<br>
&gt;             m_uri = uri;<br>
&gt;             m_nieUrl = uri;<br>
&gt;          }<br>
&gt;<br>
&gt;     The last two lines. AFAICT this is perfectly fine since in the latter<br>
&gt;     case both nie:url and resource URI are equal.<br>
&gt;<br>
&gt;<br>
&gt; I respectfully disagree. :)<br>
&gt;<br>
&gt; The query used is this &quot;select distinct ?r ?o where { { ?r nie:url &lt;uri&gt;<br>
&gt; . } UNION { &lt;uri&gt; ?p ?o . }  } LIMIT 1&quot;. The case where ?r isn&#39;t empty<br>
&gt; is when the &lt;uri&gt; contains the nie:url and therefore ?r will contain the<br>
&gt; resource uri.<br>
<br>
</div>You are of course correct. It should be &quot;m_nieUrl = m_kickoffUri&quot;<br>
instead. Agreed?<br>
<br></blockquote><div><br>Yup. :)<br><br>If you get the time could you please look at my horrible merge patch?<br><br>Thanks<br>- Vishesh Handa <br><br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

Cheers,<br>
<font color="#888888">Sebastian<br>
</font></blockquote></div><br>