Hey Vedant,

On 11/26/14, vedant agarwala <vedant.k...@gmail.com> wrote:
> Hello Nitul,
>
> Also paste a link to the error using paste.kde.org in the description.
>

I have done so. The link is as follows: http://goo.gl/Csp8uW

> It is a very large diff to analyze. seems like you have added a lot of
> redundant lines in the form of comments. remove them. You should have the
> old code in git and not need to keep them as comments.
>

I have removed the appropriate lines and updated my diff.

> I will take some time to run the code myself. I can only look at the code
> and see if the error is obvious.
>

That would be great.

> In any case, also provide a link to the personal scratch repo commit so
> that i can directly pull from there and run the code when I can.
>

I haven't commited the code as of yet as it still isn't working properly.

> Regards,
> Vedant.
>
> On Wed, Nov 26, 2014 at 2:01 AM, Nitul Datt <nitul1...@gmail.com> wrote:
>
>> Hey!
>>
>> I've put up an initial patch on reviewboard. It's not completely error
>> free and I need some help figuring out what's wrong.
>>
>> --
>> Regards,
>> Nitul Datt
>>
>

--
Regards,
Nitul Datt
_______________________________________________
Amarok-devel mailing list
Amarok-devel@kde.org
https://mail.kde.org/mailman/listinfo/amarok-devel

Reply via email to