[...]
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
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox
