[PATCH] portmap: new applet

gotrunks at gmail.com gotrunks at gmail.com
Sun May 1 22:33:05 UTC 2011


[...]

Hi,

> +                       if (p->pml_next == NULL) {
> +                               if (pp == NULL) {
> +                                       G.pl = NULL;
> +                               } else {
> +                                       pp->pml_next = NULL;
> +                               }
> +                       } else {
> +                               if (pp == NULL) {
> +                                       G.pl = p->pml_next;
> +                               } else {
> +                                       pp->pml_next = p->pml_next;
> +                               }
> +                       }

I think you could write less code here:

if (pp == NULL)
    G.pl = p->pml_next;
else
    pp->pml_next = p->pml_next;


Regards,
-- 
Pere


More information about the busybox mailing list