Re: [B.A.T.M.A.N.] [PATCHv2 1/4] batman-adv: don't switch byte order too often if not needed

Antonio Quartulli
2013-10-13T09:51:37Z

[-- Attachment #1: Type: text/plain, Size: 903 bytes --]

On Sun, Oct 13, 2013 at 05:44:27PM +0800, Marek Lindner wrote:
> On Sunday 13 October 2013 02:50:17 Antonio Quartulli wrote:
> > From: Antonio Quartulli <antonio@open-mesh.com>
> >
> > If possible, operations like ntohs/ntohl should not be
> > performed too often. Use a variable to locally store the
> > converted value and then use it.
> >
> > Signed-off-by: Antonio Quartulli <antonio@open-mesh.com>
> > ---
> > routing.c | 8 +++++---
> > 1 file changed, 5 insertions(+), 3 deletions(-)
>
> I don't quite see why this patch is part of this series.

because at the very beginning I wrongly did this change within patch 2/4 and
Simon correctly suggested me to separate it....

So since then it has been part of this series, but there is no technical reason.
It is just yet another patch which is not really related to the tt-synch thing.

Cheers,

--
Antonio Quartulli

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]