>From my interpretation, the FIXME in the attached patch (added in 2009
by Vincent) is now satisfied and we can remove the !empty_cell from
the condition. Is that correct or did I misunderstand something?
Thanks,
Scott
From 80a114b3f47b768a4763fac1b7f1408ed7433fe6 Mon Sep 17 00:00:00 2001
From: Scott Kostyshak <skost...@lyx.org>
Date: Fri, 7 Dec 2012 01:50:38 -0500
Subject: [PATCH 4/9] Address a FIXME in InsetTabular
The condition that was required is now met.
---
src/insets/InsetTabular.cpp | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/src/insets/InsetTabular.cpp b/src/insets/InsetTabular.cpp
index 4fe84f4..317e905 100644
--- a/src/insets/InsetTabular.cpp
+++ b/src/insets/InsetTabular.cpp
@@ -4032,10 +4032,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest
& cmd)
break;
}
- // FIXME: When we support the selection of an empty
cell, remove
- // the !empty_cell from this condition. For now we jump
to the next
- // cell if the current cell is empty.
- if (cur.result().dispatched() && !empty_cell)
+ if (cur.result().dispatched())
break;
}
--
1.7.9.5
