Mathias LANG

Results 625 comments of Mathias LANG

@MoonlightSentinel : Looks like the fix for tuple was incomplete: ``` Test 'runnable/aliasthis.d' failed. The logged output: /tmp/cirrus-ci-build/generated/linux/release/64/dmd -conf= -m64 -Irunnable -fPIC -odtest_results/runnable -oftest_results/runnable/aliasthis_0 runnable/aliasthis.d false [] = int []...

And it trips on the following code: ```D assert(( boo(4) = 2) == 2); assert(( maz(4) = 2) == 2); assert((4.boo = 2) == 2); assert((4.maz = 2) == 2);...

Looks like the rewrite still doesn't understand: ```D assert(( boo(4) = 2) == 2); assert(( maz(4) = 2) == 2); ```

``` ... runnable/previewin.d -preview=dip1000 -preview=in -fPIC () ============================== Test 'runnable/previewin.d' failed. The logged output: /home/circleci/dmd/generated/linux/debug/64/dmd -conf= -m64 -Irunnable -preview=dip1000 -preview=in -fPIC -od/home/circleci/dmd/test/test_results/runnable -of/home/circleci/dmd/test/test_results/runnable/previewin_0.o -c runnable/previewin.d /home/circleci/dmd/test/../../druntime/import/core/internal/dassert.d(335): Error: copy constructor `previewin.NonCopyable.this`...

``` Testing profile ./generated/linux/release/64/profile ./generated/linux/release/64/mytrace.log ./generated/linux/release/64/mytrace.def grep -q '1 .*_Dmain' ./generated/linux/release/64/mytrace.log grep -q '1000 .*uint profile.foo(uint)' ./generated/linux/release/64/mytrace.log diff mytrace.def.exp ./generated/linux/release/64/mytrace.def || diff mytrace.releaseopt.def.exp ./generated/linux/release/64/mytrace.def 2a3 > _D4core8internal6string__T7dstrcmpZQjFNaNbNiNeMxAaMxQeZi 5d5 < _D4core8internal6string__T7dstrcmpZQjFNaNbNiNeMxAaMxQeZi...

We've been using it for our project for a while, and it works well. The linker errors are the only issues we've seen from time to time. I think we...

On the deprecation side of things, we've had those for a while: ``` agora 0.15.0+commit.4.ga7ef19d7e: building configuration "unittest"... /Users/geod24/dlang/ldc-1.26.0/bin/../import/core/internal/dassert.d(295,30): Deprecation: function `vibe.core.path.GenericPath!(PosixPathFormat).GenericPath.Segment.toString` is deprecated - Use .name instead. /Users/geod24/dlang/ldc-1.26.0/bin/../import/core/internal/dassert.d(428,19): instantiated...

Hum, this fails now because `_d_assert_fail` is not pure ~, I assume since the introduction of `inFinalizer` :/~ Will look more into it.

> Only generate `_d_assert_fail` calls in root modules. Isn't that currently the case ?

> This is common practice in C. But D tries to get away from that. I fear this sort of thing will have a tendency to break the sanctity(!) of...