Bug 286683 - Make common-internals look like library code
Summary: Make common-internals look like library code
Status: RESOLVED FIXED
Alias: None
Product: telepathy
Classification: Frameworks and Libraries
Component: common-internals (show other bugs)
Version: unspecified
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: 0.4.0
Assignee: Dario Freddi
URL:
Keywords:
Depends on:
Blocks: 285413
  Show dependency treegraph
 
Reported: 2011-11-15 12:42 UTC by David Edmundson
Modified: 2012-07-06 12:47 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Edmundson 2011-11-15 12:42:41 UTC
This means every header file needs reviewing for:
 - sane, simple API
 - documentation
 - use of Q_SLOTS vs slots (etc.)
 - check for d pointers + other library like stuff
 - Put everything in KTelepathy namespace(?)

circular-countdown.h
error-dictionary.h
global-presence.h
kpresence.h
service-availability-checker.h
telepathy-handler-application.h
text-parser.h
wallet-interface.h

models/
accounts-filter-model.cpp
accounts-filter-model.h
accounts-model.cpp
accounts-model.h
accounts-model-item.cpp
accounts-model-item.h
contact-model-item.cpp
contact-model-item.h
groups-model.cpp
groups-model.h
groups-model-item.cpp
groups-model-item.h
proxy-tree-node.cpp
proxy-tree-node.h
tree-node.cpp
tree-node.h

Please comment if you've edited a file that you're happy for it to be reviewed - or that you think a file is ok.
Comment 1 Dario Freddi 2011-11-18 22:01:43 UTC
I'm taking care of this
Comment 2 Dario Freddi 2011-11-23 21:27:14 UTC
Oh, damn you bugzilla for not recognizing my "other" address. Whatevz, committed. Further reference here if needed: https://git.reviewboard.kde.org/r/103183/