[mythtv] [mythtv-commits] mythtv commit: r14990 by danielk
Daniel Kristjansson
danielk at cuymedia.net
Thu Nov 29 22:50:05 UTC 2007
On Fri, 2007-11-30 at 09:12 +1100, Nigel Pearson wrote:
> > Author: danielk
> > New Revision: 14990
> > Some of the new autodiscovery created QString::null's which were
> > redirected to cout. On my system this causes cout to stop sending
> > data to the console.
> Bummer. Looks like I will have to start
> using "" instead of QString::null ?
That creates a lot of extra code and a malloc when you assign it to
a QString :|
> Note: I prefer the "Blah" + sBlah syntax,
> and have sprinkled them through the code
This is perfectly safe if you know sBlah is never going to be a null,
but if it is it will cause problems. (sBlah + sBlah2) might be safe, I
haven't tested it. But (sBlah) and (sBlah << sBlah2) can redirect a
null to cout. The worst that happens is that you don't get debugging
anymore; so it's not the end of the world but it's not so good when
you're trying to debug stuff..
-- Daniel
More information about the mythtv-dev
mailing list