They are not necessary. In fact, I needed const only in `init_dst_from_rcv` for an internal module, but, with the occasion, I added more const for consistency - including existing code like `su_cmp` and `su_getport`. So, apart from consistency and style, they are neither useful for the community, I guess.