On 5/10/19 11:28 AM, Burakov, Anatoly wrote:
On 10-May-19 3:53 PM, Michael Santana wrote:
snprintf guarantees to always correctly place a null terminator in the buffer string. So manually placing a null terminator in a buffer right after a call
to snprintf is redundant code.

Additionally, there is no need to use 'sizeof(buffer) - 1' in snprintf as this means we are not using the last character in the buffer. 'sizeof(buffer)' is
enough.

Cc: sta...@dpdk.org

Signed-off-by: Michael Santana <msant...@redhat.com>
---

LGTM

Acked-by: Anatoly Burakov <anatoly.bura...@intel.com>

ping

Reply via email to