I vote vote for removing unnecessary braces.


On Mon, Apr 28, 2014 at 12:37 PM, Derek Gaston <notifications@github.com>wrote:

> In modules/contact/src/MechanicalContactConstraint.C:
>
> > +    }
> > +    updateContactSet(beginning_of_step);
> > +    _updateContactSet = false;
> > +    _time_last_called = _t;
> > +  }
> > +}
> > +
> > +void
> > +MechanicalContactConstraint::jacobianSetup()
> > +{
> > +  if (_component == 0)
> > +  {
> > +    if (_updateContactSet)
> > +    {
> > +      updateContactSet();
> > +    }
>
> I know you guys in BISON like to do this - but I think it makes it harder
> to read if you have unnecessary curly braces... what do you think
> @permcody <https://github.com/permcody> @andrsd<https://github.com/andrsd>
> @jwpeterson <https://github.com/jwpeterson> @aeslaughter<https://github.com/aeslaughter>??
>
> I vote for removing these unnecessary curly braces everywhere...
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/idaholab/moose/pull/3020/files#r12060902>
> .
>