From: Steve Brokenshire <sbrokenshire@xestia.co.uk>
Date: Sat, 1 Sep 2018 20:47:13 +0000 (+0100)
Subject: odthelpbrowser: Remove unused variable and all code paths return values
X-Git-Tag: release-0.23~7
X-Git-Url: http://Server1/repobrowser/?p=xestiaab%2F.git;a=commitdiff_plain;h=cdba047c3835e54dcd0db567050d77a5740e8a52;hp=c167db0a2e9f20e7a3060064555a3596df4ca8ed

odthelpbrowser: Remove unused variable and all code paths return values
---

diff --git a/source/tools/odthelpbrowser/frmMain.cpp b/source/tools/odthelpbrowser/frmMain.cpp
index 193cbb0..f1048e5 100644
--- a/source/tools/odthelpbrowser/frmMain.cpp
+++ b/source/tools/odthelpbrowser/frmMain.cpp
@@ -23,7 +23,6 @@ frmMainADT( parent )
 	
 	wxTreeItemId lastItemAtLevel[9];
 	lastItemAtLevel[ODT::HelpTopicCurrentLevel::TOPIC_LEVEL1] = rootItem;
-	ODT::HelpTopicCurrentLevel previousLevel;
 	uint32_t itemIndex = 0;
 
 	for (auto tocItem : document->tocData)
diff --git a/source/tools/odthelpbrowser/odt.cpp b/source/tools/odthelpbrowser/odt.cpp
index 59f7dad..3ce4456 100644
--- a/source/tools/odthelpbrowser/odt.cpp
+++ b/source/tools/odthelpbrowser/odt.cpp
@@ -40,6 +40,8 @@ namespace ODT
 		xmlODTDoc = xmlReadMemory(odtDocument.c_str(), (int)odtDocument.size(), "noname.xml", NULL, 0);
 		if (!ProcessDocument(xmlODTDoc))
 			return false;
+
+		return true;
 	}
 	
 	bool ODT::ProcessDocument(xmlDocPtr document)
@@ -493,5 +495,7 @@ namespace ODT
 			return TOPIC_LEVEL3;
 		else if (styleTextIterator->second == "Contents_20_4")
 			return TOPIC_LEVEL4;
+
+		return TOPIC_LEVEL1;
 	}
 }
\ No newline at end of file