On Tue, Mar 31, 2020 at 12:25 AM Bart Van Assche <bvanassche@xxxxxxx> wrote: > > On 2020-03-30 03:34, Jinpu Wang wrote: > > On Sat, Mar 28, 2020 at 5:26 AM Bart Van Assche <bvanassche@xxxxxxx> wrote: > >> On 2020-03-20 05:16, Jack Wang wrote: > >>> +/** > >>> + * rtrs_addr_to_sockaddr() - convert path string "src,dst" to sockaddreses > >>> + * @str: string containing source and destination addr of a path > >>> + * separated by comma. I.e. "ip:1.1.1.1,ip:1.1.1.2". If str > >>> + * contains only one address it's considered to be destination. > >>> + * @len: string length > >>> + * @port: will be set to port. > >> ^^^^^^^^^^^^^^^^^^^ > >> What does this mean? Please make comments easy to comprehend. > > how about just "port number"? > > I don't think that's enough information. How about "destination port > number"? Sounds better, thank you!