Ahh okay. I don't think the GitHub mobile on iOS takes me to the comment in that case. Will check out on desktop!

Sent from my iPhone

> On 28 May 2014, at 22:40, Caitlin Potter <notifications@github.com> wrote:
> 
> In src/ng/rootScope.js:
> 
> > @@ -400,11 +400,7 @@ function $RootScopeProvider(){
> >  
> >          deregisterFns.push(self.$watch(watchGroupFn, function () {
> >            listener(newValues, oldValues, self);
> > -          if (unwatchCount === 0) {
> > -            watchGroupFn.$$unwatch = true;
> > -          } else {
> > -            watchGroupFn.$$unwatch = false;
> > -          }
> > +          watchGroupFn.$$unwatch = unwatchCount === 0 ? true : false;
> I specifically linked to one of Igor's review comments from the PR which implemented this --- @lgalfaso originally used a similar expression, and was asked to replace it with if / else blocks.
> 
> —
> Reply to this email directly or view it on GitHub.