Hi Iain,

>> > I'd say go for it.  I see that there's a tab that found its way into
>> > lib/gdc.exp, and there's a copyright notice that needs fixing up.
>>
>> that tab is both due the gcc convention (GCS actually) of using tabs
>> instead of 8 spaces, unlike D, and Emacs' tcl mode that follows it.
>> I've now fixed it up to be consistent with the rest of gdc.exp.
>>
>
> I didn't know that applied to dejagnu test scripts.  Feel free to
> re-tab the file so it's consistent, I only noticed because of the
> mixed 8 spaces/tabs.

done now with the following ChangeLog entry:

2019-02-16  Rainer Orth  <r...@cebitec.uni-bielefeld.de>

        libphobos:
        * testsuite/lib/libphobos-dg.exp: Tabify.
        * testsuite/lib/libphobos.exp: Likewise.
        * testsuite/libphobos.cycles/cycles.exp: Likewise.
        * testsuite/libphobos.shared/shared.exp: Likewise.
        * testsuite/libphobos.unittests/unittests.exp: Likewise.

        gcc/testsuite:
        * gdc.dg/dg.exp: Tabify.
        * gdc.dg/lto/lto.exp: Likewise.
        * gdc.test/gdc-test.exp: Likewise.
        * lib/gdc-dg.exp: Likewise.
        * lib/gdc.exp: Likewise.

Given that diff -w came up empty, there's no point in posting the actual
patch.  Bootstrapped without regressions on i386-pc-solaris2.11 and
x86_64-pc-linux-gnu for good measure.

Thanks.
        Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University

Reply via email to