signed vs. unsigned char returning in master
https://launchpad.net/~mscore-ubuntu/+archive/ubuntu/mscore-nightly/+so…
Some architectures fail to build, e.g. https://launchpadlibrarian.net/399724960/buildlog_ubuntu-bionic-arm64.m… has:
In file included from /<>/musescore-snapshot-3.0~pre20181127+dfsg1/libmscore/note.h:26:0,
from /<>/musescore-snapshot-3.0~pre20181127+dfsg1/libmscore/durationtype.h:17,
from /<>/musescore-snapshot-3.0~pre20181127+dfsg1/libmscore/stafftype.h:19,
from /<>/musescore-snapshot-3.0~pre20181127+dfsg1/libmscore/xml.h:17,
from /<>/musescore-snapshot-3.0~pre20181127+dfsg1/midi/midifile.cpp:14:
/<>/musescore-snapshot-3.0~pre20181127+dfsg1/libmscore/tremolo.h:24:26: error: enumerator value -1 is outside the range of underlying type ‘char’
INVALID_TREMOLO = -1,
^
We fixed this in 2.x IIRC, mostly by specifying 'signed char' or 'unsigned char' explicitly as enum type. If you just use 'char' you can only use values 0‥127 portably (ASCII).
Comments
Is that the only such enum, the only compile error?
In reply to Is that the only such enum,… by Jojo-Schmitz
I wouldn’t know, the build breaks afterwards.
But I can test patches. Slowly.
Besides your
TremoloType
I found a nother one,HairpinType
. All otherenum class ... : char
look clean, as far as I can tell. See https://github.com/musescore/MuseScore/pull/4272Fixed in branch master, commit 9dc411ffa0
fix #279123: signed vs. unsigned char returning in master
Fixed in branch master, commit 1b1912d6ed
Merge pull request #4272 from Jojo-Schmitz/signed-char-enums
fix #279123: signed vs. unsigned char returning in master
Fix confirmed, thanks a lot!
https://launchpad.net/~mscore-ubuntu/+archive/ubuntu/mscore-nightly/+so…
Automatically closed -- issue fixed for 2 weeks with no activity.