[cairo] [PATCH v3 0/7] test: Don't ignore test output files when written to the wrong directory
Uli Schlachter
psychon at znc.in
Sun Sep 8 11:00:59 PDT 2013
Hi,
On 08.09.2013 19:46, Bryce W. Harrington wrote:
> On Sun, Sep 08, 2013 at 03:03:18PM +0200, Uli Schlachter wrote:
[...]
>>> test: Make cairo_test_mkdir() usable throughout tests
>>
>> Why does this copy&paste _cairo_test_mkdir() to cairo_test_mkdir()? Wouldn't it
>> be enough to rename _cairo_test_mkdir() and make it non-static (and adding it to
>> cairo-test.h)?
>
> _cairo_test_mkdir() is implemented in indivdual tests. This commit
> places it in the core test code so other tests can link to it. You
> can't just rename one of the existing routine implementations.
[...]
Uhm, that patch adds it to cairo-test.c, around line 1791. Right now, we have an
identical implementation called _cairo_test_mkdir() at line 107 in that file.
(Although I have to admit that I mixed this up and thought some other patch
removed that _cairo_test_mkdir() implementation. I should have been clearer, sorry.)
Uli
--
- Captain, I think I should tell you I've never
actually landed a starship before.
- That's all right, Lieutenant, neither have I.
More information about the cairo
mailing list