bobtfish at bobtfish
Oct 28, 2009, 5:11 PM
Post #4 of 4
(754 views)
Permalink
|
On 27 Oct 2009, at 10:29, Bernhard Graf wrote:
>> I think that your patch may be a little too enthusiastic about
>> encoding
>> as it is currently, but also looks like a reasonable first attempt.
>
> hmm, what do you mean with too enthusiastic?
I mean may encode too much - so, for example, people working around
the fact that it doesn't work right currently may end up with things
double encoded etc.. I may be wrong - I just glanced at the patch. I
know for _certain_ that the code in my branch will fail like this
currently.
If you have spare tuits to work on this then please feel free to turn
up on irc for a commit bit - I've written out a list of tests I think
are needed for the second attempt at this (as previously noted, I
tried to do this before, but broke stuff - so many more tests and
much more care is being employed this time round)
Cheers
t0m
_______________________________________________
List: Catalyst [at] lists
Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst
Searchable archive: http://www.mail-archive.com/catalyst [at] lists/
Dev site: http://dev.catalyst.perl.org/
|
