| Group |
Location |
Level |
Code |
Message |
| General Analysis |
qglobal.h:274 |
Medium |
V677 |
Custom declaration of a standard 'ushort' type. The declaration from system header files should be used instead. |
| General Analysis |
qglobal.h:275 |
Medium |
V677 |
Custom declaration of a standard 'uint' type. The declaration from system header files should be used instead. |
| General Analysis |
qglobal.h:276 |
Medium |
V677 |
Custom declaration of a standard 'ulong' type. The declaration from system header files should be used instead. |
| General Analysis |
qobjectdefs.h:589 |
Medium |
V690 |
The 'QMetaObject::Connection' class implements the copy assignment operator, but lacks a copy constructor. It is dangerous to use such a class. |
| General Analysis |
qobjectdefs.h:589 |
Medium |
V690 |
The 'QMetaObject::Connection' class implements the move assignment operator, but lacks a move constructor. It is dangerous to use such a class. |
| General Analysis |
qstringliteral.h:101 |
Medium |
V717 |
It is suspicious to cast object of base class 'QArrayData' to derived class 'QTypedArrayData'. |
| General Analysis |
qstringview.h:222 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstringview.h:236 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstringview.h:238 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstringview.h:240 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstringview.h:242 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstringview.h:244 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstringview.h:288 |
High |
V521 |
Such expressions using the ',' operator are dangerous. Make sure the expression is correct. |
| General Analysis |
qstringview.h:288 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstringview.h:289 |
High |
V521 |
Such expressions using the ',' operator are dangerous. Make sure the expression is correct. |
| General Analysis |
qstringview.h:289 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstringview.h:297 |
High |
V521 |
Such expressions using the ',' operator are dangerous. Make sure the expression is correct. |
| General Analysis |
qstringview.h:297 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstring.h:109 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstring.h:155 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstring.h:157 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstring.h:159 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstring.h:161 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstring.h:163 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qstring.h:926 |
High |
V521 |
Such expressions using the ',' operator are dangerous. Make sure the expression is correct. |
| General Analysis |
qstring.h:926 |
High |
V685 |
Consider inspecting the return statement. The expression contains a comma. |
| General Analysis |
qlist.h:224 |
Medium |
V690 |
The 'iterator' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
qlist.h:277 |
Medium |
V690 |
The 'const_iterator' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
qregexp.h:58 |
Medium |
V690 |
The 'QRegExp' class implements the move assignment operator, but lacks a move constructor. It is dangerous to use such a class. |
| General Analysis |
qmap.h:214 |
High |
V717 |
It is suspicious to cast object of base class 'QMapNodeBase' to derived class 'QMapNode'. |
| General Analysis |
qmap.h:215 |
High |
V717 |
It is suspicious to cast object of base class 'QMapNodeBase' to derived class 'QMapNode'. |
| General Analysis |
qmap.h:328 |
Medium |
V717 |
It is suspicious to cast object of base class 'QMapDataBase' to derived class 'QMapData'. |
| General Analysis |
qmap.h:331 |
Medium |
V717 |
It is suspicious to cast object of base class 'QMapDataBase' to derived class 'QMapData'. |
| General Analysis |
qmap.h:346 |
Medium |
V717 |
It is suspicious to cast object of base class 'QMapDataBase' to derived class 'QMapData'. |
| General Analysis |
qdatetime.h:240 |
Medium |
V602 |
Consider inspecting the 'sizeof (ShortData) * 8 > 50' expression. '>' possibly should be replaced with '>>'. |
| General Analysis |
qdatetime.h:235 |
Medium |
V690 |
The 'Data' class implements a move constructor, but lacks the move assignment operator. It is dangerous to use such a class. |
| General Analysis |
qshareddata.h:68 |
Medium |
V690 |
Instantiation of QSharedDataPointer < QLocalePrivate >: The 'QSharedDataPointer' class implements a move constructor, but lacks the move assignment operator. It is dangerous to use such a class. |
| General Analysis |
qcontiguouscache.h:340 |
Medium |
V678 |
An object is used as an argument to its own method. Consider checking the first actual argument of the 'freeData' function. |
| General Analysis |
qshareddata.h:151 |
Medium |
V690 |
Instantiation of QExplicitlySharedDataPointer < QRegularExpressionPrivate >: The 'QExplicitlySharedDataPointer' class implements a move constructor, but lacks the move assignment operator. It is dangerous to use such a class. |
| General Analysis |
qcborvalue.h:327 |
Medium |
V690 |
The 'QCborValueRef' class implements the copy assignment operator, but lacks a copy constructor. It is dangerous to use such a class. |
| General Analysis |
qcbormap.h:294 |
Medium |
V519 |
The 'v' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 293, 294. |
| General Analysis |
qcbormap.h:300 |
Medium |
V519 |
The 'v' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 299, 300. |
| General Analysis |
qcbormap.h:306 |
Medium |
V519 |
The 'v' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 305, 306. |
| General Analysis |
qcbormap.h:312 |
Medium |
V519 |
The 'v' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 311, 312. |
| General Analysis |
qresultstore.h:149 |
Medium |
V572 |
It is odd that the object which was created using 'new' operator is immediately cast to another type. |
| General Analysis |
qjsonarray.h:171 |
Medium |
V690 |
The 'const_iterator' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
qversionnumber.h:109 |
Medium |
V794 |
The assignment operator should be protected from the case of 'this == &other'. |
| General Analysis |
qbasicatomic.h:199 |
Medium |
V601 |
Instantiation of QBasicAtomicInteger <bool>: The 'fetchAndAddOrdered(1) + 1' value is implicitly cast to the bool type. |
| General Analysis |
qbasicatomic.h:208 |
Medium |
V601 |
Instantiation of QBasicAtomicInteger <bool>: The 'fetchAndAddOrdered(v) + v' value is implicitly cast to the bool type. |
| General Analysis |
qkeysequence.h:71 |
Medium |
V690 |
The 'QKeySequence' class implements the move assignment operator, but lacks a move constructor. It is dangerous to use such a class. |
| General Analysis |
qtextobject.h:255 |
Medium |
V690 |
The 'iterator' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
qsslconfiguration.h:82 |
Medium |
V690 |
The 'QSslConfiguration' class implements the move assignment operator, but lacks a move constructor. It is dangerous to use such a class. |
| General Analysis |
qstyleoption.h:402 |
Medium |
V690 |
The 'QStyleOptionViewItem' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
qtconcurrentthreadengine.h:201 |
Medium |
V690 |
The 'ThreadEngineStarterBase' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
qtconcurrentrunbase.h:84 |
High |
V678 |
An object is used as an argument to its own method. Consider checking the first actual argument of the 'setRunnable' function. |
| General Analysis |
qvariant.h:395 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: data. |
| General Analysis |
qcache.h:52 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: t, c, p, n. |
| General Analysis |
qsharedpointer_impl.h:154 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: destroyer. |
| General Analysis |
qarraydataops.h:352 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: n. |
| General Analysis |
qfuture.h:113 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: future, index. |
| General Analysis |
qtransform.h:69 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: m_13, m_23, m_33, m_type, m_dirty, d. |
| General Analysis |
qmatrix.h:60 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _m11, _m12, _m21, _m22, _dx, _dy. |
| General Analysis |
qquaternion.h:60 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: wp, xp, yp, zp. |
| General Analysis |
qmatrix4x4.h:63 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: m. |
| General Analysis |
qtexttable.h:57 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: fragment. |
| General Analysis |
qtconcurrentiteratekernel.h:120 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: currentResultCount. |
| General Analysis |
audiofile.cpp:43 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: hasInstrument, idx, _type. |
| General Analysis |
audiofile.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
all.h:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
qtlockedfile_unix.cpp:74 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
qtlockedfile_unix.cpp:102 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
qtlocalpeer.cpp:129 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
qtlocalpeer.cpp:193 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
qshareddata.h:151 |
Medium |
V690 |
Instantiation of QExplicitlySharedDataPointer < QSslCertificatePrivate >: The 'QExplicitlySharedDataPointer' class implements a move constructor, but lacks the move assignment operator. It is dangerous to use such a class. |
| General Analysis |
modulessetup.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
qshareddata.h:151 |
Medium |
V690 |
Instantiation of QExplicitlySharedDataPointer < QPlatformPixmap >: The 'QExplicitlySharedDataPointer' class implements a move constructor, but lacks the move assignment operator. It is dangerous to use such a class. |
| General Analysis |
actioneventobserver.cpp:34 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: m_scoreState. |
| General Analysis |
shortcut.h:126 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _assignedWidget. |
| General Analysis |
actioneventobserver.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
telemetrypermissionmodel.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
telemetryservice.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
telemetrysetup.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
telemetrypermissiondialog.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
synthesizerstate.h:32 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: id. |
| General Analysis |
zita.h:118 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _gmf, _glo, _wlo, _whi. |
| General Analysis |
compressor.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
compressorgui.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
effect.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
effectgui.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
noeffect.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
zita.cpp:236 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _i. |
| General Analysis |
zita.cpp:261 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _ir, _iw. |
| General Analysis |
zita.cpp:5 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
zitagui.cpp:36 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: mx, my, oval. |
| General Analysis |
zitagui.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
colorlabel.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
aslider.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
denomspinbox.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
knob.cpp:256 |
Medium |
V636 |
The '_scaleWidth / 2' expression was implicitly cast from 'int' type to 'double' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;. |
| General Analysis |
knob.cpp:29 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: startY. |
| General Analysis |
knob.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
midipanknob.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
mslider.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
panknob.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
pitchedit.cpp:66 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
pitchedit.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
pitchlabel.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
chordlist.h:193 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: id, generated, renderListGenerated, exportOk. |
| General Analysis |
chordlist.h:99 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: type, movex, movey. |
| General Analysis |
chordlist.h:100 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: movex, movey. |
| General Analysis |
poslabel.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
slider.cpp:28 |
Medium |
V730 |
It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: dragppos. |
| General Analysis |
slider.cpp:38 |
Medium |
V730 |
It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: dragppos. |
| General Analysis |
slider.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
styledslider.cpp:56 |
Medium |
V636 |
The 'ad.y() / 120' expression was implicitly cast from 'int' type to 'double' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;. |
| General Analysis |
styledslider.cpp:34 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _value, draggingMouse, mouseDownVal. |
| General Analysis |
styledslider.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
utils.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
volknob.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
volslider.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
lexer.cpp:9 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
parser.cpp:465 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: beat, beats. |
| General Analysis |
parser.cpp:9 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
symbols.cpp:53 |
Medium |
V1016 |
Expression 's < 0' is always false. |
| General Analysis |
symbols.cpp:55 |
Medium |
V547 |
Expression is always false. |
| General Analysis |
symbols.cpp:9 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
event.h:264 |
Medium |
V690 |
The 'Event' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
scoreElement.h:173 |
Medium |
V690 |
The 'ScoreElement' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
symbol.h:66 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'FSymbol' class. |
| General Analysis |
note.h:53 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'NoteHead' class. |
| General Analysis |
note.h:212 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Note' class. |
| General Analysis |
stafftype.h:420 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'TabDurationSymbol' class. |
| General Analysis |
notifier.hpp:117 |
High |
V591 |
Non-void function should return a value. |
| General Analysis |
clef.h:132 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Clef' class. |
| General Analysis |
text.h:24 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Text' class. |
| General Analysis |
tremolo.h:39 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Tremolo' class. |
| General Analysis |
sym.h:2768 |
Medium |
V690 |
The 'ScoreFont' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
midifile.cpp:367 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
midifile.cpp:621 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 621, 625, 638 |
| General Analysis |
midifile.cpp:57 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _division, status, sstatus, click, curPos. |
| General Analysis |
event.h:202 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _edata, _len, _metaType. |
| General Analysis |
event.h:203 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _metaType. |
| General Analysis |
stafftype.h:134 |
Medium |
V730 |
Not all members of a class are initialized inside the compiler generated constructor. Consider inspecting: defPitch, defYOffset. |
| General Analysis |
select.h:52 |
Medium |
V730 |
Not all members of a class are initialized inside the compiler generated constructor. Consider inspecting: staffStart, staffEnd, voice, system. |
| General Analysis |
instrument.h:45 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _pos. |
| General Analysis |
instrument.h:91 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: velocity, gateTime. |
| General Analysis |
xml.h:110 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: hasAccidental. |
| General Analysis |
xml.h:111 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: hasAccidental. |
| General Analysis |
xml.h:112 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: hasAccidental. |
| General Analysis |
xml.h:113 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: hasAccidental. |
| General Analysis |
textbase.h:185 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _lineSpacing. |
| General Analysis |
sym.h:2688 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _index, _advance. |
| General Analysis |
drumset.h:53 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: notehead, line, stemDirection, voice, shortcut. |
| General Analysis |
midifile.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
midiinstrument.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
beam.h:37 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Beam' class. |
| General Analysis |
event.cpp:417 |
Medium |
V522 |
There might be dereferencing of a potential null pointer 'info'. Check lines: 417, 410. |
| General Analysis |
changeMap.h:57 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: method, direction. |
| General Analysis |
changeMap.h:58 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: method, direction. |
| General Analysis |
event.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
msynthesizer.cpp:111 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
msynthesizer.cpp:265 |
Medium |
V649 |
There are two 'if' statements with identical conditional expressions. The first 'if' statement contains function return. This means that the second 'if' statement is senseless. Check lines: 262, 265. |
| General Analysis |
msynthesizer.cpp:366 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
msynthesizer.cpp:407 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
synthesizer.h:49 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _sampleRate. |
| General Analysis |
msynthesizer.cpp:29 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _sampleRate, effect1Buffer, effect2Buffer. |
| General Analysis |
msynthesizer.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
synthesizergui.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
fluid.h:668 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: data. |
| General Analysis |
chan.cpp:149 |
High |
V576 |
Incorrect format. A different number of actual arguments is expected while calling 'debug' function. Expected: 2. Present: 5. |
| General Analysis |
chan.cpp:171 |
High |
V576 |
Incorrect format. A different number of actual arguments is expected while calling 'debug' function. Expected: 2. Present: 4. |
| General Analysis |
chan.cpp:6 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
conv.cpp:6 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
dsp.cpp:6 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
fluid.cpp:679 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
fluid.cpp:685 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
fluid.cpp:73 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: sample_rate, _masterTuning, _tuning, _state, sfont_id, noteid. |
| General Analysis |
fluid.cpp:6 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
fluidgui.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
gen.cpp:6 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
mod.cpp:274 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 274, 277 |
| General Analysis |
mod.cpp:250 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 250, 253 |
| General Analysis |
mod.cpp:6 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
sfont.cpp:209 |
Medium |
V560 |
A part of conditional expression is always true: (sample != 0). |
| General Analysis |
sfont.cpp:703 |
Medium |
V641 |
The size of the '& idlist' buffer is not a multiple of the element size of the type 'unsigned int'. |
| General Analysis |
sfont.cpp:732 |
High |
V570 |
The 'var->size' variable is assigned to itself. |
| General Analysis |
sfont.cpp:743 |
Medium |
V618 |
It's dangerous to call the 'critical' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sfont.cpp:781 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sfont.cpp:906 |
Medium |
V557 |
Array overrun is possible. The '1' index is pointing beyond array bound. |
| General Analysis |
sfont.cpp:1072 |
High |
V773 |
The exception was thrown without releasing the 'z' pointer. A memory leak is possible. |
| General Analysis |
sfont.cpp:1310 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sfont.cpp:1316 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sfont.cpp:1320 |
Medium |
V774 |
The 'z' pointer was used after the memory was released. |
| General Analysis |
sfont.cpp:1325 |
Medium |
V774 |
The 'z' pointer was used after the memory was released. |
| General Analysis |
sfont.cpp:1329 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sfont.cpp:1585 |
Medium |
V774 |
The 'z' pointer was used after the memory was released. |
| General Analysis |
sfont.cpp:1590 |
Medium |
V774 |
The 'z' pointer was used after the memory was released. |
| General Analysis |
sfont.cpp:50 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _id. |
| General Analysis |
sfont.cpp:9 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
sfont3.cpp:21 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sfont3.cpp:27 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
voice.cpp:1087 |
Medium |
V1051 |
Consider checking for misprints. It's possible that the 'modlfo_dur' should be checked here. |
| General Analysis |
voice.cpp:6 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
dynamic.h:31 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Dynamic' class. |
| General Analysis |
layoutbreak.h:27 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'LayoutBreak' class. |
| General Analysis |
lyrics.h:27 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Lyrics' class. |
| General Analysis |
lyrics.h:111 |
Medium |
V690 |
The 'LyricsLine' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
lyrics.h:111 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'LyricsLine' class. |
| General Analysis |
lyrics.h:139 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'LyricsLineSegment' class. |
| General Analysis |
IntervalTree.h:71 |
High |
V630 |
The 'malloc' function is used to allocate memory for an array of objects which are classes containing constructors and destructors. |
| General Analysis |
IntervalTree.h:72 |
Medium |
V522 |
There might be dereferencing of a potential null pointer 'left'. Check lines: 72, 71. |
| General Analysis |
IntervalTree.h:84 |
Medium |
V794 |
The assignment operator should be protected from the case of 'this == &other'. |
| General Analysis |
mscoreview.h:38 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _score. |
| General Analysis |
accidental.h:65 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Accidental' class. |
| General Analysis |
icon.h:26 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Icon' class. |
| General Analysis |
timesig.h:41 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'TimeSig' class. |
| General Analysis |
chord.h:55 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Chord' class. |
| General Analysis |
notedot.h:27 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'NoteDot' class. |
| General Analysis |
harmony.h:72 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Harmony' class. |
| General Analysis |
fret.h:122 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'FretDiagram' class. |
| General Analysis |
pitchvalue.h:31 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: time, pitch, vibrato. |
| General Analysis |
fret.h:57 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: fingering. |
| General Analysis |
fret.h:58 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: fingering. |
| General Analysis |
undo.h:1232 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: le, e. |
| General Analysis |
undo.h:1400 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: oldC1, oldC2. |
| General Analysis |
accidental.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
arpeggio.h:30 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Arpeggio' class. |
| General Analysis |
segment.h:50 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Segment' class. |
| General Analysis |
arpeggio.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
ambitus.h:26 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Ambitus' class. |
| General Analysis |
measure.h:65 |
Medium |
V690 |
The 'Measure' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
measure.h:65 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Measure' class. |
| General Analysis |
system.h:79 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'System' class. |
| General Analysis |
ambitus.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
articulation.h:57 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Articulation' class. |
| General Analysis |
page.h:34 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Page' class. |
| General Analysis |
barline.h:57 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'BarLine' class. |
| General Analysis |
articulation.cpp:541 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 541, 544 |
| General Analysis |
articulation.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
audio.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
bagpembell.h:51 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'BagpipeEmbellishment' class. |
| General Analysis |
bagpembell.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
marker.h:27 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Marker' class. |
| General Analysis |
stafflines.h:23 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'StaffLines' class. |
| General Analysis |
fermata.h:33 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Fermata' class. |
| General Analysis |
image.h:28 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Image' class. |
| General Analysis |
barline.cpp:1436 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
barline.cpp:1453 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
barline.cpp:1456 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
barline.cpp:281 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: y1, y2. |
| General Analysis |
barline.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
tuplet.h:37 |
Medium |
V690 |
The 'Tuplet' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
tuplet.h:37 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Tuplet' class. |
| General Analysis |
stem.h:27 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Stem' class. |
| General Analysis |
hook.h:26 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Hook' class. |
| General Analysis |
stemslash.h:26 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'StemSlash' class. |
| General Analysis |
beam.cpp:1719 |
Medium |
V547 |
Expression 'i' is always true. |
| General Analysis |
beam.cpp:1769 |
Medium |
V560 |
A part of conditional expression is always true: !growDownGroup. |
| General Analysis |
beam.cpp:58 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _noSlope, _beamDist, _id, minMove, maxMove. |
| General Analysis |
beam.cpp:77 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _beamDist. |
| General Analysis |
beam.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
bend.h:27 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Bend' class. |
| General Analysis |
bend.cpp:44 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _fontSize, _fontStyle, _lineWidth, noteWidth. |
| General Analysis |
bend.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
box.h:103 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'HBox' class. |
| General Analysis |
textframe.h:27 |
Medium |
V690 |
The 'TBox' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
repeat.h:28 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'RepeatMeasure' class. |
| General Analysis |
stafftext.h:27 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'StaffText' class. |
| General Analysis |
box.cpp:489 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
box.cpp:527 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
box.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
bracket.h:29 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Bracket' class. |
| General Analysis |
bracket.cpp:529 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
bracket.cpp:30 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: ay1, _magx. |
| General Analysis |
bracket.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
bracketItem.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
breath.h:37 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Breath' class. |
| General Analysis |
breath.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
bsp.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
bsymbol.cpp:100 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
bsymbol.cpp:111 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
bsymbol.cpp:114 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
bsymbol.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
changeMap.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
slurtie.h:80 |
Medium |
V690 |
The 'SlurTieSegment' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
slur.h:25 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'SlurSegment' class. |
| General Analysis |
slur.h:55 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Slur' class. |
| General Analysis |
keysig.h:31 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'KeySig' class. |
| General Analysis |
check.cpp:241 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
check.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
tie.h:25 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'TieSegment' class. |
| General Analysis |
tie.h:60 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Tie' class. |
| General Analysis |
glissando.h:33 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'GlissandoSegment' class. |
| General Analysis |
glissando.h:49 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Glissando' class. |
| General Analysis |
chordline.h:34 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'ChordLine' class. |
| General Analysis |
ledgerline.h:31 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'LedgerLine' class. |
| General Analysis |
fingering.h:24 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Fingering' class. |
| General Analysis |
chord.cpp:1183 |
Medium |
V728 |
An excessive check can be simplified. The '||' operator is surrounded by opposite expressions '!st' and 'st'. |
| General Analysis |
chord.cpp:1529 |
Medium |
V1004 |
The 'hook()' pointer was used unsafely after it was verified against nullptr. Check lines: 1523, 1529. |
| General Analysis |
chord.cpp:2495 |
Medium |
V519 |
The '_crossMeasure' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 2490, 2495. |
| General Analysis |
chord.cpp:3099 |
High |
V557 |
Array underrun is possible. The '-1' index is pointing beyond array bound. |
| General Analysis |
chord.cpp:3122 |
High |
V557 |
Array underrun is possible. The '-1' index is pointing beyond array bound. |
| General Analysis |
chord.cpp:3146 |
High |
V557 |
Array underrun is possible. The '-1' index is pointing beyond array bound. |
| General Analysis |
chord.cpp:207 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _spaceLw, _spaceRw. |
| General Analysis |
chord.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
chordline.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
chordlist.cpp:233 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
chordlist.cpp:1145 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
chordlist.cpp:1418 |
High |
V595 |
The 'cl' pointer was utilized before it was verified against nullptr. Check lines: 1418, 1425. |
| General Analysis |
chordlist.cpp:1825 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
chordlist.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
tempotext.h:29 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'TempoText' class. |
| General Analysis |
textlinebase.h:31 |
Medium |
V690 |
The 'TextLineBaseSegment' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
hairpin.h:41 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'HairpinSegment' class. |
| General Analysis |
hairpin.h:78 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Hairpin' class. |
| General Analysis |
figuredbass.h:81 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'FiguredBassItem' class. |
| General Analysis |
figuredbass.h:232 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'FiguredBass' class. |
| General Analysis |
systemtext.h:24 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'SystemText' class. |
| General Analysis |
rehearsalmark.h:24 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'RehearsalMark' class. |
| General Analysis |
chordrest.cpp:265 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
chordrest.cpp:633 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
chordrest.cpp:747 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
chordrest.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
clef.cpp:191 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 191, 194 |
| General Analysis |
clef.cpp:321 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
clef.cpp:424 |
Medium |
V560 |
A part of conditional expression is always false: !segm->parent(). |
| General Analysis |
clef.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
cleflist.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
ottava.h:71 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'OttavaSegment' class. |
| General Analysis |
ottava.h:89 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Ottava' class. |
| General Analysis |
trill.h:27 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'TrillSegment' class. |
| General Analysis |
trill.h:62 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Trill' class. |
| General Analysis |
pedal.h:26 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'PedalSegment' class. |
| General Analysis |
pedal.h:44 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Pedal' class. |
| General Analysis |
textline.h:26 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'TextLineSegment' class. |
| General Analysis |
textline.h:42 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'TextLine' class. |
| General Analysis |
volta.h:32 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'VoltaSegment' class. |
| General Analysis |
volta.h:48 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Volta' class. |
| General Analysis |
cmd.cpp:1749 |
Medium |
V547 |
Expression 'stringData' is always true. |
| General Analysis |
cmd.cpp:2322 |
Medium |
V560 |
A part of conditional expression is always true: cr. |
| General Analysis |
cmd.cpp:2361 |
Medium |
V560 |
A part of conditional expression is always true: cr. |
| General Analysis |
cmd.cpp:2774 |
Medium |
V728 |
An excessive check can be simplified. The '||' operator is surrounded by opposite expressions '!m->hasVoice(j)' and 'm->hasVoice(j)'. |
| General Analysis |
cmd.cpp:2947 |
Medium |
V560 |
A part of conditional expression is always true: startSegment. |
| General Analysis |
cmd.cpp:3024 |
Medium |
V1048 |
The 'p.fret' variable was assigned the same value. |
| General Analysis |
cmd.cpp:3767 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
cmd.cpp:3930 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
cmd.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
connector.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
drumset.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
duration.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
durationtype.cpp:77 |
Medium |
V793 |
It is odd that the result of the 'ticks / t' statement is a part of the condition. Perhaps, this statement should have been compared with something else. |
| General Analysis |
durationtype.cpp:175 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 175, 193 |
| General Analysis |
durationtype.cpp:185 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 185, 188 |
| General Analysis |
durationtype.cpp:287 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
durationtype.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
dynamic.cpp:304 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
dynamic.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
iname.h:28 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'InstrumentName' class. |
| General Analysis |
instrchange.h:25 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'InstrumentChange' class. |
| General Analysis |
edit.cpp:439 |
Medium |
V547 |
Expression 'staffIdx < 0' is always true. |
| General Analysis |
edit.cpp:1971 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
edit.cpp:2138 |
Medium |
V547 |
Expression 'createEndBar' is always false. |
| General Analysis |
edit.cpp:2263 |
Medium |
V560 |
A part of conditional expression is always true: s1. |
| General Analysis |
edit.cpp:2263 |
Medium |
V571 |
Recurring check. The 's1' condition was already verified in line 2261. |
| General Analysis |
edit.cpp:3565 |
Medium |
V547 |
Expression '!ns' is always true. |
| General Analysis |
edit.cpp:4396 |
Medium |
V560 |
A part of conditional expression is always true: et != ElementType::NOTE. |
| General Analysis |
edit.cpp:4407 |
Medium |
V560 |
A part of conditional expression is always true: et != ElementType::SYSTEM_TEXT. |
| General Analysis |
edit.cpp:4761 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
range.h:45 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _track. |
| General Analysis |
range.h:85 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _first, _last. |
| General Analysis |
edit.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
jump.h:31 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Jump' class. |
| General Analysis |
noteline.h:26 |
Medium |
V690 |
The 'NoteLine' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
noteline.h:26 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'NoteLine' class. |
| General Analysis |
ossia.h:25 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Ossia' class. |
| General Analysis |
spacer.h:33 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Spacer' class. |
| General Analysis |
staffstate.h:32 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'StaffState' class. |
| General Analysis |
sticking.h:32 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Sticking' class. |
| General Analysis |
measurenumber.h:24 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'MeasureNumber' class. |
| General Analysis |
tremolobar.h:29 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'TremoloBar' class. |
| General Analysis |
systemdivider.h:25 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'SystemDivider' class. |
| General Analysis |
stafftypechange.h:26 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'StaffTypeChange' class. |
| General Analysis |
letring.h:26 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'LetRingSegment' class. |
| General Analysis |
letring.h:41 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'LetRing' class. |
| General Analysis |
vibrato.h:27 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'VibratoSegment' class. |
| General Analysis |
vibrato.h:54 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'Vibrato' class. |
| General Analysis |
palmmute.h:26 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'PalmMuteSegment' class. |
| General Analysis |
palmmute.h:44 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'PalmMute' class. |
| General Analysis |
element.cpp:408 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
element.cpp:415 |
Medium |
V547 |
Expression 'parent()->parent()' is always true. |
| General Analysis |
element.cpp:538 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
element.cpp:622 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
element.cpp:1121 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
element.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
elementmap.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
excerpt.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
fermata.cpp:106 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
fermata.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
fifo.h:42 |
Medium |
V730 |
It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: maxCount. |
| General Analysis |
fifo.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
figuredbass.cpp:1329 |
High |
V773 |
The function was exited without releasing the 'pItem' pointer. A memory leak is possible. |
| General Analysis |
figuredbass.cpp:1652 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
figuredbass.cpp:67 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: textWidth. |
| General Analysis |
figuredbass.cpp:976 |
Medium |
V730 |
It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: _printedLineLength. |
| General Analysis |
figuredbass.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
fingering.cpp:164 |
Medium |
V547 |
Expression 'above' is always true. |
| General Analysis |
fingering.cpp:200 |
Medium |
V547 |
Expression 'above' is always false. |
| General Analysis |
fingering.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
fret.cpp:468 |
Medium |
V547 |
Expression '!parent()' is always false. |
| General Analysis |
fret.cpp:473 |
Medium |
V560 |
A part of conditional expression is always true: _harmony->parent(). |
| General Analysis |
fret.cpp:1133 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
fret.cpp:1145 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
fret.cpp:1171 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
fret.cpp:49 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: stringLw, nutLw, stringDist, fretDist, _numPos. |
| General Analysis |
fret.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
glissando.cpp:155 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _glissandoType, _glissandoStyle, _fontSize, _showText, _playGlissando, _fontStyle. |
| General Analysis |
glissando.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
groups.cpp:251 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
groups.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
hairpin.cpp:60 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: drawCircledTip, circledTipRadius. |
| General Analysis |
hairpin.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
harmony.cpp:130 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
harmony.cpp:683 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
harmony.cpp:1016 |
Medium |
V547 |
Expression 'cl' is always true. |
| General Analysis |
harmony.cpp:1845 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
harmony.cpp:152 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _rootSpelling, _baseSpelling, _rootRenderCase, _baseRenderCase. |
| General Analysis |
harmony.cpp:167 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _rootSpelling, _baseSpelling, _rootRenderCase, _baseRenderCase. |
| General Analysis |
harmony.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
hook.cpp:25 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _hookType. |
| General Analysis |
hook.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
icon.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
image.cpp:133 |
High |
V595 |
The 'score()' pointer was utilized before it was verified against nullptr. Check lines: 133, 162. |
| General Analysis |
image.cpp:269 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
image.cpp:270 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
image.cpp:306 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
image.cpp:318 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
image.cpp:343 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
image.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
imageStore.cpp:169 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
imageStore.cpp:181 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
imageStore.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
iname.cpp:66 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
iname.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
input.cpp:145 |
Medium |
V728 |
An excessive check can be simplified. The '(A && !B) || (!A && B)' expression is equivalent to the 'bool(A) != bool(B)' expression. |
| General Analysis |
input.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
instrchange.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
instrtemplate.h:48 |
Medium |
V690 |
The 'InstrumentTemplate' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
instrtemplate.cpp:111 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
instrtemplate.cpp:496 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
instrtemplate.cpp:533 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
instrtemplate.cpp:561 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
instrtemplate.cpp:598 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
instrtemplate.cpp:644 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
instrtemplate.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
instrument.cpp:131 |
Medium |
V794 |
The assignment operator should be protected from the case of 'this == &i'. |
| General Analysis |
instrument.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
interval.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
joinMeasure.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
jump.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
key.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
keylist.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
keysig.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
lasso.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
layout.cpp:214 |
Medium |
V793 |
It is odd that the result of the 'upVoices + downVoices' statement is a part of the condition. Perhaps, this statement should have been compared with something else. |
| General Analysis |
layout.cpp:508 |
Medium |
V728 |
An excessive check can be simplified. The '(A && !B) || (!A && B)' expression is equivalent to the 'bool(A) != bool(B)' expression. |
| General Analysis |
layout.cpp:1904 |
High |
V789 |
Iterators for the 'oldList' container, used in the range-based for loop, become invalid upon the call of the 'erase' function. |
| General Analysis |
layout.cpp:3287 |
Medium |
V1004 |
The 'lc.curMeasure' pointer was used unsafely after it was verified against nullptr. Check lines: 3276, 3287. |
| General Analysis |
layout.cpp:3443 |
High |
V595 |
The 'lc.prevMeasure' pointer was utilized before it was verified against nullptr. Check lines: 3443, 3464. |
| General Analysis |
layout.cpp:3829 |
Medium |
V547 |
Expression 'useRange' is always false. |
| General Analysis |
layout.cpp:3830 |
Medium |
V547 |
Expression 'useRange' is always false. |
| General Analysis |
layout.h:51 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: systemOldMeasure, pageOldMeasure. |
| General Analysis |
layout.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
layoutbreak.cpp:32 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _pause, _startWithLongNames, _startWithMeasureOne. |
| General Analysis |
layoutbreak.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
layoutlinear.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
ledgerline.cpp:27 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _width, _len. |
| General Analysis |
ledgerline.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
letring.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
line.cpp:314 |
Medium |
V547 |
Expression 'ls' is always false. |
| General Analysis |
line.cpp:956 |
Medium |
V547 |
Expression '!endElement()' is always true. |
| General Analysis |
line.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
location.cpp:215 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
location.cpp:270 |
High |
V501 |
There are identical sub-expressions to the left and to the right of the '-' operator: graceIndex(end) - graceIndex(end) |
| General Analysis |
location.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
lyrics.cpp:169 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
lyrics.cpp:191 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
textedit.h:32 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: startUndoIdx. |
| General Analysis |
lyrics.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
lyricsline.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
marker.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
mcursor.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
measure.cpp:90 |
Medium |
V690 |
The 'MStaff' class implements a copy constructor, but lacks the copy assignment operator. It is dangerous to use such a class. |
| General Analysis |
measure.cpp:832 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
measure.cpp:1229 |
Medium |
V668 |
There is no sense in testing the 'ots' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. |
| General Analysis |
measure.cpp:1668 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
measure.cpp:1409 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 1409, 1414 |
| General Analysis |
measure.cpp:1957 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
measure.cpp:3215 |
Medium |
V1004 |
The 'seg' pointer was used unsafely after it was verified against nullptr. Check lines: 3214, 3215. |
| General Analysis |
measure.cpp:3678 |
Medium |
V547 |
Expression 'clefSeg' is always true. |
| General Analysis |
measure.cpp:3678 |
High |
V571 |
Recurring check. The 'if (clefSeg)' condition was already verified in line 3677. |
| General Analysis |
measure.cpp:225 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _noMode. |
| General Analysis |
measure.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
measurebase.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
measurenumber.cpp:66 |
High |
V595 |
The 'staff()' pointer was utilized before it was verified against nullptr. Check lines: 66, 67. |
| General Analysis |
measurenumber.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
midimapping.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
mscore.cpp:327 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
mscore.cpp:337 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
mscore.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
mscoreview.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
navigate.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
note.cpp:1037 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
note.cpp:1060 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
note.cpp:1080 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
note.cpp:1154 |
High |
V595 |
The 'chord()' pointer was utilized before it was verified against nullptr. Check lines: 1154, 1158. |
| General Analysis |
note.cpp:1282 |
Medium |
V547 |
Expression 'staff()' is always true. |
| General Analysis |
note.cpp:2517 |
Medium |
V1004 |
The 'seg' pointer was used unsafely after it was verified against nullptr. Check lines: 2507, 2517. |
| General Analysis |
note.cpp:3084 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 3084, 3097 |
| General Analysis |
note.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
notedot.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
noteentry.cpp:185 |
Medium |
V547 |
Expression '!addFlag' is always true. |
| General Analysis |
noteentry.cpp:410 |
Medium |
V1004 |
The 'cr' pointer was used unsafely after it was verified against nullptr. Check lines: 345, 410. |
| General Analysis |
noteentry.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
noteevent.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
noteline.cpp:18 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _startNote, _endNote. |
| General Analysis |
noteline.cpp:24 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _startNote, _endNote. |
| General Analysis |
noteline.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
ossia.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
ottava.cpp:212 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _numbersOnly. |
| General Analysis |
ottava.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
page.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
palmmute.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
part.cpp:457 |
Medium |
V796 |
It is possible that 'break' statement is missing in switch statement. |
| General Analysis |
part.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
paste.cpp:425 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
paste.cpp:711 |
Medium |
V547 |
Expression 'done' is always false. |
| General Analysis |
paste.cpp:722 |
Medium |
V547 |
Expression 'done' is always false. |
| General Analysis |
paste.cpp:755 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
paste.cpp:815 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
paste.cpp:821 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
paste.cpp:890 |
Medium |
V668 |
There is no sense in testing the 'nextSegm' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. |
| General Analysis |
paste.cpp:943 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
paste.cpp:1019 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
paste.cpp:1038 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
paste.cpp:1052 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
paste.cpp:1067 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
paste.cpp:1105 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
paste.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
pedal.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
pitch.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
pitchspelling.cpp:642 |
Medium |
V1004 |
The 'n' pointer was used unsafely after it was verified against nullptr. Check lines: 636, 642. |
| General Analysis |
pitchspelling.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
pos.cpp:96 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 96, 99 |
| General Analysis |
pos.cpp:434 |
Medium |
V1048 |
The '_lenTick' variable was assigned the same value. |
| General Analysis |
pos.cpp:548 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 548, 551 |
| General Analysis |
pos.cpp:59 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _frame. |
| General Analysis |
pos.cpp:71 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _frame. |
| General Analysis |
pos.cpp:81 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _tick. |
| General Analysis |
pos.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
property.cpp:485 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
property.cpp:546 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
property.cpp:557 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
property.cpp:569 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
property.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
range.cpp:794 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
range.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
read114.cpp:386 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read114.cpp:402 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read114.cpp:576 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read114.cpp:1539 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read114.cpp:2122 |
High |
V522 |
Dereferencing of the null pointer 'segment' might take place. |
| General Analysis |
read114.cpp:2781 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read114.cpp:3018 |
High |
V773 |
Visibility scope of the 'beam' pointer was exited without releasing the memory. A memory leak is possible. |
| General Analysis |
read114.cpp:3176 |
Medium |
V1048 |
The 'first' variable was assigned the same value. |
| General Analysis |
read114.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
read206.cpp:485 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:498 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:631 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:798 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:1515 |
Medium |
V774 |
The 'sp' pointer was used after the memory was released. |
| General Analysis |
read206.cpp:1631 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:1647 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:1954 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:2059 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:2628 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:2668 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:2786 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:3644 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:3828 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read206.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
read301.cpp:193 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
read301.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
rehearsalmark.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
rendermidi.cpp:941 |
Medium |
V636 |
The '((i - lastPointTick) * 60) / delta' expression was implicitly cast from 'int' type to 'double' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;. |
| General Analysis |
rendermidi.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
repeat.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
repeatlist.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
rest.cpp:557 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 557, 560, 563 |
| General Analysis |
rest.cpp:542 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 542, 545 |
| General Analysis |
rest.cpp:845 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
rest.cpp:713 |
Medium |
V524 |
It is odd that the body of 'downLine' function is fully equivalent to the body of 'upLine' function. |
| General Analysis |
rest.cpp:39 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _mmWidth. |
| General Analysis |
rest.cpp:46 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _mmWidth. |
| General Analysis |
rest.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
revisions.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
score.cpp:725 |
Medium |
V1048 |
The '* seg' variable was assigned the same value. |
| General Analysis |
score.cpp:3695 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
score.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
scoreElement.cpp:824 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
scoreElement.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
Diff.hpp:397 |
Medium |
V547 |
Expression 'isMiddle' is always true. |
| General Analysis |
Diff.hpp:409 |
Medium |
V547 |
Expression 'isMiddle' is always true. |
| General Analysis |
scorediff.cpp:613 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
Diff3.hpp:63 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: conflict. |
| General Analysis |
scorediff.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
scorefile.cpp:926 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
scorefile.cpp:1020 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
scorefile.cpp:1254 |
Medium |
V1051 |
Consider checking for misprints. It's possible that the 'voiceTagWritten' should be checked here. |
| General Analysis |
scorefile.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
segment.cpp:644 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 644, 703 |
| General Analysis |
segment.cpp:744 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
segment.cpp:2089 |
High |
V595 |
The 'ns' pointer was utilized before it was verified against nullptr. Check lines: 2089, 2099. |
| General Analysis |
segment.cpp:113 |
Medium |
V730 |
It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: _stretch. |
| General Analysis |
segment.cpp:120 |
Medium |
V730 |
It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: _stretch. |
| General Analysis |
segment.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
segmentlist.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
select.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
shadownote.h:30 |
High |
V1052 |
Declaring virtual methods in a class marked as 'final' is pointless. Consider inspecting the 'ShadowNote' class. |
| General Analysis |
shadownote.cpp:46 |
Medium |
V519 |
The '_notehead' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 44, 46. |
| General Analysis |
shadownote.cpp:60 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 60, 69, 72 |
| General Analysis |
shadownote.cpp:63 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 63, 66 |
| General Analysis |
shadownote.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
shape.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
sig.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
skyline.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
slur.cpp:194 |
Medium |
V547 |
Expression 'scr' is always true. |
| General Analysis |
slur.cpp:204 |
Medium |
V547 |
Expression 'ecr' is always true. |
| General Analysis |
slur.cpp:606 |
Medium |
V1051 |
Consider checking for misprints. It's possible that the 'sp->system2' should be checked here. |
| General Analysis |
slur.cpp:615 |
Medium |
V560 |
A part of conditional expression is always true: sp->system1. |
| General Analysis |
slur.cpp:1027 |
Medium |
V560 |
A part of conditional expression is always false: startCR() == 0. |
| General Analysis |
slur.cpp:1152 |
Medium |
V560 |
A part of conditional expression is always false: startCR() == 0. |
| General Analysis |
slur.cpp:1238 |
Medium |
V560 |
A part of conditional expression is always true: i != 0. |
| General Analysis |
slur.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
slurtie.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
spacer.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
spanner.cpp:816 |
Medium |
V592 |
The expression was enclosed by parentheses twice: ((expression)). One pair of parentheses is unnecessary or misprint is present. |
| General Analysis |
spanner.cpp:1209 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
spanner.cpp:1375 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
spanner.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
spannermap.cpp:22 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: tree. |
| General Analysis |
spannermap.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
splitMeasure.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
staff.cpp:1304 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
staff.cpp:1374 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
staff.cpp:1407 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
staff.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
stafflines.cpp:40 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: lw. |
| General Analysis |
stafflines.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
staffstate.cpp:118 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 118, 125, 132 |
| General Analysis |
staffstate.cpp:28 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: lw. |
| General Analysis |
staffstate.cpp:35 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _staffStateType, lw. |
| General Analysis |
staffstate.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
stafftext.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
stafftextbase.cpp:199 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
stafftextbase.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
stafftype.cpp:273 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
stafftype.cpp:1185 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
stafftype.cpp:1381 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1382 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1383 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1384 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1385 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1386 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1387 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1388 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1389 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1390 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1391 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1394 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1395 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1396 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:1397 |
Medium |
V601 |
The bool type is implicitly cast to the double type. Inspect the 14th argument. |
| General Analysis |
stafftype.cpp:833 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _beamLevel, _repeat. |
| General Analysis |
stafftype.cpp:843 |
Medium |
V730 |
It is possible that not all members of a class are initialized inside the constructor. Consider inspecting: _beamLevel, _repeat. |
| General Analysis |
stafftype.cpp:852 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _beamLength, _beamLevel, _beamGrid, _repeat. |
| General Analysis |
stafftype.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
stafftypechange.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
stafftypelist.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
stem.cpp:155 |
High |
V595 |
The 'chord()' pointer was utilized before it was verified against nullptr. Check lines: 155, 161. |
| General Analysis |
stem.cpp:38 |
Medium |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _lineWidth, _userLen. |
| General Analysis |
stem.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
stemslash.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
sticking.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
stringdata.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
style.cpp:2303 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
style.cpp:2326 |
Medium |
V618 |
It's dangerous to call the 'warning' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
style.cpp:2586 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
style.cpp:2597 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
style.cpp:2609 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
style.cpp:2786 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
style.cpp:2470 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _precomputedValues. |
| General Analysis |
style.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
sym.cpp:5705 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sym.cpp:5852 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sym.cpp:5925 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sym.cpp:5951 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sym.cpp:6241 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sym.cpp:6243 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sym.cpp:6244 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sym.cpp:6282 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
sym.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
symbol.cpp:125 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
symbol.cpp:127 |
Medium |
V1048 |
The 'symId' variable was assigned the same value. |
| General Analysis |
symbol.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
synthesizerstate.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
system.cpp:627 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
system.cpp:910 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
system.cpp:844 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 844, 873 |
| General Analysis |
system.cpp:968 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
system.cpp:926 |
Medium |
V1037 |
Two or more case-branches perform the same actions. Check lines: 926, 940 |
| General Analysis |
system.cpp:63 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _distance. |
| General Analysis |
system.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
systemdivider.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
systemtext.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
tempo.cpp:22 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: time. |
| General Analysis |
tempo.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
tempotext.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
text.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
textbase.cpp:1319 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
textbase.cpp:1402 |
High |
V501 |
There are identical sub-expressions to the left and to the right of the '-' operator: i - i |
| General Analysis |
textbase.cpp:2307 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
textbase.cpp:2720 |
Medium |
V1001 |
The 'pen' variable is assigned but is not used by the end of the function. |
| General Analysis |
textbase.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
textedit.cpp:549 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
textedit.cpp:563 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
textedit.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
textframe.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
textline.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
textlinebase.cpp:263 |
High |
V501 |
There are identical sub-expressions 'textLineBase()->beginHookType() == HookType::NONE' to the left and to the right of the '&&' operator. |
| General Analysis |
textlinebase.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
tie.cpp:282 |
Medium |
V1004 |
The 'staff()' pointer was used unsafely after it was verified against nullptr. Check lines: 240, 282. |
| General Analysis |
tie.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
timesig.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
transpose.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
tremolo.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
tremolobar.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
trill.cpp:405 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
trill.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
tuplet.cpp:911 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
tuplet.cpp:990 |
Medium |
V618 |
It's dangerous to call the 'debug' function in such a manner, as the line being passed could contain format specification. The example of the safe code: printf("%s", str); |
| General Analysis |
tuplet.cpp:51 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _direction, _numberType, _bracketType, _id. |
| General Analysis |
tuplet.cpp:61 |
High |
V730 |
Not all members of a class are initialized inside the constructor. Consider inspecting: _id. |
| General Analysis |
tuplet.cpp:8 |
High |
V1042 |
This file is marked with copyleft license, which requires you to open the derived source code. |
| General Analysis |
undo.cpp:147 |
Medium |
V1044 |
Loop break conditions do not depend on the number of iterations. |
| General Analysis |
undo.cpp:161 |
Medium |
V1044 |
Loop break conditions do not depend on the number of iterations. |
| General Analysis |
undo.cpp:237 |
Medium |
V618 |