On Wednesday 23 of October 2013 23:28:58 Darrell
Anderson wrote:
Um, yes, I think so. I presumed something similar
was needed but I
wanted one of the experts to say as much. :-)
* Are you going to push the patch?
* How do we test the binaries compiled correctly when
WITH_TDEICONLOADER_DEBUG=true? Backtraces?
* Is this an option normally left as false and changed to true only
when a developer asks for that change to help debug related
problems?
I prepared a patch right now - I myself have not tested it yet :)
However, yes, the idea is that by default will be WITH_TDEICONLOADER_DEBUG
turned OFF. And if anyone will wish to debug TDEIconLoader class, then need
to rebuild tdelibs with option WITH_TDEICONLOADER_DEBUG turned ON.
Slavek
I tested that option works as intended and
pushed a patch into GIT in hash 208d93bf.
Slavek