-
Notifications
You must be signed in to change notification settings - Fork 79
Description
First I want to thanks for the work to this library! It has been really helpful to me.
Recently when I tried to delete an address with this library, I found it was much more complex compared to adding an address. When adding an address we have dedicated add() function in AddressHandle, which accepts the index, address, and prefix_len directly, but when deleting, the del() function we can use is too simple: we need to construct the AddressMessage on our own. There is even no example for that and we already have some issues for this: #80, #100.
I have no idea why del() is made this way. I think it's not too much work to add another function like del_addr_prefix_len() to delete an address as easy as adding them with add(), and also to add an example for it. I can implement it if this is accepted.