[Kde-bindings] kdejava/koala compile errors

[Richard Dale]

On Tuesday 11 May 2004 08:42, Peter Hyman wrote:
> The file qpopupmenu.h MUST be included explicitly in file
> kdebindings/kdejava/koala/kdejava/KDirSelectDialog.cpp before the header
> file kdirselectdialog.h is #included.
>
> This is consistent with the manner in which it is handled in
> kdelibs/kio/kfile/kdirselectdialog.cpp.
>
> IMO it really should be #included within the header kdirselectdialog.h,
> but it's not. I'll pass that along to the kdelibs folks as a possible
> bug.
>
> Adding the qpopupmenu.h file allows compile to conclude.
I've just updated my kdelibs from the cvs HEAD, and I'm still building it. But 
I tried the kdirselectdialog.h header from the HEAD and generated the 
attached KDirSelectDialog.cpp source, and it compiles fine for me. An include 
for QPopupMenu isn't needed because it's only a private variable.

class QPopupMenu;
...
private:
    void readConfig( KConfig *config, const QString& group );
    void saveConfig( KConfig *config, const QString& group );
    void openNextDir( KFileTreeViewItem *parent );
    KFileTreeBranch * createBranch( const KURL& url );

    KFileTreeView *m_treeView;
    QPopupMenu *m_contextMenu;
...

g++ -DHAVE_CONFIG_H -I. -I. -I../../.. -I./../../../qtjava/javalib -I./.. 
-I/usr/local/j2sdk1.3.1/include/ -I/usr/local/j2sdk1.3.1/include//linux 
-I/opt/kde3/include -I/home/duke/src/kde/HEAD/qt-copy/include 
-I/usr/X11R6/include -DQT_THREAD_SUPPORT -D_REENTRANT -Wnon-virtual-dtor 
-Wno-long-long -Wundef -Wall -W -Wpointer-arith -Wwrite-strings -ansi 
-D_XOPEN_SOURCE=500 -D_BSD_SOURCE -Wcast-align -Wconversion -Wchar-subscripts 
-O2 -Wformat-security -Wmissing-format-attribute -fno-exceptions 
-fno-check-new -fno-common -c -o KDirSelectDialog.o `test -f 
'KDirSelectDialog.cpp' || echo './'`KDirSelectDialog.cpp 
*** success *** 

As KDE 3.3 alpha is only a week to two away (sooner than I thought), perhaps 
it would be a good idea to regenerate the java bindings in the HEAD anyway. 
I'm intending to remove the sources from the cvs, as they're autogenerated, 
and generate them as part of kdebindings config if there is time.

You can edit kdebindings/smoke/kde/generate.pl.in and change '-fsmoke' to 
'-fjni'. Then configure kdebindings with '-with-smoke=kde' option, and it 
will generate the .cpp and .java sources in smoke/kde. The headers are 
generated from the compiled .class files via javah.

-- Richard
-------------- next part --------------
A non-text attachment was scrubbed...
Name: KDirSelectDialog.cpp
Type: text/x-c++src
Size: 20225 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-bindings/attachments/20040511/342e8cb2/attachment.cpp>