Here is an update with include cleanup in headers.

Yuriy
From 6773333ae51c11b1441cc4e1cb616d5fef42c691 Mon Sep 17 00:00:00 2001
From: Yuriy Skalko <yuriy.ska...@gmail.com>
Date: Thu, 15 Oct 2020 21:09:21 +0300
Subject: [PATCH 1/3] Move Cursor.h from TocWidget.h

---
 src/frontends/qt/TocWidget.cpp |  1 +
 src/frontends/qt/TocWidget.h   | 13 ++++++++++---
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/src/frontends/qt/TocWidget.cpp b/src/frontends/qt/TocWidget.cpp
index d001b703f1..45cc6112d2 100644
--- a/src/frontends/qt/TocWidget.cpp
+++ b/src/frontends/qt/TocWidget.cpp
@@ -20,6 +20,7 @@
 
 #include "Buffer.h"
 #include "BufferView.h"
+#include "Cursor.h"
 #include "CutAndPaste.h"
 #include "FuncRequest.h"
 #include "FuncStatus.h"
diff --git a/src/frontends/qt/TocWidget.h b/src/frontends/qt/TocWidget.h
index 15006fac38..c8fa08a6f1 100644
--- a/src/frontends/qt/TocWidget.h
+++ b/src/frontends/qt/TocWidget.h
@@ -15,15 +15,22 @@
 
 #include "ui_TocUi.h"
 
-#include "Cursor.h"
-#include "FuncCode.h"
+#include "FuncRequest.h"
 
+#include <QString>
 #include <QWidget>
 
 class QModelIndex;
-class QString;
+class QPoint;
+class QTimer;
 
 namespace lyx {
+
+class Cursor;
+class DispatchResult;
+class FuncStatus;
+class Inset;
+
 namespace frontend {
 
 class GuiView;
-- 
2.28.0.windows.1

From 3ece5cad24e5fd68bf91ebb9d4f597128ce3a4f0 Mon Sep 17 00:00:00 2001
From: Yuriy Skalko <yuriy.ska...@gmail.com>
Date: Thu, 15 Oct 2020 22:15:21 +0300
Subject: [PATCH 2/3] Const refactor

---
 src/BufferParams.cpp | 4 ++--
 src/BufferParams.h   | 4 ++--
 src/TextClass.h      | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/BufferParams.cpp b/src/BufferParams.cpp
index 5f5a4cb231..c44200d4ab 100644
--- a/src/BufferParams.cpp
+++ b/src/BufferParams.cpp
@@ -567,7 +567,7 @@ AuthorList const & BufferParams::authors() const
 }
 
 
-void BufferParams::addAuthor(Author a)
+void BufferParams::addAuthor(Author const & a)
 {
        author_map_[a.bufferId()] = pimpl_->authorlist.record(a);
 }
@@ -3509,7 +3509,7 @@ string const & BufferParams::defaultBiblioStyle() const
 }
 
 
-bool const & BufferParams::fullAuthorList() const
+bool BufferParams::fullAuthorList() const
 {
        return documentClass().fullAuthorList();
 }
diff --git a/src/BufferParams.h b/src/BufferParams.h
index b42d622fb3..0a68c479a8 100644
--- a/src/BufferParams.h
+++ b/src/BufferParams.h
@@ -446,7 +446,7 @@ public:
        /// the author list for the document
        AuthorList & authors();
        AuthorList const & authors() const;
-       void addAuthor(Author a);
+       void addAuthor(Author const & a);
 
        /// map of the file's author IDs to AuthorList indexes
        typedef std::map<int, int> AuthorMap;
@@ -513,7 +513,7 @@ public:
        /// Get the default BibTeX style file from the TextClass
        std::string const & defaultBiblioStyle() const;
        /// whether the BibTeX style supports full author lists
-       bool const & fullAuthorList() const;
+       bool fullAuthorList() const;
        /// Check if a citation style is an alias to another style
        std::string getCiteAlias(std::string const & s) const;
 
diff --git a/src/TextClass.h b/src/TextClass.h
index fa9ac20e1d..6ba185aa9c 100644
--- a/src/TextClass.h
+++ b/src/TextClass.h
@@ -553,9 +553,9 @@ public:
        /// The maximum number of citations before "et al."
        size_t max_citenames() const { return maxcitenames_; }
        ///
-       bool const & fullAuthorList() const { return cite_full_author_list_; }
+       bool fullAuthorList() const { return cite_full_author_list_; }
        ///
-       bool const & bibInToc() const { return bibintoc_; }
+       bool bibInToc() const { return bibintoc_; }
 protected:
        /// Constructs a DocumentClass based upon a LayoutFile.
        DocumentClass(LayoutFile const & tc);
-- 
2.28.0.windows.1

From 07a93e32e1cc866d1ea0638ed75d214f1cbbdbbc Mon Sep 17 00:00:00 2001
From: Yuriy Skalko <yuriy.ska...@gmail.com>
Date: Thu, 15 Oct 2020 22:27:52 +0300
Subject: [PATCH 3/3] Move Author.h, Format.h from BufferParams.h

---
 src/BufferParams.cpp             | 1 +
 src/BufferParams.h               | 5 +++--
 src/CutAndPaste.cpp              | 1 +
 src/frontends/qt/GuiDocument.cpp | 1 +
 src/frontends/qt/GuiView.cpp     | 1 +
 src/insets/InsetTabular.cpp      | 1 +
 6 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/BufferParams.cpp b/src/BufferParams.cpp
index c44200d4ab..8d0d7d13d4 100644
--- a/src/BufferParams.cpp
+++ b/src/BufferParams.cpp
@@ -28,6 +28,7 @@
 #include "ColorSet.h"
 #include "Converter.h"
 #include "Encoding.h"
+#include "Format.h"
 #include "IndicesList.h"
 #include "Language.h"
 #include "LaTeXFeatures.h"
diff --git a/src/BufferParams.h b/src/BufferParams.h
index 0a68c479a8..6a5ded8260 100644
--- a/src/BufferParams.h
+++ b/src/BufferParams.h
@@ -15,10 +15,8 @@
 #ifndef BUFFERPARAMS_H
 #define BUFFERPARAMS_H
 
-#include "Author.h"
 #include "Citation.h"
 #include "DocumentClassPtr.h"
-#include "Format.h"
 #include "LayoutModuleList.h"
 #include "OutputParams.h"
 #include "paper.h"
@@ -34,11 +32,14 @@ namespace lyx {
 
 namespace support { class FileName; }
 
+class Author;
+class AuthorList;
 class BranchList;
 class Bullet;
 class DocumentClass;
 class Encoding;
 class Font;
+class Format;
 class IndicesList;
 class Language;
 class LayoutFile;
diff --git a/src/CutAndPaste.cpp b/src/CutAndPaste.cpp
index fd0b73ce6f..772fa49b5e 100644
--- a/src/CutAndPaste.cpp
+++ b/src/CutAndPaste.cpp
@@ -15,6 +15,7 @@
 
 #include "CutAndPaste.h"
 
+#include "Author.h"
 #include "BranchList.h"
 #include "Buffer.h"
 #include "buffer_funcs.h"
diff --git a/src/frontends/qt/GuiDocument.cpp b/src/frontends/qt/GuiDocument.cpp
index d4f388db08..883b283629 100644
--- a/src/frontends/qt/GuiDocument.cpp
+++ b/src/frontends/qt/GuiDocument.cpp
@@ -33,6 +33,7 @@
 #include "Cursor.h"
 #include "Encoding.h"
 #include "FloatPlacement.h"
+#include "Format.h"
 #include "FuncRequest.h"
 #include "IndicesList.h"
 #include "Language.h"
diff --git a/src/frontends/qt/GuiView.cpp b/src/frontends/qt/GuiView.cpp
index c33a06f104..d417ffecd9 100644
--- a/src/frontends/qt/GuiView.cpp
+++ b/src/frontends/qt/GuiView.cpp
@@ -47,6 +47,7 @@
 #include "CutAndPaste.h"
 #include "Encoding.h"
 #include "ErrorList.h"
+#include "Format.h"
 #include "FuncStatus.h"
 #include "FuncRequest.h"
 #include "Language.h"
diff --git a/src/insets/InsetTabular.cpp b/src/insets/InsetTabular.cpp
index afd29761e9..00d9bc1fbb 100644
--- a/src/insets/InsetTabular.cpp
+++ b/src/insets/InsetTabular.cpp
@@ -22,6 +22,7 @@
 
 #include "InsetTabular.h"
 
+#include "Author.h"
 #include "buffer_funcs.h"
 #include "Buffer.h"
 #include "BufferParams.h"
-- 
2.28.0.windows.1

-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel

Reply via email to