>>>>> "Juergen" == Juergen Spitzmueller <[EMAIL PROTECTED]> writes:

Juergen> Georg Baum wrote:
>> I see. I would also document the argument of forceDefaultParagraphs
>> in insetbase.h so that misunderstandings like in the current code
>> do not longer happen:
>> 
>> /// if this inset has paragraphs should they be output all as
>> default /// paragraphs with "Standard" layout? /// \p in is a cell
>> in case of tables. virtual bool forceDefaultParagraphs(InsetBase
>> const * in) const { return false; }

Juergen> Is it worth it to add the argument for documentation
Juergen> purposes? Attached is a patch with some mor FIXME's and the
Juergen> proposed documentation (without having added the argument).

What about redefining as

virtual bool forceDefaultParagraphs(idx_type cell =0)

Paragraph::forceDefaultParagraph would pass 0 (or no value), the
others would use the cursor.

Considering the low number of instance of this method, it seems both
easy and correct. Thoughts?

JMarc

Reply via email to