[ovs-dev] [PATCH] tests: switch from signed to unsigned when formatting

Sairam Venugopal vsairam at vmware.com
Tue Jun 20 20:53:43 UTC 2017


Acked-by: Sairam Venugopal <vsairam at vmware.com>





On 6/20/17, 12:44 PM, "ovs-dev-bounces at openvswitch.org on behalf of Alin Serdean" <ovs-dev-bounces at openvswitch.org on behalf of aserdean at cloudbasesolutions.com> wrote:

>x->errline is a signed int
>(https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_openvswitch_ovs_blob_934386dd129a3f15b2546562a775f2f43dd3aac6_tests_test-2Dsflow.c-23L88&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=Z6vowHUOjP5ysP_g372c49Nqc1vEKqHKNBkR5Q5Z7uo&m=52CvYQ6uQ_8gMUb3CO0zBOQi3EAEo7U12sf_jRaLMto&s=jGs1qklVubrW1fOsUpNILtvW2UjkxaFlMmcyjv1eM1o&e= )
>Flagged by static code analysis.
>
>Signed-off-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
>---
> tests/test-sflow.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/tests/test-sflow.c b/tests/test-sflow.c
>index 6125d38..039a1b1 100644
>--- a/tests/test-sflow.c
>+++ b/tests/test-sflow.c
>@@ -675,7 +675,7 @@ print_sflow(struct ofpbuf *buf)
>         process_datagram(x);
>     } else {
>         // CATCH
>-        printf("\n>>>>> ERROR in " __FILE__ " at line %u\n", x->errline);
>+        printf("\n>>>>> ERROR in " __FILE__ " at line %d\n", x->errline);
>     }
> }
> 
>-- 
>2.10.2.windows.1
>_______________________________________________
>dev mailing list
>dev at openvswitch.org
>https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=Z6vowHUOjP5ysP_g372c49Nqc1vEKqHKNBkR5Q5Z7uo&m=52CvYQ6uQ_8gMUb3CO0zBOQi3EAEo7U12sf_jRaLMto&s=rOqbBbyjEA6AKF-RlAt-BrQ1BrGinru0mkM-FdoOoks&e= 


More information about the dev mailing list