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

Reply via email to