Mesut,

Some thought about your patch:
First of all it looks extensive as if it could work.  But I couldn't
compile it because of many unused-parameter errors.  There are
differences between the gcc versions.  If you use a more current one
you should see these errors too.

Some smaller things:
- The year in the copyright section of new files should be the current one only.
- I think the section with the Qt message in src/libdm/CMakeLists.txt
is something what should be removed.
- You should maintain the Makefiles.am too.
- Has the QT4_INSTALLED flag in the main CMakeLists.txt something to
do with the FIND_PACKAGE(Qt4)?
- You shouldn't deactivate Tk or X11.

> I have a small question about Tk and TCL. If we use Qt, we dont need them
> for the qt-dm?

I think we will still need them.  TCL is BRL-CAD's command language.
I.e. the Qt display manager has to work together with TCL.

> At this week I will port the dm-qt.c file to cpp. After that I will
> implement the first logic. Begining with opening a window and closing it.
> After that I will look at the eventprocessing. Have you some advices?

Try renaming dm-qt.c to dm-qt.cpp and using __BEGIN_DECLS and
__END_DECLS in the corresponding header file(s).


Daniel

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
BRL-CAD Developer mailing list
brlcad-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/brlcad-devel

Reply via email to