Review Request: Further tidying up with the parameter model change

David Edmundson kde at davidedmundson.co.uk
Thu Jan 20 13:53:09 CET 2011



> On Jan. 20, 2011, 12:39 p.m., Dominik Schmidt wrote:
> > src/add-account-assistant.cpp, line 154
> > <http://git.reviewboard.kde.org/r/100427/diff/1/?file=7331#file7331line154>
> >
> >     This looks very weird, can you come up with any name which doesn't start with "set"?

I totally agree, though I can't think of what it should be right now.


- David


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


On Jan. 20, 2011, 12:24 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100427/
> -----------------------------------------------------------
> 
> (Updated Jan. 20, 2011, 12:24 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Summary
> -------
> 
> Add getters for set/unset parameters to the ParameterModel.
> Also deprecated the class "ProtocolParameterValue" as it was basically the same as Parameter-item.
>     
> This removes the risk of the parameter model and the list of parameter values getting out of sync, and makes the interface to the plugins a lot clearer wrt to saving.
> 
> 
> Diffs
> -----
> 
>   src/KCMTelepathyAccounts/CMakeLists.txt a74ae27c89e1ad9e0ad6948bf959f4c93bd1ab7c 
>   src/KCMTelepathyAccounts/abstract-account-parameters-widget.h 6357b4eaff3b5579b44d9040ffe33314bbf09d80 
>   src/KCMTelepathyAccounts/abstract-account-parameters-widget.cpp d0ad71a3660ed2514817a67e6fd2a283481b54d9 
>   src/KCMTelepathyAccounts/abstract-account-ui.h a90395ca4e429fa37ccf935f19fcd0063397d4e9 
>   src/KCMTelepathyAccounts/account-edit-widget.h 4c78338603dc0b88230b52190ca9d373dff2628d 
>   src/KCMTelepathyAccounts/account-edit-widget.cpp b554db71a17a4c9664d04f781561ce62cbc17562 
>   src/KCMTelepathyAccounts/generic-advanced-options-widget.h PRE-CREATION 
>   src/KCMTelepathyAccounts/generic-advanced-options-widget.cpp PRE-CREATION 
>   src/KCMTelepathyAccounts/parameter-edit-model.h 0c762ea5a30253cd1c4b772f3859b681f4fa8dd4 
>   src/KCMTelepathyAccounts/parameter-edit-model.cpp 0fe52a10798b03dff5491d78c1b16ad72e2f0533 
>   src/add-account-assistant.cpp 3933763957b1448aacf9e6c7efbc6b296a5e3664 
>   src/edit-account-dialog.cpp 73462f2e2a93e22e6d5d67297c575f4e10d44c5c 
> 
> Diff: http://git.reviewboard.kde.org/r/100427/diff
> 
> 
> Testing
> -------
> 
> Added and edited an account. Seems to still work.
> 
> 
> Thanks,
> 
> David
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110120/150e9014/attachment.htm 


More information about the KDE-Telepathy mailing list