[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