From: Jérémie NIKAES <jeremie.nikaes@ensimag.imag.fr>
Date: Mon, 06 Jun 2011 22:11:12 GMT
Subject: Re: [PATCH] Add a remote helper to interact with mediawiki, pull & clone handled
Message-ID: <BANLkTi=ABYYY7KwVa6=bY_DXXopKnzZOtg@mail.gmail.com>
In-Reply-To: <BANLkTi=nOa2B_KNfv-XkYFVr8Ngdd2msFA@mail.gmail.com>

> use warnings; ?

Thanks, added this, showed some uses of undefined variables now fixed.
Will figure in the next patch.
>
>> +use Switch;
>
> Ugh -- no.  This is terrible.  Look at this:
>
> [~]% corelist Switch
> Switch was first released with perl v5.7.3 and removed from v5.13.1
>
> Since you do not specify a minimum perl version you might be alright,
> but for those people on 5.14 -- they won't have this module, for good
> reason.  You can, if you wanted use "given/when" as alternate
> constructs to this.

Yes, as I said to Junio Hamano in my previous mail, that's a mistake
from my part, we don't use switch anymore. Sorry about that

>
>> +use MediaWiki::API;
>> +use Storable qw(freeze thaw);
>
> This might have problems transcending storable formats made on a 32bit
> machine, and then trying to unpack them again on 64bit.  Do you really
> need the need for these storable items to be encoded as binary?

Hmmm that is a leftover use from the script of Jeff King that we used.
We don't use this anymore, so that should not be a problem.

Thanks for your advices !

--
Jérémie Nikaes