remove internal usage of redundant uci_ptr.last
authorJan Venekamp <jan@venekamp.net>
Fri, 14 Jul 2023 18:28:11 +0000 (20:28 +0200)
committerHauke Mehrtens <hauke@hauke-m.de>
Thu, 10 Aug 2023 17:43:45 +0000 (19:43 +0200)
commit5781664d5087ccc4b5ab58505883231212dbedbc
treeb8b923625f22f389a8e68c63324d0e559cf61208
parent3cda25127b1963ae7167e8a01a7bcacc7c89c29c
remove internal usage of redundant uci_ptr.last

In uci_lookup_ptr and uci_set the pointer uci_ptr ptr.last is set to
the element corresponding to the first of: ptr.o, ptr.s, ptr.p.

Thus, ptr.last is redundant and in case of uci_set is (and was) not
always consistently set.

In order to simplify the code this commit removes internal usage
of ptr.last, and remove setting it from uci_set (and from uci_add_list
that was never used anyway).

As it is part of the public C api ptr.last cannot be completely
removed though. A search on lxr.openwrt.org shows that it is used as
the output of uci_lookup_ptr in several packages.

So we leave setting ptr.last in uci_lookup_ptr intact.

Signed-off-by: Jan Venekamp <jan@venekamp.net>
cli.c
delta.c
list.c
lua/uci.c