Review Request: Reimplemented the Peer Interface, removed some unused code and added the status bar.

Thu Aug 16 12:40:57 UTC 2012

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


kwhiteboard/CMakeLists.txt
<http://git.reviewboard.kde.org/r/106037/#comment13714>

    Why is this still here?


kwhiteboard/kwhiteboard.h
<http://git.reviewboard.kde.org/r/106037/#comment13713>

    Add only the headers you need


kwhiteboard/kwhiteboard.h
<http://git.reviewboard.kde.org/r/106037/#comment13715>

    This is not (yet) the tabbed widget


kwhiteboard/kwhiteboard.h
<http://git.reviewboard.kde.org/r/106037/#comment13712>

    m_


kwhiteboard/kwhiteboard.cpp
<http://git.reviewboard.kde.org/r/106037/#comment13716>

    Why? I _DON'T WANT_ a Ping action


kwhiteboard/kwhiteboardui.rc
<http://git.reviewboard.kde.org/r/106037/#comment13717>

    NO!


kwhiteboard/kwhiteboardwidget.cpp
<http://git.reviewboard.kde.org/r/106037/#comment13718>

    Why are you changing this line?


kwhiteboard/org.freedesktop.DBus.Introspectable.xml
<http://git.reviewboard.kde.org/r/106037/#comment13719>

    Remove this file!


- Daniele Elmo Domenichelli


On Aug. 16, 2012, 12:27 p.m., Puneet Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106037/
> -----------------------------------------------------------
> 
> (Updated Aug. 16, 2012, 12:27 p.m.)
> 
> 
> Review request for Telepathy and Daniele Elmo Domenichelli.
> 
> 
> Description
> -------
> 
> Reimplemented the Peer Interface, removed some unused code and added the status bar.
> 
> 
> Diffs
> -----
> 
>   kwhiteboard/CMakeLists.txt 4a86efc94cd872224a147e24f005f2d3090d6f1a 
>   kwhiteboard/kwhiteboard-handler.cpp ade1218d950689e7aa809bac92850d6e01424e2e 
>   kwhiteboard/kwhiteboard.h 33361c620369eac011411211a2662e8a2855e448 
>   kwhiteboard/kwhiteboard.cpp c16fd04d185046cf6e413a7b59b958bf45748d5a 
>   kwhiteboard/kwhiteboardui.rc a35398d5fda9ce8a00b9cb3a119c724caa8b262e 
>   kwhiteboard/kwhiteboardwidget.cpp ce1b2a9bcb6b67a1f8ab4dd2d2184b9ac25e927d 
>   kwhiteboard/main.cpp 4371285a3c7fbfff0b53f84d9a909408c1037337 
>   kwhiteboard/org.freedesktop.DBus.Introspectable.xml PRE-CREATION 
>   kwhiteboard/org.kde.DBus.Peer.xml PRE-CREATION 
>   kwhiteboard/peer.h PRE-CREATION 
>   kwhiteboard/peer.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/106037/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Puneet Goyal
> 
>

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