On Tue, 21 Jun 2022 at 17:57, Yann Ylavic <ylavic....@gmail.com> wrote:

> On Mon, Jun 20, 2022 at 11:57 PM <i...@apache.org> wrote:
> >
> > Author: ivan
> > Date: Mon Jun 20 21:57:13 2022
> > New Revision: 1902120
> >
> > URL: http://svn.apache.org/viewvc?rev=1902120&view=rev
> >
> []
> > --- apr/apr/branches/1.8.x/test/testfile.c (original)
> > +++ apr/apr/branches/1.8.x/test/testfile.c Mon Jun 20 21:57:13 2022
> []
> > @@ -1298,8 +1366,7 @@ abts_suite *testfile(abts_suite *suite)
> >      abts_run_test(suite, test_fail_read_flush, NULL);
> >      abts_run_test(suite, test_buffer_set_get, NULL);
> >      abts_run_test(suite, test_xthread, NULL);
> > -    abts_run_test(suite, test_datasync_on_file, NULL);
> > -    abts_run_test(suite, test_datasync_on_stream, NULL);
>
> Were these two removed intentionally? This causes the below warnings
> at least currently:
>
> testfile.c:1243:13: error: ‘test_datasync_on_stream’ defined but not
> used [-Werror=unused-function]
> testfile.c:1222:13: error: ‘test_datasync_on_file’ defined but not
> used [-Werror=unused-function]
>
> Hi Yann,

No, this change is unintentional. Just incorrect conflict resolution.
Should be fixed in r1902145.


-- 
Ivan Zhakov

Reply via email to