This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] support: Add TEST_COMPARE_BLOB, support_quote_blob
- From: Carlos O'Donell <carlos at redhat dot com>
- To: Florian Weimer <fweimer at redhat dot com>, libc-alpha at sourceware dot org
- Date: Fri, 2 Mar 2018 08:32:51 -0800
- Subject: Re: [PATCH] support: Add TEST_COMPARE_BLOB, support_quote_blob
- Authentication-results: sourceware.org; auth=none
- References: <20180302105949.E623C40454581@oldenburg.str.redhat.com>
On 03/02/2018 02:59 AM, Florian Weimer wrote:
> The declaration of support_test_compare_blob uses unsigned long,
> to avoid including <stddef.h>.
>
> 2018-03-02 Florian Weimer <fweimer@redhat.com>
>
> * support/Makefile (libsupport-routines): Add support_quote_blob,
> support_test_compare_blob.
> (tests): Add tst-support_quote_blob, tst-test_compare_blob.
> * support/check.h (TEST_COMPARE_BLOB): Define.
> (support_test_compare_blob): Declare.
> * support/support.h (support_quote_blob): Declare.
> * support/support_quote_blob.c: New file.
> * support/support_test_compare_blob.c: Likewise.
> * support/tst-support_quote_blob.c: Likewise.
> * support/tst-test_compare_blob.c: Likewise.
Please add comments to the test cases to explain why you
used the particular test data that you did.
Were you looking to test specific quoting sequences?
Were you using arbitrary data hand picked by you to test the code?
Or did you pick the data to trigger branches in the code that were
part of the implementation?
--
Cheers,
Carlos.