From 5e55cbfb3cce429c10bcae01075a8e52fb4c6b5b Mon Sep 17 00:00:00 2001
From: shun-iwasawa <shun.iwasawa@ghibli.jp>
Date: Dec 10 2020 06:41:34 +0000
Subject: fix crash on drawing empty level


---

diff --git a/toonz/sources/toonz/filmstrip.cpp b/toonz/sources/toonz/filmstrip.cpp
index 5c2824c..ee523d6 100644
--- a/toonz/sources/toonz/filmstrip.cpp
+++ b/toonz/sources/toonz/filmstrip.cpp
@@ -1643,7 +1643,8 @@ void Filmstrip::updateChooseLevelComboItems() {
     }
   }
 
-  m_chooseLevelCombo->addItem(tr("- No Current Level -"));
+  if (m_chooseLevelCombo->count() == 0)
+    m_chooseLevelCombo->addItem(tr("- No Current Level -"));
 
   // swap the list
   m_workingFrames.clear();
@@ -1657,7 +1658,8 @@ void Filmstrip::updateChooseLevelComboItems() {
 /*! synchronize the current index of combo to the current level
  */
 void Filmstrip::updateCurrentLevelComboItem() {
-  if (m_chooseLevelCombo->count() == 1) {
+  if (m_chooseLevelCombo->count() == 1 &&
+      m_chooseLevelCombo->itemText(0) == tr("- No Current Level -")) {
     m_chooseLevelCombo->setCurrentIndex(0);
     return;
   }
@@ -1666,6 +1668,10 @@ void Filmstrip::updateCurrentLevelComboItem() {
       TApp::instance()->getCurrentLevel()->getSimpleLevel();
   if (!currentLevel) {
     int noLevelIndex = m_chooseLevelCombo->findText(tr("- No Current Level -"));
+    if (noLevelIndex == -1) {
+      noLevelIndex = m_chooseLevelCombo->count();
+      m_chooseLevelCombo->addItem(tr("- No Current Level -"));
+    }
     m_chooseLevelCombo->setCurrentIndex(noLevelIndex);
     return;
   }
@@ -1673,11 +1679,18 @@ void Filmstrip::updateCurrentLevelComboItem() {
   for (int i = 0; i < m_levels.size(); i++) {
     if (currentLevel->getName() == m_levels[i]->getName()) {
       m_chooseLevelCombo->setCurrentIndex(i);
+      int noLevelIndex =
+          m_chooseLevelCombo->findText(tr("- No Current Level -"));
+      if (noLevelIndex != -1) m_chooseLevelCombo->removeItem(noLevelIndex);
       return;
     }
   }
 
   int noLevelIndex = m_chooseLevelCombo->findText(tr("- No Current Level -"));
+  if (noLevelIndex == -1) {
+    noLevelIndex = m_chooseLevelCombo->count();
+    m_chooseLevelCombo->addItem(tr("- No Current Level -"));
+  }
   m_chooseLevelCombo->setCurrentIndex(noLevelIndex);
 }