Please ignore this. I *literally* did the same thing I did with the last
'git send-email.'

On Tue, May 17, 2016 at 3:01 PM, Dan Cross <[email protected]> wrote:

> Remove several long-commented debugging statements and change
> a couple of '//' comments into '/* */' style in order to more
> closely match a) the overall style of the file, and b) the Akaros
> style guide.
>
> Change-Id: I55aed697c3b7e4ba589260ed14002dd3d44028bf
> Signed-off-by: Dan Cross <[email protected]>
> Signed-off-by: Barret Rhoden <[email protected]>
> ---
>  kern/src/net/arp.c | 12 +++++-------
>  1 file changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/kern/src/net/arp.c b/kern/src/net/arp.c
> index c821abd..bd5b6fc 100644
> --- a/kern/src/net/arp.c
> +++ b/kern/src/net/arp.c
> @@ -121,7 +121,8 @@ static struct arpent *newarp6(struct arp *arp, uint8_t
> * ip, struct Ipifc *ifc,
>                         freeblist(xp);
>                         xp = next;
>                 }
> -       } else {        // queue icmp unreachable for rxmitproc later on,
> w/o arp lock
> +       } else {
> +               /* queue icmp unreachable for rxmitproc later, w/o arp
> lock */
>                 if (xp) {
>                         if (arp->dropl == NULL)
>                                 arp->dropf = xp;
> @@ -345,7 +346,6 @@ arpenter(struct Fs *fs, int version, uint8_t * ip,
> uint8_t * mac, int n,
>         arp = fs->arp;
>
>         if (n != 6) {
> -//      print("arp: len = %d\n", n);
>                 return;
>         }
>
> @@ -364,7 +364,6 @@ arpenter(struct Fs *fs, int version, uint8_t * ip,
> uint8_t * mac, int n,
>         }
>
>         if (r == NULL) {
> -//      print("arp: no route for entry\n");
>                 return;
>         }
>
> @@ -472,7 +471,7 @@ int arpwrite(struct Fs *fs, char *s, long len)
>                         }
>                 }
>                 memset(arp->hash, 0, sizeof(arp->hash));
> -               // clear all pkts on these lists (rxmt, dropf/l)
> +               /* clear all pkts on these lists (rxmt, dropf/l) */
>                 arp->rxmt = NULL;
>                 arp->dropf = NULL;
>                 arp->dropl = NULL;
> @@ -603,11 +602,11 @@ static uint64_t rxmitsols(struct arp *arp)
>         a = arp->rxmt;
>         if (a == NULL) {
>                 nrxt = 0;
> -               goto dodrops;   //return nrxt;
> +               goto dodrops;   /* return nrxt; */
>         }
>         nrxt = a->rtime - NOW;
>         if (nrxt > 3 * ReTransTimer / 4)
> -               goto dodrops;   //return nrxt;
> +               goto dodrops;   /* return nrxt; */
>
>         for (; a; a = a->nextrxt) {
>                 ifc = a->ifc;
> @@ -693,7 +692,6 @@ static void rxmitproc(void *v)
>         uint64_t wakeupat;
>
>         arp->rxmitp = current;
> -       //print("arp rxmitproc started\n");
>         if (waserror()) {
>                 arp->rxmitp = 0;
>                 poperror();
> --
> 2.8.0.rc3.226.g39d4020
>
>

-- 
You received this message because you are subscribed to the Google Groups 
"Akaros" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to