Enable AddressSanitizer for the tests

[AntonioBL]

No description provided.

[AntonioBL]

Of course the tests will fail because there are open memory address bugs in current code.

[AntonioBL]

I enabled only "make installdebug" since it is already calling "make debug". By not calling "make debug" twice we get ~5 min of compilation time.

[AntonioBL]

I updated also PR #4001

[dmitrio95]

As #4278 got merged, maybe this PR can also be applied?
@anatoly-os could you please restart the Travis build?

[AntonioBL]

It now reports a memory access error for mtest/libmscore/clef_courtesy/tst_clef_courtesy.cpp at line 141 [inside clef_courtesy02()]. I cannot obtain the same report (i.e. everything is fine) when performing the actions of the test via MuseScore GUI, so it could be a problem of dropdata or clef initialization inside the test.

[AntonioBL]

And indeed, here:

MuseScore/mtest/libmscore/clef_courtesy/tst_clef_courtesy.cpp

Line 140 in d4443a3

dropData.element = clef;

there is a typo: dropData.element = clef; should read instead dropData.dropElement = clef;

[Jojo-Schmitz]

I'd add the fix from #4334 here (too), just to prove that it plugs a memory hole?

[AntonioBL]

Ok. If everything went ok, I should have rebased this PR onto #4334
This should not screw the commit history.

[Jojo-Schmitz]

You may want to join us on Telegram