Use the new optional template to simplify.

Thanks Yuriy!

diff --git a/src/Buffer.cpp b/src/Buffer.cpp
index c0a133fadd0bdf6ca12520d8ed0ade09d0625429..b62228c415b867604a4a07bbc0aa4038e2744519 100644 (file)
--- a/src/Buffer.cpp
+++ b/src/Buffer.cpp
@@ -97,6 +97,7 @@
 #include "support/gzstream.h"
 #include "support/lstrings.h"
 #include "support/mutex.h"
+#include "support/optional.h"
 #include "support/os.h"
 #include "support/Package.h"
 #include "support/PathChanger.h"
@@ -215,14 +216,11 @@ public:
                typedef map<DocIterator, MacroDefinition> MacroDefList;
                typedef map<docstring, MacroDefList> MacroMap;
 
-               bool haveMacro(docstring const & name) const
-                       { return macro_map_.count(name) != 0; }
-               
-               // Will assert if there is no such macro, so call haveMacro first!
-               MacroDefList const & getMacroDefinitions(docstring const & name) const
+               optional<MacroDefList> getMacroDefinitions(docstring const & name) const
                {
                        MacroMap::const_iterator it = macro_map_.find(name);
-                       LBUFERR(it != macro_map_.end());
+                       if (it == macro_map_.end())
+                               return {};
                        return it->second;
                }
 
@@ -3635,9 +3633,10 @@ MacroData const * Buffer::Impl::getBufferMacro(docstring const & name,
        MacroData const * bestData = nullptr;
 
        // find macro definitions for name
-       if (macro_table.haveMacro(name)) {
-               MacroTable::MacroDefList const & mdl = 
-                       macro_table.getMacroDefinitions(name);
+       if (optional<MacroTable::MacroDefList> m =
+                       macro_table.getMacroDefinitions(name))
+       {
+               MacroTable::MacroDefList const & mdl = m.value();
                // find last definition in front of pos
                MacroTable::MacroDefList::const_iterator it = 
                                greatest_below(mdl, pos);