Le 27/01/2016 18:53, Guillaume Munch a écrit :
commit c9f7ce0a7a6d48b9b60af92828654aa7547286b4
Author: Guillaume Munch <g...@lyx.org>
Date:   Fri Dec 11 02:15:52 2015 +0000

     New LFUN tabular-feature (#9794)

I re-read the patch rapidly and have a couple cleanup remarks.

--- a/src/insets/Inset.cpp
+++ b/src/insets/Inset.cpp
@@ -391,8 +391,6 @@ bool Inset::getStatus(Cursor &, FuncRequest const & cmd,
                // FIXME: Why don't we let the insets determine whether this
                // should be enabled or not ? Now we need this check for
                // the tabular features. (vfr)
-               if (cmd.getArg(0) == "tabular")
-                       return false;
                flag.setEnabled(true);
                return true;

I would think that the FIXME comment can be removed now.

+// function sets an object as defined in func_status.h:
+// states OK, Unknown, Disabled, On, Off.
+bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
+                             FuncStatus & status) const

The header is now FuncStatus.h

JMarc

Reply via email to