[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