Re: [PATCH] add basic support for parsing country information element

Marcel Holtmann
2009-05-04T16:29:10Z

Hi Johannes,

> > On a different note, what is your preferred coding style for loops?
> >
> > for (i = 0; i < len; i++) {
> >
> > or
> >
> > for(i=0; i<len-3; i++)
> >
> > I was using the latter one since I assumed that is what you want (it is
> > not my preferred one). Then I realized that you are using both or that
> > something slipped in. Do you want me to send a patch to fix this?
>
> Oh, well, I prefer the former, but some other people put in the latter,
> and I didn't really care too much. No need to send a patch though.

I will sent a patch since I prefer the former too, but assumed you liked
that latter one and that was my mistake in cluttering the code with more
mixed coding style than needed.

Regards

Marcel