Stephen Hemminger <step...@networkplumber.org> writes: > On Fri, 27 Jan 2017 09:56:52 -0500 > Aaron Conole <acon...@redhat.com> wrote: > >> + /* no need to TAILQ_REMOVE, we are going to disallow re-attemtps >> + * for rte_eal_init(). */ > > Please put multi-line comments in form: > /* > * this is a long comment > * because there really is lots to say > */
Okay, will do. > In many cases, having shorter comment is the best way to handle these. > Often developer writes long comment for themselves because what ever problem > they saw was urgent. Then comment becomes irrelevant later. > > /* TAILQ_REMOVE not needed, error is already fatal */ I'll fold this in. Thanks for the review!