On Wednesday 23 of October 2013 23:46:20 Sl�vek Banko wrote: > 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