search

graspit-simulator / graspit

The GraspIt! simulator
http://graspit-simulator.github.io/
Other
180 stars 81 forks source link

Fixing GRASPITDBG to properly enable/disable debug statements #39

Closed jvarley closed 8 years ago

jvarley commented 8 years ago

I don't believe this was working prior to CMakifying Graspit.

Changes: 1) added -DGRASPITDBG to CMakeLists.txt 2) removed define GRASPITDBG from main.cpp 3) fixed many of the debug statements which were trying to send QStrings to std::cout which is not possible.

Tested: Ran with the following: ccmake .. and set to Debug Built and ran, it output ALOT of debug noise, and is really slow but seems to work.

mateiciocarlie commented 8 years ago

I think this was always meant to work by manually defining the debug output flag in each file individually - with all of it on, I don't think the software is usable, even just for exercising a bug or something... Fixing it so that it actually compiles (i.e. QStrings to cout) is obviously needed though :)

Can you remove the definition of the debug output flag from the main CMakeLists?

jvarley commented 8 years ago

@mateiciocarlie Done. I removed the define from the CMakeLists.txt. Tested with debug build on 12.04. It no longer spews debug messages.