> It is safe to call delete[] with nullptr (and doing so avoids the test here as well as in the library)
Fixed.
> You could avoid this (and some related code in the test body) by:
> using LargeMessages = mir_test_framework::BasicClientServerFixture<LargeMessagesServerConfiguration>;
Without the fix, the client hangs indefinitely in mir_connect, so I need to make the connect call async to be able to make the test fail after a reasonable amount of time.
> It is safe to call delete[] with nullptr (and doing so avoids the test here as well as in the library)
Fixed.
> You could avoid this (and some related code in the test body) by: framework: :BasicClientSer verFixture< LargeMessagesSe rverConfigurati on>;
> using LargeMessages = mir_test_
Without the fix, the client hangs indefinitely in mir_connect, so I need to make the connect call async to be able to make the test fail after a reasonable amount of time.