<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/115705/">https://git.reviewboard.kde.org/r/115705/</a>
     </td>
    </tr>
   </table>
   <br />











<div>




<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
 <thead>
  <tr>
   <th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
    <a href="https://git.reviewboard.kde.org/r/115705/diff/4/?file=265030#file265030line33" style="color: black; font-weight: bold; text-decoration: underline;">kstars/tools/whatsinteresting/skyobjdescription.cpp</a>
    <span style="font-weight: normal;">

     (Diff revision 4)

    </span>
   </th>
  </tr>
 </thead>

 <tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
  <tr>

   <td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">SkyObjDescription::~SkyObjDescription()</pre></td>

  </tr>
 </tbody>



 
 

 <tbody>

  <tr>
    <th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
    <th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">33</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">const</span> <span class="n">QString</span> <span class="nf">descOpeing</span><span class="p">(</span> <span class="s">"<Description xml:space=</span><span class="se">\"</span><span class="s">preserve</span><span class="se">\"</span><span class="s">>"</span> <span class="p">);</span></pre></td>
  </tr>

 </tbody>

</table>

<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">descOpening, I suppose.</pre>
</div>
<br />

<div>




<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
 <thead>
  <tr>
   <th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
    <a href="https://git.reviewboard.kde.org/r/115705/diff/4/?file=265030#file265030line48" style="color: black; font-weight: bold; text-decoration: underline;">kstars/tools/whatsinteresting/skyobjdescription.cpp</a>
    <span style="font-weight: normal;">

     (Diff revision 4)

    </span>
   </th>
  </tr>
 </thead>

 <tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
  <tr>

   <td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">SkyObjDescription::~SkyObjDescription()</pre></td>

  </tr>
 </tbody>



 
 

 <tbody>

  <tr>
    <th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">34</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="kt">int</span> <span class="n">startIndex</span> <span class="o">=</span> <span class="n">data</span><span class="p">.</span><span class="n">lastIndexOf</span><span class="p">(</span><span class="s"><span class="hl">"<Description xml:space=</span></span><span class="se"><span class="hl">\"</span></span><span class="s"><span class="hl">preserve</span></span><span class="se"><span class="hl">\"</span></span><span class="s"><span class="hl">>"</span></span><span class="p">)</span> <span class="o">+</span> <span class="mi"><span class="hl">34</span></span><span class="p">;</span></pre></td>
    <th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">48</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="kt">int</span> <span class="n">startIndex</span> <span class="o">=</span> <span class="n">data</span><span class="p">.</span><span class="n">lastIndexOf</span><span class="p">(</span><span class="hl"> </span><span class="n"><span class="hl">urlOpening</span></span><span class="hl"> </span><span class="p">)</span> <span class="o">+</span> <span class="n"><span class="hl">urlOpening</span></span><span class="p"><span class="hl">.</span></span><span class="n"><span class="hl">length</span></span><span class="p"><span class="hl">()</span>;</span></pre></td>
  </tr>

 </tbody>

</table>

<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Inconsistent formatting.</pre>
</div>
<br />



<p>- Rafal Kulaga</p>


<br />
<p>On April 12th, 2014, 10:11 p.m. UTC, Vijay Dhameliya wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for KStars and Rafal Kulaga.</div>
<div>By Vijay Dhameliya.</div>


<p style="color: grey;"><i>Updated April 12, 2014, 10:11 p.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
kstars
</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The Wikipedia link for skyobject will be retrieved from data received by Wikipedia API and it is being passed to QML so that when user clicks on source in "What's Interesting..." UI it will be open in browser.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Tested</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>kstars/tools/whatsinteresting/qml/wiview.qml <span style="color: grey">(47a15ff)</span></li>

 <li>kstars/tools/whatsinteresting/skyobjdescription.h <span style="color: grey">(d909fa1)</span></li>

 <li>kstars/tools/whatsinteresting/skyobjdescription.cpp <span style="color: grey">(2b83155)</span></li>

 <li>kstars/tools/whatsinteresting/skyobjitem.cpp <span style="color: grey">(a397c1a)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/115705/diff/" style="margin-left: 3em;">View Diff</a></p>







  </td>
 </tr>
</table>








  </div>
 </body>
</html>