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



<p>

Fix it, then Ship it!

</p>



 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Yes, I had that on my todo for a long time. Is your format <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">"Bytes:Bits"</code> common? I've never seen it, but I don't often have to play around with bits ;-)</p></pre>
 <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/127061/diff/1/?file=444167#file444167line977" style="color: black; font-weight: bold; text-decoration: underline;">languages/clang/duchain/builder.cpp</a>
    <span style="font-weight: normal;">

     (Diff revision 1)

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



 
 

 <tbody>

  <tr>
    <th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">977</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">                                    <span class="s">"aligned to: %<span class="hl">3</span> Bytes"</span><span class="p">,</span> <span class="n">offset</span><span class="p">,</span> <span class="n">sizeOf</span><span class="p">,</span> <span class="n">alignedTo</span><span class="p">).</span><span class="n">toUtf8</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">977</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">                                    <span class="s">"aligned to: %<span class="hl">4</span> Bytes"</span><span class="p">,</span> <span class="n">offset</span><span class="hl"> </span><span class="o"><span class="hl">/</span></span><span class="hl"> </span><span class="mi"><span class="hl">8</span></span><span class="p"><span class="hl">,</span></span><span class="hl"> </span><span class="p"><span class="hl">(</span></span><span class="n"><span class="hl">offset</span></span><span class="hl"> </span><span class="o"><span class="hl">%</span></span><span class="hl"> </span><span class="mi"><span class="hl">8</span></span><span class="p"><span class="hl">)</span></span><span class="hl"> </span><span class="o"><span class="hl">?</span></span><span class="hl"> </span><span class="n"><span class="hl">i18n</span></span><span class="p"><span class="hl">(</span></span><span class="s"><span class="hl">":%1"</span></span><span class="p"><span class="hl">,</span></span><span class="hl"> </span><span class="n"><span class="hl">offset</span></span><span class="hl"> </span><span class="o"><span class="hl">%</span></span><span class="hl"> </span><span class="mi"><span class="hl">8</span></span><span class="p"><span class="hl">)</span></span><span class="hl"> </span><span class="o"><span class="hl">:</span></span><span class="hl"> </span><span class="n"><span class="hl">QString</span></span><span class="p"><span class="hl">()</span>,</span> <span class="n">sizeOf</span><span class="p">,</span> <span class="n">alignedTo</span><span class="p">).</span><span class="n">toUtf8</span><span class="p">());</span></pre></td>
  </tr>

 </tbody>

</table>

 <div style="margin-left: 2em;">

  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I'd pull this out into its own line(s) for readability</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;"><div class="codehilite" style="background: #f8f8f8"><pre style="line-height: 125%"><span style="color: #008000; font-weight: bold">auto</span> <span style="color: #008000; font-weight: bold">offsetString</span> <span style="color: #666666">=</span> <span style="color: #008000; font-weight: bold">QString</span><span style="color: #666666">:</span><span style="color: #AA22FF">:number</span><span style="color: #666666">(</span><span style="color: #008000; font-weight: bold">offset</span> <span style="color: #666666">/</span> <span style="color: #008000; font-weight: bold">8</span><span style="color: #666666">);</span>
<span style="color: #008000; font-weight: bold">if</span> <span style="color: #666666">(</span><span style="color: #008000; font-weight: bold">auto</span> <span style="color: #008000; font-weight: bold">rest</span> <span style="color: #666666">=</span> <span style="color: #008000; font-weight: bold">offset</span> <span style="color: #666666">%</span> <span style="color: #008000; font-weight: bold">8</span><span style="color: #666666">)</span> {
    offsetString <span style="color: #666666">+=</span> QLatin1Char(<span style="color: #BA2121">':'</span>) <span style="color: #666666">+</span> QString<span style="color: #666666">::</span>number(rest);
}
</pre></div>
</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">and then just replace offset with offsetString in the old i18n call.</p></pre>
 </div>
</div>
<br />



<p>- Milian Wolff</p>


<br />
<p>On February 13th, 2016, 6:25 a.m. UTC, Alexandre Courbot wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
 <tr>
  <td>

<div>Review request for KDevelop.</div>
<div>By Alexandre Courbot.</div>


<p style="color: grey;"><i>Updated Feb. 13, 2016, 6:25 a.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
kdevelop
</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;">Having the offsets of members in a structure displayed is extremely
useful for people who work with data structures. However the unit
chosen (bits) is not really common. This patch changes it to bytes. If
an offset doesn't round to a byte position, the extra bits are also
displayed.</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>languages/clang/duchain/builder.cpp <span style="color: grey">(22d00f0fbbbcbce2eab92df2d9ed2b33e237a9e2)</span></li>

</ul>

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






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







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