Review Request: Forwarding headers for Kross

Ben Boeckel mathstuf at gmail.com
Sat May 2 20:33:37 BST 2009


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

Review request for kdelibs.


Summary
-------

Kross does not currently provide the forwarding headers (#include [Kross/ClassName] (angle brackets aren't being escaped?)). This patch adds headers for the public classes as well as the install command to the CMakeLists.txt.


Diffs
-----

  trunk/KDE/kdelibs/includ../../Kross/Script PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/ScriptingPlugin PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/WrapperInterface PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/Object PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/MetaTypeVoidStar PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/MetaTypeVariant PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/MetaTypeImpl PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/MetaTypeHandler PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/MetaType PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/Manager PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/MetaFunction PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/InterpreterInfo PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/Interpreter PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/ErrorInterface PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/ChildrenInterface PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/ActionCollectionView PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/ActionCollectionProxyModel PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/ActionCollectionModel PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/ActionCollectionEditor PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/ActionCollection PRE-CREATION 
  trunk/KDE/kdelibs/includ../../Kross/Action PRE-CREATION 
  trunk/KDE/kdelibs/includ../../CMakeLists.txt 962163 

Diff: http://reviewboard.kde.org/r/655/diff


Testing
-------

None right now, though the headers are trivial. I'm still working to get a setup for tracking trunk.


Thanks,

Ben





More information about the kde-core-devel mailing list