Modified: lyx-devel/branches/BRANCH_1_6_X/src/insets/InsetCommandParams.cpp
==============================================================================
--- lyx-devel/branches/BRANCH_1_6_X/src/insets/InsetCommandParams.cpp   Tue Mar 
 2 16:13:00 2010        (r33610)
+++ lyx-devel/branches/BRANCH_1_6_X/src/insets/InsetCommandParams.cpp   Tue Mar 
 2 16:19:12 2010        (r33611)
@@ -324,8 +324,9 @@

  bool InsetCommandParams::writeEmptyOptional(ParamInfo::const_iterator ci) 
const
  {
-       if (!ci->isOptional())
+       if (!ci->isOptional()) {
                LASSERT(false, /**/);
+       }
        ++ci; // we want to start with the next one
        ParamInfo::const_iterator end = info_.end();
        for (; ci != end; ++ci) {
@@ -394,8 +395,9 @@
        ParamInfo::const_iterator it =
                find_if(info_.begin(), info_.end(),
                        not1(mem_fun_ref(&ParamInfo::ParamData::isOptional)));
-       if (it == info_.end())
+       if (it == info_.end()) {
                LASSERT(false, return docstring());
+       }
        return (*this)[it->name()];
  }

I meant to commit that next. Just a warning fix due to the "else" embedded in LASSERT.

rh

Reply via email to