[LyX/master] Amend 48d9d01a: remove debug output

Thibaut Cuvelier dourouc05 at gmail.com
Sun Nov 20 15:36:08 UTC 2022


On Sun, 20 Nov 2022 at 16:26, Scott Kostyshak <skostysh at lyx.org> wrote:

> On Sun, Nov 20, 2022 at 03:25:09PM +0100, Thibaut Cuvelier wrote:
> > commit f3862130cf687854dbff2df348f34659ca602a02
> > Author: Thibaut Cuvelier <tcuvelier at lyx.org>
> > Date:   Sun Nov 20 16:19:17 2022 +0100
> >
> >     Amend 48d9d01a: remove debug output
> > ---
> >  src/insets/InsetIndex.cpp |    6 ------
> >  1 files changed, 0 insertions(+), 6 deletions(-)
> >
> > diff --git a/src/insets/InsetIndex.cpp b/src/insets/InsetIndex.cpp
> > index bd38cf5..f40ba03 100644
> > --- a/src/insets/InsetIndex.cpp
> > +++ b/src/insets/InsetIndex.cpp
> > @@ -1723,12 +1723,6 @@ IndexNode* buildIndexTree(vector<IndexEntry>&
> entries)
> >       // as children.
> >       auto* index_root = new IndexNode{{}, {}};
> >       for (const IndexEntry& entry : entries) {
> > -             std::cout << "Entry: " << std::endl;
> > -             std::cout << entry.terms().empty() << std::endl;
> > -             std::cout << entry.terms().size() << std::endl;
> > -             for (const docstring& d : entry.terms()) {
> > -                     std::cout << "\"" << to_utf8(d) << "\"" <<
> std::endl;
> > -             }
> >               insertIntoNode(entry, index_root);
> >       }
> >
>
> Ah thanks for catching that. I have been wanting to track down where the
> output was coming from.
>
> Just to check, you could keep the output and just guard it with
> LYX_INSET_INDEX_DEBUG if you preferred. I don't understand any of this
> stuff, so do whatever you prefer.
>

I'm more or less confident that this output will not be required in the
future for debugging the new code for indices. Anyway, it is still
available with printTree(), in a more readable presentation;
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lyx.org/pipermail/lyx-devel/attachments/20221120/c792a86c/attachment.html>


More information about the lyx-devel mailing list