[Bug]Instrument list keeps jumping to GM1 Melodic bank after changing LSB about opn2bankeditor

[Wohlstand]

🤔
2020-11-28_23-50-57.mp4.zip

[freq-mod]

It's only bank 16 that does that, look:
ice_video_20201128-220621_edit_0.zip

[Wohlstand]

Didn't reproduced at me... maybe I should try this at my Win8.1 computer?

[Wohlstand]

BLAK MAGIK!!!
2020-11-29_00-17-23.mp4.zip

It reproduced on Windows build ran on Wine!

[Wohlstand]

Okay, that damn happens on Windows, because the list of banks gets be cleaned-up:

void BankEditor::reloadBanks()
{
    ui->bank_no->clear();
    int countOfBanks = 1;
    bool isDrum = isDrumsMode();
    if(isDrum)
        countOfBanks = ((m_bank.countDrums() - 1) / 128) + 1;
    else
        countOfBanks = ((m_bank.countMelodic() - 1) / 128) + 1;
    for(int i = 0; i < countOfBanks; i++)
    {
        ui->bank_no->addItem(QString(), i);
        refreshBankName(i);
    }
}

because the instrument names reloading call do calling the "setMelodic()" call that calls "reloadBanks()"

🤔

[Wohlstand]

And I know why: QListWidget item search by wildcard sux: I found a different way to iterate items inside of it, gonna to try...

[Wohlstand]

@freq-mod , just now I made a fix, please verify the work! 🦊

[Wohlstand]

// I feel, I need to apply the same fix to OPL3-BE too...

[Wohlstand]

Done at OPL3-BE too.

[freq-mod]

Looks like that bug is gone, thanks!