Review Request: Renaming var name in settings/config/mobileproviders.cpp

Swami Dhyan Nataraj [Nikolay Shaplov] dhyan at nataraj.su
Sat Feb 11 07:02:07 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103921/
-----------------------------------------------------------

(Updated Feb. 11, 2012, 7:02 a.m.)


Review request for Network Management and Lamarque Vieira Souza.


Description
-------

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.


Diffs
-----

  settings/config/mobileproviders.cpp 0bee2b3eb75766d01f7235ccf0971d150a09bcff 

Diff: http://git.reviewboard.kde.org/r/103921/diff/diff


Testing
-------


Thanks,

Swami Dhyan Nataraj [Nikolay Shaplov]

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-networkmanager/attachments/20120211/9810b14f/attachment.html>


More information about the kde-networkmanager mailing list