Makes sense. I'll do that when I wake heh. Just about to sleep.
On Dec 23, 2013 4:03 AM, "Tasos Bekos" <notifications@github.com> wrote:

> In src/carousel/carousel.js:
>
> > @@ -106,6 +106,7 @@ angular.module('ui.bootstrap.carousel', ['ui.bootstrap.transition'])
> >    function restartTimer() {
> >      if (currentTimeout) {
> >        $timeout.cancel(currentTimeout);
> > +      currentTimeout = null;
> >      }
>
> @chrisirhc <https://github.com/chrisirhc> You should also extract this
> code:
>
> if (currentTimeout) {
>       $timeout.cancel(currentTimeout);
>       currentTimeout = null;}
>
> into its own function as it is called multiple times.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/angular-ui/bootstrap/pull/1451/files#r8525583>
> .
>