On 03/21, Stephen Hemminger wrote: >On Thu, 21 Mar 2019 17:18:41 +0800 >Xiaolong Ye <xiaolong...@intel.com> wrote: > >> + if (strnlen(value, IFNAMSIZ) > IFNAMSIZ - 1) { >> + RTE_LOG(ERR, AF_XDP, "Invalid name %s, should be less than " >> + "%u bytes.\n", value, IFNAMSIZ) > >Please don't break error message strings across multiple source lines. >It makes it harder to use tools like grep to find errors in source.
Good point, will keep this in mind. Thanks, Xiaolong