Andre Poenitz wrote:
> for (int i = 0; i < n; ++i) {
> InsetText * in = &tabular.getCellInset(i);
> if (inset == in) {
> @@ -451,11 +450,9 @@ bool InsetTabular::lockInsetInInset(Buff
> resetPos(bv);
> return true;
> }
> - if (in->getInsetFromID(id)) {
> - actcell = i;
> - in->dispatch(FuncRequest(bv,
> LFUN_INSET_EDIT));
> - return
> the_locking_inset->lockInsetInInset(bv, inset);
> - }
> + actcell = i;
> + in->dispatch(FuncRequest(bv, LFUN_INSET_EDIT));
> + return the_locking_inset->lockInsetInInset(bv,
> inset);
shouldn't this last line be
if (the_locking_inset->lockInsetInInset(bv, inset))
return true;
instead? If that is not needed, then you can get rid of the loop altoghether
...
Alfredo
