]> git.lyx.org Git - lyx.git/blobdiff - src/lyxfind.C
somewhat clearer logic
[lyx.git] / src / lyxfind.C
index 9f474c5a36456a5b8c0f58c569ce1f4b0618ebf9..b5bd7fb7f4e1892118044fe6440f93bff84b9787 100644 (file)
@@ -27,8 +27,6 @@
 
 #include "frontends/Alert.h"
 
-#include "insets/insettext.h"
-
 #include "support/textutils.h"
 
 using lyx::support::lowercase;
@@ -47,61 +45,67 @@ class MatchString
 {
 public:
        MatchString(string const & str, bool cs, bool mw)
-               : str(str), cs(cs), mw(mw) {};
-// returns true if the specified string is at the specified position
+               : str(str), cs(cs), mw(mw)
+       {}
+
+       // returns true if the specified string is at the specified position
        bool operator()(Paragraph const & par, pos_type pos) const
        {                       
                string::size_type size = str.length();
                pos_type i = 0;
-               pos_type parsize = par.size();
+               pos_type const parsize = par.size();
                while ((pos + i < parsize)
                       && (string::size_type(i) < size)
                       && (cs ? (str[i] == par.getChar(pos + i))
                           : (uppercase(str[i]) == uppercase(par.getChar(pos + i))))) {
                        ++i;
                }
-               if (size == string::size_type(i)) {
-                       // if necessary, check whether string matches word
-                       if (!mw)
-                               return true;
-                       if ((pos <= 0 || !IsLetterCharOrDigit(par.getChar(pos - 1)))
-                           && (pos + pos_type(size) >= parsize
-                               || !IsLetterCharOrDigit(par.getChar(pos + size)))) {
-                               return true;
-                       }
+
+               if (size != string::size_type(i))
+                       return false;
+
+               // if necessary, check whether string matches word
+               if (mw) {
+                       if (pos > 0     && IsLetterCharOrDigit(par.getChar(pos - 1)))
+                               return false;
+                       if (pos + pos_type(size) < parsize
+                                       && IsLetterCharOrDigit(par.getChar(pos + size)));
+                               return false;
                }
-               return false;
+
+               return true;
        }
        
 private:
+       // search string
        string str;
+       // case sensitive
        bool cs;
+       // match whole words only
        bool mw;
 };
 
 
 bool findForward(PosIterator & cur, PosIterator const & end,
-                MatchString & match)
+                MatchString const & match)
 {
-       for (; cur != end && !match(*cur.pit(), cur.pos()); ++cur)
-               ;
-
-       return cur != end;
+       for (; cur != end; ++cur) {
+               if (match(*cur.pit(), cur.pos()))
+                       return true;
+       }
+       return false;
 }
 
 
 bool findBackwards(PosIterator & cur, PosIterator const & beg,
-                  MatchString & match)
+                  MatchString const & match)
 {
-       if (beg == cur)
-               return false;
-       do {
+       while (beg != cur) {
                --cur;
                if (match(*cur.pit(), cur.pos()))
-                       break;
-       } while (cur != beg);
-
-       return match(*cur.pit(), cur.pos());
+                       return true;
+       }
+       return false;
 }
 
 
@@ -110,10 +114,9 @@ bool findChange(PosIterator & cur, PosIterator const & end)
        for (; cur != end; ++cur) {
                if ((!cur.pit()->size() || !cur.at_end())
                    && cur.pit()->lookupChange(cur.pos()) != Change::UNCHANGED)
-                       break;
+                       return true;
        }
-       
-       return cur != end;
+       return false;
 }
 
 
@@ -130,26 +133,21 @@ bool searchAllowed(BufferView * bv, string const & str)
 
 
 
-bool find(BufferView * bv, string const & searchstr,
-         bool cs, bool mw, bool fw)
+bool find(BufferView * bv, string const & searchstr, bool cs, bool mw, bool fw)
 {
        if (!searchAllowed(bv, searchstr))
                return false;
 
        PosIterator cur = PosIterator(*bv);
 
-       MatchString match(searchstr, cs, mw);
-       
-       bool found;
-
-       if (fw) {
-               PosIterator const end = bv->buffer()->pos_iterator_end();
-               found = findForward(cur, end, match);
-       } else {
-               PosIterator const beg = bv->buffer()->pos_iterator_begin();
-               found = findBackwards(cur, beg, match);
-       }
-       
+       MatchString const match(searchstr, cs, mw);
+
+       PosIterator const end = bv->buffer()->pos_iterator_end();
+       PosIterator const beg = bv->buffer()->pos_iterator_begin();
+
+       bool found = fw ? findForward(cur, end, match)
+               : findBackwards(cur, beg, match);
+
        if (found)
                put_selection_at(bv, cur, searchstr.length(), !fw);
 
@@ -166,16 +164,16 @@ int replaceAll(BufferView * bv,
        if (!searchAllowed(bv, searchstr) || buf.isReadonly())
                return 0;
        
-       recordUndo(Undo::ATOMIC, bv->text, 0,
-                  buf.paragraphs().size() - 1);
+       recordUndo(Undo::ATOMIC, bv->text(), 0, buf.paragraphs().size() - 1);
        
        PosIterator cur = buf.pos_iterator_begin();
        PosIterator const end = buf.pos_iterator_end();
-       MatchString match(searchstr, cs, mw);
+       MatchString const match(searchstr, cs, mw);
        int num = 0;
 
        int const rsize = replacestr.size();
        int const ssize = searchstr.size();
+
        while (findForward(cur, end, match)) {
                pos_type pos = cur.pos();
                LyXFont const font
@@ -185,8 +183,9 @@ int replaceAll(BufferView * bv,
                advance(cur, rsize + striked);
                ++num;
        }
+
        PosIterator beg = buf.pos_iterator_begin();
-       bv->text->init(bv);
+       bv->text()->init(bv);
        put_selection_at(bv, beg, 0, false);
        if (num)
                buf.markDirty();
@@ -194,40 +193,44 @@ int replaceAll(BufferView * bv,
 }
 
 
+namespace {
+
+bool stringSelected(BufferView * bv,
+                   string const & searchstr, 
+                   bool cs, bool mw, bool fw)
+{
+       LyXText * text = bv->getLyXText();
+       // if nothing selected or selection does not equal search
+       // string search and select next occurance and return
+       string const & str1 = searchstr;
+       string const str2 = text->selectionAsString(*bv->buffer(),
+                                                   false);
+       if ((cs && str1 != str2) || lowercase(str1) != lowercase(str2)) {
+               find(bv, searchstr, cs, mw, fw);
+               return false;
+       }
+
+       return true;
+}
+
+} //namespace anon
+
+
 int replace(BufferView * bv,
            string const & searchstr, string const & replacestr,
            bool cs, bool mw, bool fw)
 {
        if (!searchAllowed(bv, searchstr) || bv->buffer()->isReadonly())
                return 0;
-       
-       {
-               LyXText * text = bv->getLyXText();
-               // if nothing selected or selection does not equal search
-               // string search and select next occurance and return
-               string const str1 = searchstr;
-               string const str2 = text->selectionAsString(*bv->buffer(),
-                                                           false);
-               if ((cs && str1 != str2)
-                   || lowercase(str1) != lowercase(str2)) {
-                       find(bv, searchstr, cs, mw, fw);
-                       return 0;
-               }
-       }
 
-#ifdef LOCK
+       if (!stringSelected(bv, searchstr, cs, mw, fw))
+               return 0;
+
        LyXText * text = bv->getLyXText();
-       // We have to do this check only because mathed insets don't
-       // return their own LyXText but the LyXText of it's parent!
-       if (!bv->innerInset() ||
-           ((text != bv->text) &&
-            (text->inset_owner == text->inset_owner->getLockingInset()))) {
-               text->replaceSelectionWithString(replacestr);
-               text->setSelectionRange(replacestr.length());
-               text->cursor = fw ? text->selection.end
-                       : text->selection.start;
-       }
-#endif
+
+       text->replaceSelectionWithString(replacestr);
+       text->setSelectionRange(replacestr.length());
+       text->cursor = fw ? text->selEnd() : text->selStart();
 
        bv->buffer()->markDirty();
        find(bv, searchstr, cs, mw, fw);
@@ -248,7 +251,6 @@ bool findNextChange(BufferView * bv)
        if (!findChange(cur, endit))
                return false;
        
-       
        ParagraphList::iterator pit = cur.pit();
        pos_type pos = cur.pos();
        
@@ -267,7 +269,6 @@ bool findNextChange(BufferView * bv)
                }
        }
        pos_type length = end - pos;
-       bv->text->init(bv);
        put_selection_at(bv, cur, length, true);
        return true;
 }