<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Oct 19, 2015 at 10:07 PM, Jorge Villase├▒or <span dir="ltr"><<a href="mailto:salinasv@gmail.com" target="_blank">salinasv@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br><div dir="ltr"><span class=""><div class="gmail_extra"><div class="gmail_quote">On Mon, Oct 19, 2015 at 1:00 PM, Koosha Khajehmoogahi <span dir="ltr"><<a href="mailto:koosha.khajeh@gmail.com" target="_blank">koosha.khajeh@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I *think* I did it but the head apparently has changed. Sorry, I don't<br>
have professional experience with Mercurial; I'm a git guy. Please let<br>
me know if there is anything that needs to be changed/reverted.<br clear="all"><br></blockquote></div><br></div></span><div class="gmail_extra">Hi Koosha, maybe you didn't pushed to your repository after the merge.<br></div><div class="gmail_extra">I tried to pull your changes, merge the main default branch with your changes but it doesn't merge cleanly.<br><br></div><div class="gmail_extra">Can you try again so we can review it easily? If you can't we can try it ourself but it makes more sense for you to merge the changes since you know better what is going on in there<br><br></div><span class=""></span></div></blockquote></div><br></div><div class="gmail_extra">Hi all. I reviewed the code for XMPP improvements and I am not 100% sure it is ready to be merged.<br><br></div><div class="gmail_extra">The main concerns are that XEP-280 does a change in sever.c (rev<span style="font-weight:600"> 9c71168e8a68</span>) that I think could break other prpls. We need to verify that and/or fix other prpls.<br></div><div class="gmail_extra">On this regard, jgeboski and dx mentioned that our message API may need a modification to completely support this usecase which is also used by facebook's plugin. So it may worth revisit the Libpurple side of this change.<br><br></div><div class="gmail_extra">Also, there are a couple of TODOs<br>which may break the history┬á (rev<span style="font-weight:600"> 9681c39416ec</span>) or<br></div><div class="gmail_extra">looks like the feature is not complete (rev<span style="font-weight:600"> 9681c39416ec</span>).<br><br></div><div class="gmail_extra">I merged main/default to GSoC branch  and did a nice diff to make it easier to see only the changes introduced on this branch (find attached the diff).<br><br></div><div class="gmail_extra">I would like you guys to review and suggest if we should cherry-pick some of the complete features / code improvements <br><br></div><div class="gmail_extra">repository history: <a href="https://hg.pidgin.im/soc/2015/koosha/main/graph">https://hg.pidgin.im/soc/2015/koosha/main/graph</a><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">-- <br><div class="gmail_signature">Masca<br><br>A: Because it messes up the order in which people normally read text.<br>Q: Why is top-posting such a bad thing?<br>A: Top-posting.<br>Q: What is the most annoying thing on usenet and in e-mail?</div>
</div></div>