[aur-dev] [PATCH] Fixed a couple more lingering JSON bugs
Simo Leone
simo at archlinux.org
Wed Feb 20 16:21:27 EST 2008
On Wed, Feb 20, 2008 at 03:26:06PM -0500, Loui wrote:
> On 2/20/08, Simo Leone <simo at archlinux.org> wrote:
> > I'm pretty sure eliott wants to uber-optimize the json interface because
> > it's very possible that this code will get hit more frequently than
> > any other aur code.
> > Generally though, I don't really see anything wrong with string
> > concatenation, espcially if it enhances readability (which php tends to
> > lack sometimes imho).
> >
> > Patch applied, without string concatenation
> >
> > :) zoom zoom
>
> Alrighty good stuff. That line that I did the string concatenation
> actually had a valid fix though :P the space after AND at the end of
> the string is required. Either that, or a space before "Name=" further
> down in the code.
>
hehehe, wow I missed that, I totally looked.
Fixed in testing though, and some git magic says you fixed it :)
-S
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://archlinux.org/pipermail/aur-dev/attachments/20080220/3afd2971/attachment.pgp>
More information about the aur-dev
mailing list