Thanks, updated in v3.
> -----Original Message----- > From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Saturday, September 9, 2017 12:04 AM > To: Xueming(Steven) Li <xuemi...@mellanox.com> > Cc: Sergio Gonzalez Monroy <sergio.gonzalez.mon...@intel.com>; > dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 1/2] eal/malloc: fix RTE malloc debug macro > > On Fri, 8 Sep 2017 22:50:54 +0800 > Xueming Li <xuemi...@mellanox.com> wrote: > > > - printf("rte_zmalloc didn't zero" > > + printf("rte_zmalloc didn't zero " > > "the allocated memory\n"); > > Please don't break error messages onto two lines. > It makes it harder to use tools like grep and google searches to find where > message is located. > > Checkpatch and other tools make an exception for long lines in quoted strings.
