Resolves: fdo#51572 catch CLuceneError throws and extract the error message

I can't reproduce fdo#51572, but catching the exception in
HelpIndexer::indexDocuments should resolve it anyway and make it non-fatal.
Collect the error message for retrieval via HelpIndexer::getErrorMessage

Change-Id: Id557b9f5ff968c398f76969591f5ee765e56aa5a

l10ntools/source/help/HelpIndexer.cxx

@@ -34,7 +34,7 @@
 #include <rtl/ustrbuf.hxx>
 #include <osl/file.hxx>
 #include <osl/thread.h>
-
+#include <boost/scoped_ptr.hpp>
 #include <algorithm>
 
 #include "LuceneHelper.hxx"
@@ -51,26 +51,28 @@ HelpIndexer::HelpIndexer(rtl::OUString const &lang, rtl::OUString const &module,
     d_contentDir = srcDir + rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("/content"));
 }
 
-bool HelpIndexer::indexDocuments() {
-    if (!scanForFiles()) {
+bool HelpIndexer::indexDocuments()
+{
+    if (!scanForFiles())
         return false;
-    }
 
+    try
+    {
         rtl::OUString sLang = d_lang.getToken(0, '-');
         bool bUseCJK = sLang == "ja" || sLang == "ko" || sLang == "zh";
 
         // Construct the analyzer appropriate for the given language
-    lucene::analysis::Analyzer *analyzer;
+        boost::scoped_ptr<lucene::analysis::Analyzer> analyzer;
         if (bUseCJK)
-        analyzer = new lucene::analysis::LanguageBasedAnalyzer(L"cjk");
+            analyzer.reset(new lucene::analysis::LanguageBasedAnalyzer(L"cjk"));
         else
-        analyzer = new lucene::analysis::standard::StandardAnalyzer();
+            analyzer.reset(new lucene::analysis::standard::StandardAnalyzer());
 
         rtl::OUString ustrSystemPath;
         osl::File::getSystemPathFromFileURL(d_indexDir, ustrSystemPath);
 
         rtl::OString indexDirStr = rtl::OUStringToOString(ustrSystemPath, osl_getThreadTextEncoding());
-    lucene::index::IndexWriter writer(indexDirStr.getStr(), analyzer, true);
+        lucene::index::IndexWriter writer(indexDirStr.getStr(), analyzer.get(), true);
         //Double limit of tokens allowed, otherwise we'll get a too-many-tokens
         //exception for ja help. Could alternative ignore the exception and get
         //truncated results as per java-Lucene apparently
@@ -87,8 +89,13 @@ bool HelpIndexer::indexDocuments() {
 
         // Optimize the index
         writer.optimize();
+    }
+    catch (CLuceneError &e)
+    {
+        d_error = rtl::OUString::createFromAscii(e.what());
+        return false;
+    }
 
-    delete analyzer;
     return true;
 }
 

l10ntools/source/help/HelpIndexer_main.cxx

@@ -99,18 +99,10 @@ int main(int argc, char **argv) {
         rtl::OUString(module.c_str(), module.size(), osl_getThreadTextEncoding()),
         sDir, sDir);
 
-    try
-    {
     if (!indexer.indexDocuments()) {
         std::cerr << rtl::OUStringToOString(indexer.getErrorMessage(), osl_getThreadTextEncoding()).getStr()  << std::endl;
         return 2;
     }
-    }
-    catch (CLuceneError &e)
-    {
-        std::cerr << e.what() << std::endl;
-        return 2;
-    }
     return 0;
 }