Yahoo Japan Removal
Michael McConville
mmcconville at mykolab.com
Thu Jun 18 13:21:31 EDT 2015
On Thu, Jun 18, 2015 at 04:01:25AM -0500, Richard Laager wrote:
> I'm just looking at the diff online:
> https://hg.pidgin.im/soc/2015/mmcc/main/rev/0cf05a71fef7
>
> You have a whitespace issue in the second hunk of
> libpurple/protocols/yahoo/yahoo_aliases.c. You're using spaces and the rest
> of the code is using tabs.
>
> Likewise twice in the 4th hunk in the same file. Also, there you've twice
> got a variable declaration (escaped_alias) that's no longer at the top of
> the block.
>
> Spaces instead of tabs in libpurple/protocols/yahoo/yahoo_filexfer.c.
>
> Spaces instead of tabs in the both hunks of
> libpurple/protocols/yahoo/yahoochat.c.
>
> Spaces instead of tabs in the second hunk of
> libpurple/protocols/yahoo/ymsg.c. And the 9th, which is "build the raw
> packet and send it". And the 10th and 11th.
>
> Otherwise, nothing jumps out at me as crazy. It all seems pretty
> straightforward.
Thanks for that.
I just committed a fix:
https://hg.pidgin.im/soc/2015/mmcc/main/rev/bfe802e8be3e
More information about the Devel
mailing list