Edwin Leuven <[EMAIL PROTECTED]> writes:

| Anyway, thanks to you guys find/replace is now guii (over here).
| I've got the  
| following dilemma on which I need your advice. Find/replace uses generalized 
| copies of the members FindBackward(), FindForward(), and IsStringInText() 
| from the LyXText class. What needs to be added for the find/replace
| dialog is  
| casesensitive searching and word matching. I don't want to keep two
| copies of  
| these functions around (like it is now) so I've added casesensitive
| searching  
| and word matching to the LyXText members.
| 
| My question is: can I move these members out of LyXText?

Yes, please.

| As far as I can see there is no need for them to be in LyXText
| (which is huge  
| already) and since they are basically only used by find/replace it
| seems more  
| appropriate to put the find/replace functionality together in lyxfind.[Ch] 
| (and possibly in a LyXFind namespace).
| 
| Any thoughts?

go for it.

        Lgb

Reply via email to