]> git.lyx.org Git - features.git/commitdiff
FindAdv, Debug: Added new debug level :findverbose
authorKornel Benko <kornel@lyx.org>
Tue, 19 Apr 2022 11:41:38 +0000 (13:41 +0200)
committerKornel Benko <kornel@lyx.org>
Tue, 19 Apr 2022 11:41:38 +0000 (13:41 +0200)
With the debug-level 'find' there are now less output.
The original output can be received now with debig-level 'findverbose'

Also constructions like
 LYXERR(Debug::FIND|Debug::FINDVERBOSE, "...");
are now possible.

src/frontends/qt/FindAndReplace.cpp
src/lyxfind.cpp
src/support/debug.cpp
src/support/debug.h

index 300c291da37c6fdcbab66565c79ad814d0965828..ab61e1ab7347c325b1f0cc2e2cd853b1d2c80698 100644 (file)
@@ -115,9 +115,9 @@ bool FindAndReplaceWidget::eventFilter(QObject * obj, QEvent * event)
        case Qt::Key_Tab:
                if (e->modifiers() == Qt::NoModifier) {
                        if (obj == find_work_area_){
-                               LYXERR(Debug::FIND, "Focusing replace WA");
+                               LYXERR(Debug::FINDVERBOSE, "Focusing replace WA");
                                replace_work_area_->setFocus();
-                               LYXERR(Debug::FIND, "Selecting entire replace buffer");
+                               LYXERR(Debug::FINDVERBOSE, "Selecting entire replace buffer");
                                dispatch(FuncRequest(LFUN_BUFFER_BEGIN));
                                dispatch(FuncRequest(LFUN_BUFFER_END_SELECT));
                                return true;
@@ -127,9 +127,9 @@ bool FindAndReplaceWidget::eventFilter(QObject * obj, QEvent * event)
 
        case Qt::Key_Backtab:
                if (obj == replace_work_area_) {
-                       LYXERR(Debug::FIND, "Focusing find WA");
+                       LYXERR(Debug::FINDVERBOSE, "Focusing find WA");
                        find_work_area_->setFocus();
-                       LYXERR(Debug::FIND, "Selecting entire find buffer");
+                       LYXERR(Debug::FINDVERBOSE, "Selecting entire find buffer");
                        dispatch(FuncRequest(LFUN_BUFFER_BEGIN));
                        dispatch(FuncRequest(LFUN_BUFFER_END_SELECT));
                        return true;
@@ -172,7 +172,7 @@ static vector<string> const & allManualsFiles()
 static bool nextDocumentBuffer(Buffer * & buf)
 {
        ListOfBuffers const children = buf->allRelatives();
-       LYXERR(Debug::FIND, "children.size()=" << children.size());
+       LYXERR(Debug::FINDVERBOSE, "children.size()=" << children.size());
        ListOfBuffers::const_iterator it =
                find(children.begin(), children.end(), buf);
        LASSERT(it != children.end(), return false);
@@ -193,7 +193,7 @@ static bool nextDocumentBuffer(Buffer * & buf)
 static bool prevDocumentBuffer(Buffer * & buf)
 {
        ListOfBuffers const children = buf->allRelatives();
-       LYXERR(Debug::FIND, "children.size()=" << children.size());
+       LYXERR(Debug::FINDVERBOSE, "children.size()=" << children.size());
        ListOfBuffers::const_iterator it =
                find(children.begin(), children.end(), buf);
        LASSERT(it != children.end(), return false)
@@ -344,9 +344,9 @@ bool FindAndReplaceWidget::findAndReplaceScope(FindAndReplaceOptions & opt, bool
        UndoGroupHelper helper(buf);
 
        do {
-               LYXERR(Debug::FIND, "Dispatching LFUN_WORD_FINDADV");
+               LYXERR(Debug::FINDVERBOSE, "Dispatching LFUN_WORD_FINDADV");
                dispatch(cmd);
-               LYXERR(Debug::FIND, "dispatched");
+               LYXERR(Debug::FINDVERBOSE, "dispatched");
                if (bv->cursor().result().dispatched()) {
                        // New match found and selected (old selection replaced if needed)
                        if (replace_all)
@@ -398,7 +398,7 @@ bool FindAndReplaceWidget::findAndReplaceScope(FindAndReplaceOptions & opt, bool
                        //lyx::dispatch(FuncRequest(LFUN_BUFFER_END));
                        bv->cursor().setCursor(doc_iterator_end(buf));
                        bv->cursor().backwardPos();
-                       LYXERR(Debug::FIND, "findBackAdv5: cur: "
+                       LYXERR(Debug::FINDVERBOSE, "findBackAdv5: cur: "
                                << bv->cursor());
                }
                bv->clearSelection();
@@ -454,7 +454,7 @@ bool FindAndReplaceWidget::findAndReplace(
                        ? FindAndReplaceOptions::R_ONLY_MATHS
                        : FindAndReplaceOptions::R_EVERYTHING;
 
-       LYXERR(Debug::FIND, "FindAndReplaceOptions: "
+       LYXERR(Debug::FINDVERBOSE, "FindAndReplaceOptions: "
               << "find_buf_name=" << find_buf_name
               << ", casesensitiv=" << casesensitive
               << ", matchword=" << matchword
@@ -556,7 +556,7 @@ void FindAndReplaceWidget::showEvent(QShowEvent * /* ev */)
        replace_work_area_->installEventFilter(this);
 
        view_.setCurrentWorkArea(find_work_area_);
-       LYXERR(Debug::FIND, "Selecting entire find buffer");
+       LYXERR(Debug::FINDVERBOSE, "Selecting entire find buffer");
        dispatch(FuncRequest(LFUN_BUFFER_BEGIN));
        dispatch(FuncRequest(LFUN_BUFFER_END_SELECT));
 }
index 2ea1e8811bf2115e669ff69e491e24e846a27771..69e740d470e51730008b9fe88a8f89f7582ec868 100644 (file)
@@ -875,7 +875,7 @@ string correctRegex(string t, bool withformat)
                                                replace = accents["braceright"];
                                        else {
                                                // else part should not exist
-                                               LASSERT(1, /**/);
+                                               LASSERT(0, /**/);
                                        }
                                }
                                else {
@@ -885,7 +885,7 @@ string correctRegex(string t, bool withformat)
                                                replace = "\\}";
                                        else {
                                                // else part should not exist
-                                               LASSERT(1, /**/);
+                                               LASSERT(0, /**/);
                                        }
                                }
                        }
@@ -1109,7 +1109,7 @@ static docstring buffer_to_latex(Buffer & buffer)
        pit_type const endpit = buffer.paragraphs().size();
        for (pit_type pit = 0; pit != endpit; ++pit) {
                TeXOnePar(buffer, buffer.text(), pit, os, runparams);
-               LYXERR(Debug::FIND, "searchString up to here: " << ods.str());
+               LYXERR(Debug::FINDVERBOSE, "searchString up to here: " << ods.str());
        }
        return ods.str();
 }
@@ -1156,7 +1156,7 @@ static string latexNamesToUtf8(docstring strIn)
                add = addtmp;
        else if (addtmp.length() > lastpos)
                add += addtmp.substr(lastpos, addtmp.length() - lastpos);
-       LYXERR(Debug::FIND, "Adding to search string: '"
+       LYXERR(Debug::FINDVERBOSE, "Adding to search string: '"
                        << add << "'");
        return add;
 }
@@ -1190,7 +1190,7 @@ static docstring stringifySearchBuffer(Buffer & buffer, FindAndReplaceOptions co
                        string add = latexNamesToUtf8(par.asString(pos_type(0), par.size(),
                                                                option,
                                                                &runparams));
-                       LYXERR(Debug::FIND, "Adding to search string: '"
+                       LYXERR(Debug::FINDVERBOSE, "Adding to search string: '"
                                << add << "'");
                        t += add;
                }
@@ -1221,7 +1221,7 @@ static size_t identifyLeading(string const & s)
               || regex_replace(t, t, "^ ?\\\\item\\{[a-z]+\\}", "")
               || regex_replace(t, t, "^\\\\begin\\{[a-zA-Z_]*\\*?\\}", ""))
               ;
-       LYXERR(Debug::FIND, "  after removing leading $, \\[ , \\emph{, \\textbf{, etc.: '" << t << "'");
+       LYXERR(Debug::FINDVERBOSE, "  after removing leading $, \\[ , \\emph{, \\textbf{, etc.: '" << t << "'");
        return s.find(t);
 }
 
@@ -2025,7 +2025,7 @@ void Intervall::handleCloseP(int i, bool closingAllowed)
 {
        if (actualdeptindex <= 0) {
                if (! closingAllowed)
-                       LYXERR(Debug::FIND, "Bad closing parenthesis in latex");  /* should not happen, but the latex input may be wrong */
+                       LYXERR(Debug::FINDVERBOSE, "Bad closing parenthesis in latex");  /* should not happen, but the latex input may be wrong */
                // if we are at the very end
                addIntervall(i, i+1);
        }
@@ -3363,7 +3363,7 @@ static string correctlanguagesetting(string par, bool isPatternString, bool with
        if (withformat) {
                // Split the latex input into pieces which
                // can be digested by our search engine
-               LYXERR(Debug::FIND, "input: \"" << par << "\"");
+               LYXERR(Debug::FINDVERBOSE, "input: \"" << par << "\"");
                if (isPatternString && (pbuf != nullptr)) { // Check if we should disable/enable test for language
                        // We check for polyglossia, because in runparams.flavor we use Flavor::XeTeX
                        string doclang = pbuf->params().language->polyglossia();
@@ -3381,7 +3381,7 @@ static string correctlanguagesetting(string par, bool isPatternString, bool with
 
                }
                result = splitOnKnownMacros(par.substr(0,parlen), isPatternString);
-               LYXERR(Debug::FIND, "After splitOnKnownMacros:\n\"" << result << "\"");
+               LYXERR(Debug::FINDVERBOSE, "After splitOnKnownMacros:\n\"" << result << "\"");
        }
        else
                result = par.substr(0, parlen);
@@ -3396,7 +3396,7 @@ static string correctlanguagesetting(string par, bool isPatternString, bool with
                                features += " " + a;
                                // LYXERR(Debug::INFO, "Identified regex format:" << a);
                        }
-                       LYXERR(Debug::FIND, "Identified Features" << features);
+                       LYXERR(Debug::FINDVERBOSE, "Identified Features" << features);
 
                }
        } else if (regex_with_format) {
@@ -3406,7 +3406,7 @@ static string correctlanguagesetting(string par, bool isPatternString, bool with
                        bool b = it->second;
                        if (b && ! info[a]) {
                                missed++;
-                               LYXERR(Debug::FIND, "Missed(" << missed << " " << a <<", srclen = " << parlen );
+                               LYXERR(Debug::FINDVERBOSE, "Missed(" << missed << " " << a <<", srclen = " << parlen );
                                return "";
                        }
                }
@@ -3424,7 +3424,7 @@ static int identifyClosing(string & t)
 {
        int open_braces = 0;
        do {
-               LYXERR(Debug::FIND, "identifyClosing(): t now is '" << t << "'");
+               LYXERR(Debug::FINDVERBOSE, "identifyClosing(): t now is '" << t << "'");
                if (regex_replace(t, t, "(.*[^\\\\])\\$$", "$1"))
                        continue;
                if (regex_replace(t, t, "(.*[^\\\\])\\\\\\]$", "$1"))
@@ -3585,14 +3585,14 @@ MatchStringAdv::MatchStringAdv(lyx::Buffer & buf, FindAndReplaceOptions & opt)
                string regexp2_str = "(" + lead_as_regex_string + ")(.*?)" + par_as_regex_string_nolead;
                CreateRegexp(opt, regexp_str, regexp2_str);
                use_regexp = true;
-               LYXERR(Debug::FIND, "Setting regexp to : '" << regexp_str << "'");
-               LYXERR(Debug::FIND, "Setting regexp2 to: '" << regexp2_str << "'");
+               LYXERR(Debug::FINDVERBOSE|Debug::FIND, "Setting regexp to : '" << regexp_str << "'");
+               LYXERR(Debug::FINDVERBOSE|Debug::FIND, "Setting regexp2 to: '" << regexp2_str << "'");
                return;
        }
 
        if (!opt.ignoreformat) {
                lead_size = identifyLeading(par_as_string);
-               LYXERR(Debug::FIND, "Lead_size: " << lead_size);
+               LYXERR(Debug::FINDVERBOSE, "Lead_size: " << lead_size);
                lead_as_string = par_as_string.substr(0, lead_size);
                par_as_string_nolead = par_as_string.substr(lead_size, par_as_string.size() - lead_size);
        }
@@ -3606,13 +3606,13 @@ MatchStringAdv::MatchStringAdv(lyx::Buffer & buf, FindAndReplaceOptions & opt)
                        (void)regex_replace(par_as_string_nolead, par_as_string_nolead, "\\$$", "");
                        (void)regex_replace(par_as_string_nolead, par_as_string_nolead, "}$", "");
                        par_as_string = par_as_string_nolead;
-                       LYXERR(Debug::FIND, "lead_as_regexp is '" << lead_as_regexp << "'");
-                       LYXERR(Debug::FIND, "par_as_string now is '" << par_as_string << "'");
+                       LYXERR(Debug::FINDVERBOSE, "lead_as_regexp is '" << lead_as_regexp << "'");
+                       LYXERR(Debug::FINDVERBOSE, "par_as_string now is '" << par_as_string << "'");
                }
-               // LYXERR(Debug::FIND, "par_as_string before escape_for_regex() is '" << par_as_string << "'");
+               // LYXERR(Debug::FINDVERBOSE, "par_as_string before escape_for_regex() is '" << par_as_string << "'");
                par_as_string = escape_for_regex(par_as_string, !opt.ignoreformat);
                // Insert (.*?) before trailing closure of math, macros and environments, so to catch parts of them.
-               // LYXERR(Debug::FIND, "par_as_string now is '" << par_as_string << "'");
+               // LYXERR(Debug::FINDVERBOSE, "par_as_string now is '" << par_as_string << "'");
                ++close_wildcards;
                size_t lng = par_as_string.size();
                if (!opt.ignoreformat) {
@@ -3632,15 +3632,15 @@ MatchStringAdv::MatchStringAdv(lyx::Buffer & buf, FindAndReplaceOptions & opt)
                        if (lng < par_as_string.size())
                                par_as_string = par_as_string.substr(0,lng);
                }
-               LYXERR(Debug::FIND, "par_as_string after correctRegex is '" << par_as_string << "'");
+               LYXERR(Debug::FINDVERBOSE, "par_as_string after correctRegex is '" << par_as_string << "'");
                if ((lng > 0) && (par_as_string[0] == '^')) {
                        par_as_string = par_as_string.substr(1);
                        --lng;
                        opt.matchAtStart = true;
                }
-               // LYXERR(Debug::FIND, "par_as_string now is '" << par_as_string << "'");
-               // LYXERR(Debug::FIND, "Open braces: " << open_braces);
-               // LYXERR(Debug::FIND, "Replaced text (to be used as regex): " << par_as_string);
+               // LYXERR(Debug::FINDVERBOSE, "par_as_string now is '" << par_as_string << "'");
+               // LYXERR(Debug::FINDVERBOSE, "Open braces: " << open_braces);
+               // LYXERR(Debug::FINDVERBOSE, "Replaced text (to be used as regex): " << par_as_string);
 
                // If entered regexp must match at begin of searched string buffer
                // Kornel: Added parentheses to use $1 for size of the leading string
@@ -3662,8 +3662,8 @@ MatchStringAdv::MatchStringAdv(lyx::Buffer & buf, FindAndReplaceOptions & opt)
                        regexp_str = "(" + lead_as_regexp + ")()" + par_as_string;
                        regexp2_str = "(" + lead_as_regexp + ")(.*?)" + par_as_string;
                }
-               LYXERR(Debug::FIND, "Setting regexp to : '" << regexp_str << "'");
-               LYXERR(Debug::FIND, "Setting regexp2 to: '" << regexp2_str << "'");
+               LYXERR(Debug::FINDVERBOSE|Debug::FIND, "Setting regexp to : '" << regexp_str << "'");
+               LYXERR(Debug::FINDVERBOSE|Debug::FIND, "Setting regexp2 to: '" << regexp2_str << "'");
                CreateRegexp(opt, regexp_str, regexp2_str, par_as_string);
        }
 }
@@ -3694,12 +3694,12 @@ MatchResult MatchStringAdv::findAux(DocIterator const & cur, int len, bool at_be
                mres.match_len = -1;
                return mres;
        }
-       LYXERR(Debug::FIND, "After normalization: Matching against:\n'" << str << "'");
+       LYXERR(Debug::FINDVERBOSE, "After normalization: Matching against:\n'" << str << "'");
 
        LASSERT(use_regexp, /**/);
        {
                // use_regexp always true
-               LYXERR(Debug::FIND, "Searching in regexp mode: at_begin=" << at_begin);
+               LYXERR(Debug::FINDVERBOSE, "Searching in regexp mode: at_begin=" << at_begin);
 #if QTSEARCH
                QString qstr = QString::fromStdString(str);
                QRegularExpression const *p_regexp;
@@ -3833,7 +3833,7 @@ MatchResult MatchStringAdv::operator()(DocIterator const & cur, int len, bool at
 {
        MatchResult mres = findAux(cur, len, at_begin);
        int res = mres.match_len;
-       LYXERR(Debug::FIND,
+       LYXERR(Debug::FINDVERBOSE,
               "res=" << res << ", at_begin=" << at_begin
               << ", matchAtStart=" << opt.matchAtStart
               << ", inTexted=" << cur.inTexted());
@@ -3919,11 +3919,11 @@ string MatchStringAdv::normalize(docstring const & s, bool ignore_format) const
        // Remove stale empty \emph{}, \textbf{} and similar blocks from latexify
        // Kornel: Added textsl, textsf, textit, texttt and noun
        // + allow to seach for colored text too
-       LYXERR(Debug::FIND, "Removing stale empty macros from: " << t);
+       LYXERR(Debug::FINDVERBOSE, "Removing stale empty macros from: " << t);
        while (regex_replace(t, t, "\\\\(emph|noun|text(bf|sl|sf|it|tt)|(u|uu)line|(s|x)out|uwave)(\\{(\\{\\})?\\})+", ""))
-               LYXERR(Debug::FIND, "  further removing stale empty \\emph{}, \\textbf{} macros from: " << t);
+               LYXERR(Debug::FINDVERBOSE, "  further removing stale empty \\emph{}, \\textbf{} macros from: " << t);
        while (regex_replace(t, t, "\\\\((sub)?(((sub)?section)|paragraph)|part)\\*?(\\{(\\{\\})?\\})+", ""))
-               LYXERR(Debug::FIND, "  further removing stale empty \\emph{}, \\textbf{} macros from: " << t);
+               LYXERR(Debug::FINDVERBOSE, "  further removing stale empty \\emph{}, \\textbf{} macros from: " << t);
        while (regex_replace(t, t, "\\\\(foreignlanguage|textcolor|item)\\{[a-z]+\\}(\\{(\\{\\})?\\})+", ""));
 
        return t;
@@ -3932,7 +3932,7 @@ string MatchStringAdv::normalize(docstring const & s, bool ignore_format) const
 
 docstring stringifyFromCursor(DocIterator const & cur, int len)
 {
-       LYXERR(Debug::FIND, "Stringifying with len=" << len << " from cursor at pos: " << cur);
+       LYXERR(Debug::FINDVERBOSE, "Stringifying with len=" << len << " from cursor at pos: " << cur);
        if (cur.inTexted()) {
                Paragraph const & par = cur.paragraph();
                // TODO what about searching beyond/across paragraph breaks ?
@@ -3957,11 +3957,13 @@ docstring stringifyFromCursor(DocIterator const & cur, int len)
                if (ignoreFormats.getNonContent()) {
                        runparams.for_search |= OutputParams::SearchNonOutput;
                }
-               LYXERR(Debug::FIND, "Stringifying with cur: "
+               LYXERR(Debug::FINDVERBOSE, "Stringifying with cur: "
                       << cur << ", from pos: " << cur.pos() << ", end: " << end);
-               return from_utf8(latexNamesToUtf8(par.asString(cur.pos(), end,
-                                                              option,
-                                                              &runparams)));
+               docstring res = from_utf8(latexNamesToUtf8(par.asString(cur.pos(), end,
+                                                                       option,
+                                                                       &runparams)));
+               LYXERR(Debug::FIND, "Stringified text from pos(" << cur.pos() << ") len(" << len << "): " << res);
+               return res;
        } else if (cur.inMathed()) {
                CursorSlice cs = cur.top();
                MathData md = cs.cell();
@@ -3972,11 +3974,11 @@ docstring stringifyFromCursor(DocIterator const & cur, int len)
                MathData md2;
                for (MathData::const_iterator it = md.begin() + cs.pos(); it != it_end; ++it)
                        md2.push_back(*it);
-               docstring s = from_utf8(latexNamesToUtf8(asString(md2)));
-               LYXERR(Debug::FIND, "Stringified math: '" << s << "'");
-               return s;
+               docstring res = from_utf8(latexNamesToUtf8(asString(md2)));
+               LYXERR(Debug::FINDVERBOSE|Debug::FIND, "Stringified math from pos(" << cur.pos() << ") len(" << len << "): " << res);
+               return res;
        }
-       LYXERR(Debug::FIND, "Don't know how to stringify from here: " << cur);
+       LYXERR(Debug::FINDVERBOSE|Debug::FIND, "Don't know how to stringify from here: " << cur);
        return docstring();
 }
 
@@ -3987,8 +3989,8 @@ docstring stringifyFromCursor(DocIterator const & cur, int len)
 docstring latexifyFromCursor(DocIterator const & cur, int len)
 {
        /*
-       LYXERR(Debug::FIND, "Latexifying with len=" << len << " from cursor at pos: " << cur);
-       LYXERR(Debug::FIND, "  with cur.lastpost=" << cur.lastpos() << ", cur.lastrow="
+       LYXERR(Debug::FINDVERBOSE, "Latexifying with len=" << len << " from cursor at pos: " << cur);
+       LYXERR(Debug::FINDVERBOSE, "  with cur.lastpost=" << cur.lastpos() << ", cur.lastrow="
               << cur.lastrow() << ", cur.lastcol=" << cur.lastcol());
        */
        Buffer const & buf = *cur.buffer();
@@ -4019,9 +4021,8 @@ docstring latexifyFromCursor(DocIterator const & cur, int len)
                        endpos = cur.pos() + len;
                TeXOnePar(buf, *cur.innerText(), cur.pit(), os, runparams,
                          string(), cur.pos(), endpos);
-               string s = lyx::to_utf8(ods.str());
-               LYXERR(Debug::FIND, "Latexified +modified text: '" << s << "'");
-               return(lyx::from_utf8(s));
+               LYXERR(Debug::FIND, "Latexified text from pos(" << cur.pos() << ") len(" << len << "): " << ods.str());
+               return(ods.str());
        } else if (cur.inMathed()) {
                // Retrieve the math environment type, and add '$' or '$[' or others (\begin{equation}) accordingly
                for (int s = cur.depth() - 1; s >= 0; --s) {
@@ -4056,9 +4057,9 @@ docstring latexifyFromCursor(DocIterator const & cur, int len)
                                break;
                        }
                }
-               LYXERR(Debug::FIND, "Latexified math: '" << lyx::to_utf8(ods.str()) << "'");
+               LYXERR(Debug::FINDVERBOSE|Debug::FIND, "Latexified math from pos(" << cur.pos() << ") len(" << len << "): " << ods.str());
        } else {
-               LYXERR(Debug::FIND, "Don't know how to stringify from here: " << cur);
+               LYXERR(Debug::FINDVERBOSE|Debug::FIND, "Don't know how to stringify from here: " << cur);
        }
        return ods.str();
 }
@@ -4158,7 +4159,7 @@ MatchResult findAdvFinalize(DocIterator & cur, MatchStringAdv const & match, Mat
                max_match = mres;
        }
        if (max_match.match_len <= 0) return fail;
-       LYXERR(Debug::FIND, "Ok");
+       LYXERR(Debug::FINDVERBOSE, "Ok");
 
        // Compute the match length
        int len = 1;
@@ -4254,7 +4255,7 @@ int findForwardAdv(DocIterator & cur, MatchStringAdv & match)
        MatchResult orig_mres;
        while (!theApp()->longOperationCancelled() && cur) {
                //(void) findAdvForwardInnermost(cur);
-               LYXERR(Debug::FIND, "findForwardAdv() cur: " << cur);
+               LYXERR(Debug::FINDVERBOSE, "findForwardAdv() cur: " << cur);
                MatchResult mres = match(cur, -1, false);
                string msg = "Starting";
                if (repeat)
@@ -4268,7 +4269,7 @@ int findForwardAdv(DocIterator & cur, MatchStringAdv & match)
                if (match_len <= 0) {
                        // This should exit nested insets, if any, or otherwise undefine the currsor.
                        cur.pos() = cur.lastpos();
-                       LYXERR(Debug::FIND, "Advancing pos: cur=" << cur);
+                       LYXERR(Debug::FINDVERBOSE, "Advancing pos: cur=" << cur);
                        cur.forwardPos();
                }
                else {  // match_len > 0
@@ -4285,7 +4286,7 @@ int findForwardAdv(DocIterator & cur, MatchStringAdv & match)
                                if (increment < 1)
                                        increment = 1;
                        }
-                       LYXERR(Debug::FIND, "Set increment to " << increment);
+                       LYXERR(Debug::FINDVERBOSE, "Set increment to " << increment);
                        while (increment > 0) {
                                DocIterator old_cur = cur;
                                if (cur.pos() + increment >= cur.lastpos()) {
@@ -4339,10 +4340,9 @@ int findForwardAdv(DocIterator & cur, MatchStringAdv & match)
                                mres = orig_mres;
                        }
                        // LYXERR0("Leaving first loop");
-                       LYXERR(Debug::FIND, "Finalizing 1");
+                       LYXERR(Debug::FINDVERBOSE, "Finalizing 1");
                        MatchResult found_match = findAdvFinalize(cur, match, mres);
                        if (found_match.match_len > 0) {
-                               LASSERT(found_match.pos_len > 0, /**/);
                                match.FillResults(found_match);
                                return found_match.pos_len;
                        }
@@ -4367,7 +4367,7 @@ MatchResult findMostBackwards(DocIterator & cur, MatchStringAdv const & match, M
        MatchResult mr = findAdvFinalize(tmp_cur, match, expected);
        Inset & inset = cur.inset();
        for (; cur != cur_begin; cur.backwardPos()) {
-               LYXERR(Debug::FIND, "findMostBackwards(): cur=" << cur);
+               LYXERR(Debug::FINDVERBOSE, "findMostBackwards(): cur=" << cur);
                DocIterator new_cur = cur;
                new_cur.backwardPos();
                if (new_cur == cur || &new_cur.inset() != &inset || !match(new_cur).match_len)
@@ -4377,7 +4377,7 @@ MatchResult findMostBackwards(DocIterator & cur, MatchStringAdv const & match, M
                        break;
                mr = new_mr;
        }
-       LYXERR(Debug::FIND, "findMostBackwards(): exiting with cur=" << cur);
+       LYXERR(Debug::FINDVERBOSE, "findMostBackwards(): exiting with cur=" << cur);
        return mr;
 }
 
@@ -4403,11 +4403,11 @@ int findBackwardsAdv(DocIterator & cur, MatchStringAdv & match)
                                cur.pos() = cur.lastpos();
                        else
                                cur.pos() = cur_orig.pos();
-                       LYXERR(Debug::FIND, "findBackAdv2: cur: " << cur);
+                       LYXERR(Debug::FINDVERBOSE, "findBackAdv2: cur: " << cur);
                        DocIterator cur_prev_iter;
                        do {
                                found_match = match(cur);
-                               LYXERR(Debug::FIND, "findBackAdv3: found_match="
+                               LYXERR(Debug::FINDVERBOSE, "findBackAdv3: found_match="
                                       << (found_match.match_len > 0) << ", cur: " << cur);
                                if (found_match.match_len > 0) {
                                        MatchResult found_mr = findMostBackwards(cur, match, found_match);
@@ -4472,7 +4472,7 @@ static bool allNonLowercase(Cursor const & cur, int len)
        pos_type beg_pos = cur.selectionBegin().pos();
        pos_type end_pos = cur.selectionBegin().pos() + len;
        if (len > cur.lastpos() + 1 - beg_pos) {
-               LYXERR(Debug::FIND, "This should not happen, more debug needed");
+               LYXERR(Debug::FINDVERBOSE, "This should not happen, more debug needed");
                len = cur.lastpos() + 1 - beg_pos;
                end_pos = beg_pos + len;
        }
@@ -4489,13 +4489,13 @@ static bool firstUppercase(Cursor const & cur)
        char_type ch1, ch2;
        pos_type pos = cur.selectionBegin().pos();
        if (pos >= cur.lastpos() - 1) {
-               LYXERR(Debug::FIND, "No upper-case at cur: " << cur);
+               LYXERR(Debug::FINDVERBOSE, "No upper-case at cur: " << cur);
                return false;
        }
        ch1 = cur.paragraph().getChar(pos);
        ch2 = cur.paragraph().getChar(pos + 1);
        bool result = isUpperCase(ch1) && isLowerCase(ch2);
-       LYXERR(Debug::FIND, "firstUppercase(): "
+       LYXERR(Debug::FINDVERBOSE, "firstUppercase(): "
               << "ch1=" << ch1 << "(" << char(ch1) << "), ch2="
               << ch2 << "(" << char(ch2) << ")"
               << ", result=" << result << ", cur=" << cur);
@@ -4562,7 +4562,7 @@ static int findAdvReplace(BufferView * bv, FindAndReplaceOptions const & opt, Ma
                        || sel_beg.idx() != sel_end.idx())
                return 0;
        int sel_len = sel_end.pos() - sel_beg.pos();
-       LYXERR(Debug::FIND, "sel_beg: " << sel_beg << ", sel_end: " << sel_end
+       LYXERR(Debug::FINDVERBOSE, "sel_beg: " << sel_beg << ", sel_end: " << sel_end
               << ", sel_len: " << sel_len << endl);
        if (sel_len == 0)
                return 0;
@@ -4583,7 +4583,7 @@ static int findAdvReplace(BufferView * bv, FindAndReplaceOptions const & opt, Ma
        repl_buffer.setUnnamed(true);
        LASSERT(repl_buffer.readString(lyx), return 0);
        if (opt.keep_case && sel_len >= 2) {
-               LYXERR(Debug::FIND, "keep_case true: cur.pos()=" << cur.pos() << ", sel_len=" << sel_len);
+               LYXERR(Debug::FINDVERBOSE, "keep_case true: cur.pos()=" << cur.pos() << ", sel_len=" << sel_len);
                if (cur.inTexted()) {
                        if (firstUppercase(cur))
                                changeFirstCase(repl_buffer, text_uppercase, text_lowercase);
@@ -4596,13 +4596,13 @@ static int findAdvReplace(BufferView * bv, FindAndReplaceOptions const & opt, Ma
                repl_buffer.changeLanguage(
                                        repl_buffer.language(),
                                        cur.getFont().language());
-               LYXERR(Debug::FIND, "Replacing by pasteParagraphList()ing repl_buffer");
-               LYXERR(Debug::FIND, "Before pasteParagraphList() cur=" << cur << endl);
+               LYXERR(Debug::FINDVERBOSE, "Replacing by pasteParagraphList()ing repl_buffer");
+               LYXERR(Debug::FINDVERBOSE, "Before pasteParagraphList() cur=" << cur << endl);
                cap::pasteParagraphList(cur, repl_buffer.paragraphs(),
                                        repl_buffer.params().documentClassPtr(),
                                        repl_buffer.params().authors(),
                                        bv->buffer().errorList("Paste"));
-               LYXERR(Debug::FIND, "After pasteParagraphList() cur=" << cur << endl);
+               LYXERR(Debug::FINDVERBOSE, "After pasteParagraphList() cur=" << cur << endl);
                sel_len = repl_buffer.paragraphs().begin()->size();
        } else if (cur.inMathed()) {
                odocstringstream ods;
@@ -4616,23 +4616,23 @@ static int findAdvReplace(BufferView * bv, FindAndReplaceOptions const & opt, Ma
                TeXOnePar(repl_buffer, repl_buffer.text(), 0, os, runparams);
                //repl_buffer.getSourceCode(ods, 0, repl_buffer.paragraphs().size(), false);
                docstring repl_latex = ods.str();
-               LYXERR(Debug::FIND, "Latexified replace_buffer: '" << repl_latex << "'");
+               LYXERR(Debug::FINDVERBOSE, "Latexified replace_buffer: '" << repl_latex << "'");
                string s;
                (void)regex_replace(to_utf8(repl_latex), s, "\\$(.*)\\$", "$1");
                (void)regex_replace(s, s, "\\\\\\[(.*)\\\\\\]", "$1");
                repl_latex = from_utf8(s);
-               LYXERR(Debug::FIND, "Replacing by insert()ing latex: '" << repl_latex << "' cur=" << cur << " with depth=" << cur.depth());
+               LYXERR(Debug::FINDVERBOSE, "Replacing by insert()ing latex: '" << repl_latex << "' cur=" << cur << " with depth=" << cur.depth());
                MathData ar(cur.buffer());
                asArray(repl_latex, ar, Parse::NORMAL);
                cur.insert(ar);
                sel_len = ar.size();
-               LYXERR(Debug::FIND, "After insert() cur=" << cur << " with depth: " << cur.depth() << " and len: " << sel_len);
+               LYXERR(Debug::FINDVERBOSE, "After insert() cur=" << cur << " with depth: " << cur.depth() << " and len: " << sel_len);
        }
        if (cur.pos() >= sel_len)
                cur.pos() -= sel_len;
        else
                cur.pos() = 0;
-       LYXERR(Debug::FIND, "After pos adj cur=" << cur << " with depth: " << cur.depth() << " and len: " << sel_len);
+       LYXERR(Debug::FINDVERBOSE, "After pos adj cur=" << cur << " with depth: " << cur.depth() << " and len: " << sel_len);
        bv->putSelectionAt(DocIterator(cur), sel_len, !opt.forward);
        bv->processUpdateFlags(Update::Force);
        return 1;
@@ -4704,7 +4704,7 @@ bool findAdv(BufferView * bv, FindAndReplaceOptions & opt)
                // Should never happen, maybe LASSERT() here?
                pos_len = cur.lastpos() - cur.pos();
        }
-       LYXERR(Debug::FIND, "Putting selection at cur=" << cur << " with len: " << pos_len);
+       LYXERR(Debug::FINDVERBOSE|Debug::FIND, "Putting selection at cur=" << cur << " with len: " << pos_len);
        bv->putSelectionAt(cur, pos_len, !opt.forward);
 
        return true;
@@ -4725,7 +4725,7 @@ ostringstream & operator<<(ostringstream & os, FindAndReplaceOptions const & opt
           << int(opt.scope) << ' '
           << int(opt.restr);
 
-       LYXERR(Debug::FIND, "built: " << os.str());
+       LYXERR(Debug::FINDVERBOSE, "built: " << os.str());
 
        return os;
 }
@@ -4733,7 +4733,7 @@ ostringstream & operator<<(ostringstream & os, FindAndReplaceOptions const & opt
 
 istringstream & operator>>(istringstream & is, FindAndReplaceOptions & opt)
 {
-       // LYXERR(Debug::FIND, "parsing");
+       // LYXERR(Debug::FINDVERBOSE, "parsing");
        string s;
        string line;
        getline(is, line);
@@ -4745,7 +4745,7 @@ istringstream & operator>>(istringstream & is, FindAndReplaceOptions & opt)
                        break;
                getline(is, line);
        }
-       // LYXERR(Debug::FIND, "file_buf_name: '" << s << "'");
+       // LYXERR(Debug::FINDVERBOSE, "file_buf_name: '" << s << "'");
        opt.find_buf_name = from_utf8(s);
        is >> opt.casesensitive >> opt.matchword >> opt.forward >> opt.expandmacros >> opt.ignoreformat >> opt.replace_all;
        is.get();       // Waste space before replace string
@@ -4759,7 +4759,7 @@ istringstream & operator>>(istringstream & is, FindAndReplaceOptions & opt)
                        break;
                getline(is, line);
        }
-       // LYXERR(Debug::FIND, "repl_buf_name: '" << s << "'");
+       // LYXERR(Debug::FINDVERBOSE, "repl_buf_name: '" << s << "'");
        opt.repl_buf_name = from_utf8(s);
        is >> opt.keep_case;
        int i;
@@ -4769,7 +4769,7 @@ istringstream & operator>>(istringstream & is, FindAndReplaceOptions & opt)
        opt.restr = FindAndReplaceOptions::SearchRestriction(i);
 
        /*
-       LYXERR(Debug::FIND, "parsed: " << opt.casesensitive << ' ' << opt.matchword << ' ' << opt.forward << ' '
+       LYXERR(Debug::FINDVERBOSE, "parsed: " << opt.casesensitive << ' ' << opt.matchword << ' ' << opt.forward << ' '
               << opt.expandmacros << ' ' << opt.ignoreformat << ' ' << opt.keep_case << ' '
               << opt.scope << ' ' << opt.restr);
        */
index b89074120e5a6fe68692fd551d85f4adaa5f6aed..63e9abd8c8d277bc8bd70710917fe00ddd866290 100644 (file)
@@ -71,7 +71,8 @@ DebugErrorItem errorTags[] = {
        { Debug::RTL,       "rtl",       N_("RTL/Bidi")},
        { Debug::LOCALE,    "locale",    N_("Locale/Internationalisation")},
        { Debug::SELECTION, "selection", N_("Selection copy/paste mechanism")},
-       { Debug::FIND,      "find",      N_("Find and replace mechanism")},
+       { Debug::FIND,      "find",      N_("Find and replace mechanism, short version")},
+       { Debug::FINDVERBOSE,"findverbose", N_("Find and replace mechanism, verbose version")},
        { Debug::DEBUG,     "debug",     N_("Developers' general debug messages")},
        { Debug::ANY,       "any",       N_("All debugging messages")},
        { Debug::ANY,       "all",       N_("All debugging messages")}
@@ -193,7 +194,7 @@ void Debug::showLevel(ostream & os, Debug::Type level)
 void Debug::showTags(ostream & os)
 {
        for (DebugErrorItem const & item : errorTags)
-               os << setw(10) << static_cast<unsigned int>(item.level)
+               os << setw(12) << static_cast<Debug::base_type>(item.level)
                   << setw(13) << item.name
                   << "  " << to_utf8(_(item.desc)) << '\n';
        os.flush();
@@ -212,7 +213,7 @@ void LyXErr::enable()
 }
 
 
-bool LyXErr::debugging(Debug::Type t) const
+bool LyXErr::debugging(Debug::base_type t) const
 {
        return (dt_ & t);
 }
index 18dd5fa8ce18fd44c1bb46922c9986e0e5e7aa4a..681ef3fbafa598e05e7b841f090c2b7c8d2d84c2 100644 (file)
@@ -37,7 +37,8 @@ namespace lyx {
 ///  This is all the different debug levels that we have.
 namespace Debug {
        ///
-       enum Type {
+       typedef uint64_t base_type;
+       enum Type : base_type {
                ///
                NONE = 0,
                ///
@@ -103,9 +104,11 @@ namespace Debug {
                /// Find and Replace
                FIND       = (1u << 30),
                ///
-               DEBUG      = (1u << 31),
+               FINDVERBOSE= (1u << 31),
                ///
-               ANY = 0xffffffff
+               DEBUG      = (1L << 32),
+               ///
+               ANY = 0x1ffffffff
        };
 
        /// Return number of levels
@@ -179,7 +182,7 @@ public:
        /// Returns the current debug level
        Debug::Type level() const { return dt_; }
        /// Returns true if t is part of the current debug level
-       bool debugging(Debug::Type t = Debug::ANY) const;
+       bool debugging(Debug::base_type t = Debug::ANY) const;
 
        ///
        static char const * stripName(char const *);