On 01/04/2010 06:37 PM, Tommaso Cucinotta wrote:
3) let the LFUN dispatch in lyxfind.cpp return the required information (match position and length) to the caller; for example, by means of a DispatchResult sub-class (any strong objection to adding a "DispatchResult &" to some of the existing ::dispatch() methods?

I think this is something we have been meaning to do anyway. At the moment, the DispatchResult argument is where it has been needed, or where JMarc (usually) suggested I should put it.

(I suspect this would allow for removing the cursor().disp_ field and related methods) Or, adding a "DispatchResult &" field to the FuncRequest class itself ?

I'd like to hear from JMarc as to which he thought was best before doing anything. We should definitely have one, consistent interface for these methods.

Richard

Reply via email to