<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="http://git.reviewboard.kde.org/r/103921/">http://git.reviewboard.kde.org/r/103921/</a>
     </td>
    </tr>
   </table>
   <br />





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 79bd5d65b76a82583e75407db99499ef273f5224 by Lamarque V. Souza to branch nm09.</pre>
 <br />







<p>- Commit</p>


<br />
<p>On February 11th, 2012, 7:02 a.m., Swami Dhyan Nataraj [Nikolay Shaplov] wrote:</p>






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

<div>Review request for Network Management and Lamarque Vieira Souza.</div>
<div>By Swami Dhyan Nataraj [Nikolay Shaplov].</div>


<p style="color: grey;"><i>Updated Feb. 11, 2012, 7:02 a.m.</i></p>






<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;">In this part of code we do localize plan names, not providers. This patch changes name of the list. It also would be good to rename getProviderNameByLocale function, because it is used both for plans and providers, but I am not quite familiar with naming tradition to choose the name properly.</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>settings/config/mobileproviders.cpp <span style="color: grey">(0bee2b3eb75766d01f7235ccf0971d150a09bcff)</span></li>

</ul>

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




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








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