[mythtv] Re: [PATCH] mythmusic amd64 support yet again

[Daniel Paessler]

sounds good, i will try this out.
the only problem, that remains (for me), is the segfault when selecting 
"video list" in mythvideo. is someone working on this? or am i the only
one with this problem?
if someone could tell me, how to produce a backtrace, i would be glad to
help.

thanks,

daniel

Am Freitag, den 21.01.2005, 02:59 -0500 schrieb Isaac Richards:
> > On Wed, 12 Jan 2005 00:01:38 -0500, Kyle Schlansker <kylesch at gmail.com> 
> wrote:
> > > The attached patch is yet another rework to add x86_64 support for
> > > mythmusic.  Basically this is just an upgrade of goom and a few
> > > changes to synaesthesia.
> 
> I've applied this.
> 
> > > Ive fixed a memory leak in goom that was re-introduced in the last
> > > version of the patch and also removed some unnecessary comments in
> > > synaesthesia.cpp as requested by Isaac.
> 
> It wasn't the unnecessary comments, it was the unnecessary re-indenting that 
> you did.
> 
> > > Isaac: was there more than one memory leak in goom?
> 
> Yes, the other one was pretty visible from looking at the diff.  I've fixed 
> it, though.
> 
> > > I'd like to see if we can finally get this committed to cvs and then
> > > work on any bugfixes necessary.  It would be nice to get more testers
> > > and that will only happen once it gets committed.  I've gotten quite a
> > > few emails about this, so I know people desire amd64 support for
> > > mythmusic.
> 
> You are aware that you had goom's mmx support completely disabled with this 
> patch, right?  =)  I also noticed a small logic error in the synaesthesia 
> draw algorithm that lead to some corruption..  I've fixed everything I saw, 
> though, and it seems to work well so I've checked it all in.
> 
> Isaac
> _______________________________________________
> mythtv-dev mailing list
> mythtv-dev at mythtv.org
> http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev
>