Review Request: Rename ktp-common-internals according to the new policy

Dario Freddi drf at kde.org
Wed Dec 21 18:01:49 UTC 2011



> On Dec. 21, 2011, 5:48 p.m., George Kiagiadakis wrote:
> > KTp/wallet-interface.cpp, line 27
> > <http://git.reviewboard.kde.org/r/103491/diff/1/?file=44182#file44182line27>
> >
> >     Now that's interesting... I guess we can't just change it, as users will lose their passwords... but still, it's ugly.

Given the state of the software, we can do that, but we have to do it now.


> On Dec. 21, 2011, 5:48 p.m., George Kiagiadakis wrote:
> > KTp/circular-countdown.h, line 25
> > <http://git.reviewboard.kde.org/r/103491/diff/1/?file=44166#file44166line25>
> >
> >     I think I've asked again, but I don't remember. Is there a reason this is ktp-export.h instead of the standard <libname>_export.h ? i.e. ktpcommoninternalsprivate_export.h

they are used in both models and commoninternals, and they're consistent with the namespace


- Dario


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


On Dec. 21, 2011, 5:32 p.m., Dario Freddi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103491/
> -----------------------------------------------------------
> 
> (Updated Dec. 21, 2011, 5:32 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> As per title.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 2476ba7 
>   KTelepathy/CMakeLists.txt 9a92e6d 
>   KTelepathy/Models/CMakeLists.txt 32da52e 
>   KTelepathy/Models/accounts-filter-model.h 9c799b1 
>   KTelepathy/Models/accounts-filter-model.cpp 859c673 
>   KTelepathy/Models/accounts-model-item.h 14692d7 
>   KTelepathy/Models/accounts-model-item.cpp a057714 
>   KTelepathy/Models/accounts-model.h 09c6c7f 
>   KTelepathy/Models/accounts-model.cpp c2a78c3 
>   KTelepathy/Models/contact-model-item.h b22d059 
>   KTelepathy/Models/contact-model-item.cpp 5014568 
>   KTelepathy/Models/groups-model-item.h c994f3d 
>   KTelepathy/Models/groups-model-item.cpp 9e8f07a 
>   KTelepathy/Models/groups-model.h e38cbf0 
>   KTelepathy/Models/groups-model.cpp f576d1c 
>   KTelepathy/Models/proxy-tree-node.h d6cfb36 
>   KTelepathy/Models/proxy-tree-node.cpp fb5674b 
>   KTelepathy/Models/tree-node.h 7155b01 
>   KTelepathy/Models/tree-node.cpp f9413ce 
>   KTelepathy/circular-countdown.h 10f1658 
>   KTelepathy/circular-countdown.cpp 4cbb896 
>   KTelepathy/error-dictionary.h 414fd97 
>   KTelepathy/error-dictionary.cpp d91bfe5 
>   KTelepathy/global-presence.h 194f8cc 
>   KTelepathy/global-presence.cpp 4fe4407 
>   KTelepathy/ktelepathy-export.h 22bb37e 
>   KTelepathy/presence.h 71e7c6e 
>   KTelepathy/presence.cpp d291a81 
>   KTelepathy/service-availability-checker.h c2d7b83 
>   KTelepathy/service-availability-checker.cpp e1f7e51 
>   KTelepathy/telepathy-handler-application.h 64ab876 
>   KTelepathy/telepathy-handler-application.cpp 2e45c0f 
>   KTelepathy/text-parser.h aae4672 
>   KTelepathy/text-parser.cpp dc4c57b 
>   KTelepathy/wallet-interface.h 9c4eb27 
>   KTelepathy/wallet-interface.cpp 6e271fe 
>   KTp/CMakeLists.txt PRE-CREATION 
>   KTp/Models/CMakeLists.txt PRE-CREATION 
>   KTp/Models/accounts-filter-model.h PRE-CREATION 
>   KTp/Models/accounts-filter-model.cpp PRE-CREATION 
>   KTp/Models/accounts-model-item.h PRE-CREATION 
>   KTp/Models/accounts-model-item.cpp PRE-CREATION 
>   KTp/Models/accounts-model.h PRE-CREATION 
>   KTp/Models/accounts-model.cpp PRE-CREATION 
>   KTp/Models/contact-model-item.h PRE-CREATION 
>   KTp/Models/contact-model-item.cpp PRE-CREATION 
>   KTp/Models/groups-model-item.h PRE-CREATION 
>   KTp/Models/groups-model-item.cpp PRE-CREATION 
>   KTp/Models/groups-model.h PRE-CREATION 
>   KTp/Models/groups-model.cpp PRE-CREATION 
>   KTp/Models/proxy-tree-node.h PRE-CREATION 
>   KTp/Models/proxy-tree-node.cpp PRE-CREATION 
>   KTp/Models/tree-node.h PRE-CREATION 
>   KTp/Models/tree-node.cpp PRE-CREATION 
>   KTp/circular-countdown.h PRE-CREATION 
>   KTp/circular-countdown.cpp PRE-CREATION 
>   KTp/error-dictionary.h PRE-CREATION 
>   KTp/error-dictionary.cpp PRE-CREATION 
>   KTp/global-presence.h PRE-CREATION 
>   KTp/global-presence.cpp PRE-CREATION 
>   KTp/ktp-export.h PRE-CREATION 
>   KTp/presence.h PRE-CREATION 
>   KTp/presence.cpp PRE-CREATION 
>   KTp/service-availability-checker.h PRE-CREATION 
>   KTp/service-availability-checker.cpp PRE-CREATION 
>   KTp/telepathy-handler-application.h PRE-CREATION 
>   KTp/telepathy-handler-application.cpp PRE-CREATION 
>   KTp/text-parser.h PRE-CREATION 
>   KTp/text-parser.cpp PRE-CREATION 
>   KTp/wallet-interface.h PRE-CREATION 
>   KTp/wallet-interface.cpp PRE-CREATION 
>   Messages.sh 615a372 
> 
> Diff: http://git.reviewboard.kde.org/r/103491/diff/diff
> 
> 
> Testing
> -------
> 
> Compiles
> 
> 
> Thanks,
> 
> Dario Freddi
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20111221/cedb46c0/attachment.html>


More information about the KDE-Telepathy mailing list