Member alg_error_syntax

Maybe we should only set alg_error_syntax when there is a real problem with the file as opposed to when there is some warning message for the user

Member lmms::gui::AutomationEditor::paintEvent (QPaintEvent *pe) override

move this horizontal line drawing code into the same loop as the value ticks?

Member lmms::gui::ClipView::fixedClips ()

In what circumstance are they fixed?

Member lmms::gui::ClipView::mouseMoveEvent (QMouseEvent *me) override

What does the final else case do here?

Member lmms::gui::PianoRoll::showVolTextFloat (volume_t vol, const QPoint &pos, int timeout=-1)

display velocity for MIDI-based instruments

Member lmms::gui::PianoView::contextMenuEvent (QContextMenuEvent *me) override

Is this right, or does this create the context menu?

Member lmms::gui::PianoView::focusOutEvent (QFocusEvent *) override

Is there supposed to be a parameter given here?

Member lmms::gui::PianoView::getKeyX (int key_num) const

Is this description of what the method does correct?

Member lmms::gui::PianoView::mouseMoveEvent (QMouseEvent *me) override

Paul Wayper thinks that this code should be refactored to reduce or remove the duplication between this, the mousePressEvent() and mouseReleaseEvent() methods.

Member lmms::gui::PianoView::PianoView (QWidget *parent)

Are the descriptions of the m_startkey and m_lastkey properties correct?

Member lmms::Plugin::Descriptor::SubPluginFeatures::Key::name

This is a bug, there should be a function in SubPluginFeatures (to get the name) instead

Member lmms::Track::getClip (std::size_t clipNum) -> Clip *

Reject Clip numbers less than zero.

If we create a Clip here, should we somehow attach it to the track?

Member lmms::Track::insertBar (const TimePos &pos)

If we stepped through this list last to first, and the list was in ascending order by Clip time, once we hit a Clip that was earlier than the insert time, we could fall out of the loop early.

Member lmms::Track::loadTrack (const QDomElement &element, bool presetMode)

Load the track height.

Member lmms::Track::saveTrack (QDomDocument &doc, QDomElement &element, bool presetMode)

Does this accurately describe the parameters? I think not!?

Save the track height