[Catalyst] uri_for() doesn't encode to utf8 first argument

Jonathan Rockway jon at jrock.us
Tue May 6 18:00:24 BST 2008


* On Tue, May 06 2008, Dmitriy S. Sinyavskiy wrote:
> The -diff
>
> Index: Catalyst.pm
> ===================================================================

Please regenerate this diff without the whitespace changes.  It is
completely unreadable in the current form.

Examples:

> -  engine_class context_class request_class response_class stats_class 
> +  engine_class context_class request_class response_class stats_class

> -    
> +

> -        $c->stash->{data} = 
> +        $c->stash->{data} =

These spurious changes fuck up the history (when we apply the patch),
and they make it extremely difficult to review your changes.

GNU diff accepts an "--ignore-space-change" option that should remove
these extra changes.

Regards,
Jonathan Rockway

-- 
print just => another => perl => hacker => if $,=$"



More information about the Catalyst mailing list