<meta http-equiv="content-type" content="text/html; charset=utf-8"><span class="Apple-style-span" style="border-collapse: collapse; color: rgb(51, 51, 51); font-family: arial, sans-serif; font-size: 13px; ">Hi Daniel,<div>
I saw some code in EngineController when I cloned the repository.</div><div>The problem with your approach is that the path of AudioDataOutput will be always active. ADO use the CPU to manage and fill some QVector<short>.</div>
<div>I coded AudioDataCenter to activate the AudioDataOutput path only when there is some module of Amarok that require it. And deactivate it when the last client is unregistered.</div><div>Maybe a better approach could be to create </div>
<div>EngineController::registerDataClient(AudioDataClient* c); </div><div>EngineController::unregisterDataClient(audioDataClient* c);</div><div><br></div><div>instead of my approach:</div><div>AudioDataCenter *EngineController::audioDataCenter();</div>
<div>AudioDataCenter::add(AudioDataClient* c);</div><div>AudioDataCenter::remove(AudioDataClient* c);</div><div><br></div><div>About visualM: I am totally with you about the applet approach. It was only an example.</div><div>
<br></div><div>Maybe I should divide my patch in an AudioDataCenter Patch and a visualM applet patch.</div><div><br></div><div>What do u think ?</div><div><br></div><div>Alex</div></span><br><div class="gmail_quote">On Sat, Aug 13, 2011 at 2:28 PM, Daniel Dewald <span dir="ltr"><<a href="mailto:Daniel.Dewald@time-shift.de">Daniel.Dewald@time-shift.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi Alex,<br>
<br>
As I said my comment wasn't meant for you directly but more an urge I felt to<br>
bring this problem to greater attention as you are not the first (an will most<br>
likely not be the last) in this situation.. I just hate to see work wasted all<br>
the time (be it will be your work or the work of others).<br>
<br>
As for the audioDataOutput: That specific Problem was already "solved" by me by<br>
using signals and slots (then EgineController just had a signal for audio data<br>
which every class could connect to) a long time ago. However I didn't push my<br>
changes as the backends where far from ready at the time (and apart from gst<br>
still are) and some even crashed (xine which is now deprecated).<br>
<br>
Concerning your patch: The biggest problems I can see is that you have<br>
visualization specific code (the visualM stuff) in the engine controller. I<br>
haven't done a lot of work lately but I'd say it doesn't belong there as the<br>
engine controller is meant for phonon stuff only afaik.<br>
<br>
Also in my opinion (and I cant stress enough that it might very well not be<br>
the opinion of the other devs) as amarok tries to advertise modularity<br>
visualization should be done in an applet.<br>
<br>
As I said a lot of this Questions / Problems could have been answered /<br>
avoided if you had asked.<br>
<br>
Greezt<br>
<br>
Daniel<br>
<br>
P.S. I hope someone more clear minded about this then me will comment ;o). I'd<br>
love to here the opinion of the other devs.<br>
<div><div></div><div class="h5"><br>
On Friday, 12. August 2011 16:16:33 Alessandro Siniscalchi wrote:<br>
> Hi Daniel,<br>
> mmm ... ok.<br>
><br>
> No personal pissings here.<br>
><br>
> However my patch ,maybe, is more interesting for the AudioDataCenter class.<br>
> The visualM stuff is very intrusive ... and it was only an example to how to<br>
> use AudioDataCenter.<br>
><br>
> I hope that the guy of vsxu could take some advange from my code, I mean ...<br>
> vsxu is very nice.<br>
><br>
> A.<br>
><br>
> On Fri, Aug 12, 2011 at 3:47 PM, Daniel Dewald<br>
><br>
> <<a href="mailto:Daniel.Dewald@time-shift.de">Daniel.Dewald@time-shift.de</a>>wrote:<br>
> > Hey Alex,****<br>
> ><br>
> > ** **<br>
> ><br>
> > First of all nice to have you on board. So please don’t take my words<br>
> > the<br>
> > wrong way and don’t take them personal. However some other person was<br>
> > also already working on a very similar solution using vsxu. If you had<br>
> > read the mailing list or ASKED in the IRC Channel a lot of this work<br>
> > could have been SHARED. I also worked on visualization for a while but<br>
> > canceled my project because a lot of work on another feature was wasted<br>
> > because of (now obvious) miscommunication problems. I think its<br>
> > wonderful that new people are joining the Amarok team. But I kind of<br>
> > getting really annoyed by the way they do it. When I first wanted to<br>
> > contribute stuff to Amarok  I asked in the IRC channel what I could do<br>
> > and if no one else was working on that stuff (I’m still doing that<br>
> > before beginning on a new feature). I find that only to be a polite<br>
> > thing to do. Just throwing code at a project without caring what other<br>
> > people might be working on is just rude. Be it as it may what is<br>
> > pissing me of the most about it is how the community is (not) handling<br>
> > this problem. It’s just ignored and code is chosen randomly and pushed<br>
> > randomly no matter who was following “protocol” und who was just<br>
> > storming in. As a matter of facts most of the times people who<br>
> > carefully are holding back their code for bug fixing and present it<br>
> > later are left behind while other people who just throw half bred code<br>
> > at the project get their code pushed in immediately. In my opinion<br>
> > people who are that rude should not get rewarded. ****<br>
> ><br>
> > ** **<br>
> ><br>
> > Now as I said Alex this has only partly to do with your patch and has<br>
> > been observed by me before. So don’t feel to pissed about my mail. I<br>
> > just though as I seem to be the only one who cares I might bring it up.<br>
> > It’s not my code that gets wasted this time so I shouldn’t care. But I<br>
> > feel bad for  the poor soul who got his work crushed this time as I<br>
> > know how it feels.****<br>
> ><br>
> > ** **<br>
> ><br>
> > Greetz****<br>
> ><br>
> > ** **<br>
> ><br>
> > Daniel****<br>
> ><br>
> > ** **<br>
> ><br>
> > *From:* <a href="mailto:amarok-devel-bounces@kde.org">amarok-devel-bounces@kde.org</a><br>
> > [mailto:<a href="mailto:amarok-devel-bounces@kde.org">amarok-devel-bounces@kde.org</a>] *On Behalf Of *Alessandro<br>
> > Siniscalchi<br>
> > *Sent:* Donnerstag, 11. August 2011 11:01<br>
> > *To:* <a href="mailto:amarok-devel@kde.org">amarok-devel@kde.org</a><br>
> > *Subject:* Visualization in Amarok****<br>
> ><br>
> > ** **<br>
> ><br>
> > Hi all,****<br>
> ><br>
> > I just finished a patch that add the Visualization to amarok.****<br>
> ><br>
> > ** **<br>
> ><br>
> > <a href="https://git.reviewboard.kde.org/r/102277/****" target="_blank">https://git.reviewboard.kde.org/r/102277/****</a><br>
> ><br>
> > ** **<br>
> ><br>
> > As you can see there are 2 new modules:****<br>
> ><br>
> > ** **<br>
> ><br>
> > - audiodatacenter: it manage all the modules that need raw data****<br>
> ><br>
> > ** **<br>
> ><br>
> > - visualm: it uses audiodatacenter to register and receive data and send<br>
> > it to projectM-qt mainWindow.****<br>
> ><br>
> > ** **<br>
> ><br>
> > First ... I'd like to know what do u think about.****<br>
> ><br>
> > ** **<br>
> ><br>
> > Then ... I'd like to make visualm optional. Which is the way ?****<br>
> ><br>
> > ** **<br>
> ><br>
> > Cheers****<br>
> ><br>
> > Alex****<br>
> ><br>
> > _______________________________________________<br>
> > Amarok-devel mailing list<br>
> > <a href="mailto:Amarok-devel@kde.org">Amarok-devel@kde.org</a><br>
> > <a href="https://mail.kde.org/mailman/listinfo/amarok-devel" target="_blank">https://mail.kde.org/mailman/listinfo/amarok-devel</a><br>
<br>
</div></div><font color="#888888">--<br>
Daniel Dewald B.Sc.<br>
Im Ginselt 8<br>
66709 Weiskirchen<br>
<a href="https://www.time-shift.de/" target="_blank">https://www.time-shift.de/</a><br>
<br>
Zertifikat:<br>
<a href="https://www.time-shift.de/Certificates/" target="_blank">https://www.time-shift.de/Certificates/</a></font><br>_______________________________________________<br>
Amarok-devel mailing list<br>
<a href="mailto:Amarok-devel@kde.org">Amarok-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/amarok-devel" target="_blank">https://mail.kde.org/mailman/listinfo/amarok-devel</a><br>
<br></blockquote></div><br>