On Thursday 10 December 2009, Øyvind Harboe wrote:
> On Thu, Dec 10, 2009 at 9:10 PM, David Brownell <davi...@pacbell.net> wrote:
> > On Thursday 10 December 2009, Ųyvind Harboe wrote:
> >> +       values[1]=(1<<5)|reg_addr;
> >
> > Could you please add spaces around the operators?
> >
> >        values[1] = (1 << 5) | reg_addr;
> >
> > I've been carrying around a cleanup patch, as-yet unmerged,
> > to fix that routine...
> 
> Fix that routine?? It's part of that inner loop so I'll be following
> it closely :-)

I'm pretty sure that removing the whitespace only
hurts readability, and doesn't generate more code.  ;)


_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to