[Konsole-devel] Review Request: update the outdated documention and sample code of kde_terminal_interface

Jekyll Wu adaptee at gmail.com
Tue Jun 5 23:19:30 UTC 2012


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

Review request for Konsole.


Description
-------

main change:

1. update the outdated inline document for kde_terminal_interface(which contains 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/konsole-devel/attachments/20120605/98b56b55/attachment.html>


More information about the konsole-devel mailing list