Hello,<br>    Thanks for Michael.I was able to find who wrote that line of code.His name is Jens Mueller so how can I get in contact with him.Also Gilles told that wait would be better option but it's a long time since we discussed this todo and no-one has yet replied.So what should be done now and how to know way to contact Jens Mueller.<br>
PS: I know this is a minor patch but this would at least remove a todo from project<br><br>Thanks  <br><br><div class="gmail_quote">On Tue, Mar 12, 2013 at 1:02 AM, Michael G. Hansen <span dir="ltr"><<a href="mailto:mike@mghansen.de" target="_blank">mike@mghansen.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<div class="im"><br>
<br>
On 03/04/2013 01:35 PM, Saurabh Patel wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
     So what should be done.TODO should be removed or wait for the reply of<br>
one who coded<br>
</blockquote>
<br></div>
you can find out who wrote the code using git`s blame command:<br>
<br>
git blame filename<br>
<br>
This will give you the names of the people who wrote each line. If you are using an IDE such as kdevelop, it can also show this information. The feature is called "annotate" there.<br>
<br>
Best regards,<br>
<br>
Michael<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
<br>
Thanks<br>
<br>
On Mon, Mar 4, 2013 at 5:45 PM, Gilles Caulier <<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@gmail.com</a>><u></u>wrote:<br>
<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I think *m_talker->loggedIn() is a method to check if talker is already<br>
logged (as it return bool).*<br>
*<br>
*<br>
*I don't coded this tool, so i'm not sure...*<br>
*<br>
*<br>
*As well, for me, code commented can be removed...*<br>
*<br>
*<br>
*Gilles Caulier*<br>
<br>
<br>
2013/3/4 Saurabh Patel <<a href="mailto:saurabhpatel7717@gmail.com" target="_blank">saurabhpatel7717@gmail.com</a>><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Hello,<br>
     I was going through kipi-plugins in digikam.In kipi-plugin for<br>
picasawebexport I found a TODO on line 320 in picasawebwindow.cpp.I am<br>
pasting the whole method here.<br>
<br>
void PicasawebWindow::<u></u>slotLoginDone(int errCode, const QString& errMsg)<br>
{<br>
     m_widget->progressBar()->hide(<u></u>);<br>
<br></div>
    * if (errCode == 0 /*TODO&& m_talker->loggedIn()*/)*<div><div class="h5"><br>
     {<br>
         buttonStateChange(true);<br>
         m_token = m_talker->token();<br>
         m_widget->m_albumsCoB->clear()<u></u>;<br>
         m_talker->listAlbums(m_talker-<u></u>>getUserName());<br>
     }<br>
     else<br>
     {<br>
         KMessageBox::error(this, i18n("Picasaweb Call Failed: %1\n",<br>
errMsg));<br>
     }<br>
}<br>
<br>
<br>
I have a question that the method loggedIn is not defined in<br>
picasawebtalker so is it to be implemented there.Also is it necessary since<br>
signallogindone signal in picasawebtalker is emitted after checking success<br>
to get username and password<br>
<br>
Thanks<br>
<br>
______________________________<u></u>_________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org" target="_blank">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/<u></u>listinfo/digikam-devel</a><br>
<br>
<br>
</div></div></blockquote><div><div class="h5">
<br>
______________________________<u></u>_________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org" target="_blank">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/<u></u>listinfo/digikam-devel</a><br>
<br>
<br>
</div></div></blockquote><div><div class="h5">
<br>
<br>
<br>
______________________________<u></u>_________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org" target="_blank">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/<u></u>listinfo/digikam-devel</a><br>
<br>
</div></div></blockquote><div class="HOEnZb"><div class="h5">
<br>
______________________________<u></u>_________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org" target="_blank">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/<u></u>listinfo/digikam-devel</a><br>
</div></div></blockquote></div><br>