Yeap, and I still think this is messy, especially because in the end you're
manipulating a fairly small string...
On 30 Nov 2013 13:34, "Ben Scholzen" <notifications@github.com> wrote:

> In src/Dash/Router/Http/Route/Generic.php:
>
> >      {
> > +        if ($this->secure) {
> > +            $uri->setScheme('https');
>
> It should clearly *not* be mutable. Consider similar APIs where you pass
> down an Imagick object so the lower methods can draw on it. It's the exact
> same situation here. We have a drawing board, and the routes draw on it.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/DASPRiD/Dash/pull/32/files#r8010835>
> .
>