[mythtv] Re: [PATCH] mythmusic amd64 support yet again
Daniel Paessler
paessler at gmx.de
Fri Jan 21 03:11:51 EST 2005
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
>
More information about the mythtv-dev
mailing list