Review Request: update the outdated documention and sample code of kde_terminal_interface

Jekyll Wu adaptee at gmail.com
Thu Jun 7 15:44:42 BST 2012


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

(Updated June 7, 2012, 2:44 p.m.)


Review request for kdelibs and Konsole.


Changes
-------

Add kdelibs into groups


Description (updated)
-------

main change:

1. update the outdated inline document for kde_terminal_interface(which contained some signals that does not exist ever since KDE 4.0)

2. update the example code to make it really work and avoid using hardcoded library name 'libkonsolepart'.

3. update test/CMakeList.txt to make it stand-alone. I don't think that 'test' subfolder is ever used for building test case. Maybe it is more accurate to rename that folder to 'example'. But I could be wrong.


This addresses bug 257350.
    http://bugs.kde.org/show_bug.cgi?id=257350


Diffs
-----

  interfaces/terminal/kde_terminal_interface.h 649d674 
  interfaces/terminal/test/CMakeLists.txt a0fa93a 
  interfaces/terminal/test/main.h 8a3197a 
  interfaces/terminal/test/main.cc 132cee1 

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


Testing
-------

The sample code works fine with kdelibs/4.8 and konsole built from master.


Thanks,

Jekyll Wu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120607/f4fc5e96/attachment.htm>
-------------- next part --------------
_______________________________________________
konsole-devel mailing list
konsole-devel at kde.org
https://mail.kde.org/mailman/listinfo/konsole-devel


More information about the kde-core-devel mailing list