From 8074b696bcb8a3d8fe2f86bcef4233ba01763a41 Mon Sep 17 00:00:00 2001
From: Thibaut Cuvelier <tcuvelier@lyx.org>
Date: Fri, 2 Sep 2022 01:29:19 +0200
Subject: [PATCH] InsetTabular: move more helper methods to private

---
 src/insets/InsetTabular.h | 65 +++++++++++++++++++++------------------
 1 file changed, 35 insertions(+), 30 deletions(-)

diff --git a/src/insets/InsetTabular.h b/src/insets/InsetTabular.h
index 6896323b4a..1c07ba0f05 100644
--- a/src/insets/InsetTabular.h
+++ b/src/insets/InsetTabular.h
@@ -919,40 +919,10 @@ public:
 	///
 	BoxType useBox(idx_type cell) const;
 	///
-	// helper function for Latex
-	///
-	void TeXTopHLine(otexstream &, row_type row, std::list<col_type>,
-			 std::list<col_type>) const;
-	///
-	void TeXBottomHLine(otexstream &, row_type row, std::list<col_type>,
-			    std::list<col_type>) const;
-	///
-	void TeXCellPreamble(otexstream &, idx_type cell, bool & ismulticol, bool & ismultirow,
-			     bool const bidi) const;
-	///
-	void TeXCellPostamble(otexstream &, idx_type cell, bool ismulticol, bool ismultirow) const;
-	///
-	void TeXLongtableHeaderFooter(otexstream &, OutputParams const &, std::list<col_type>,
-				      std::list<col_type>) const;
-	///
 	bool isValidRow(row_type const row) const;
 	///
 	void TeXRow(otexstream &, row_type const row,
 		    OutputParams const &, std::list<col_type>, std::list<col_type>) const;
-	///
-	// helper functions for plain text
-	///
-	bool plaintextTopHLine(odocstringstream &, row_type row,
-			       std::vector<unsigned int> const &) const;
-	///
-	bool plaintextBottomHLine(odocstringstream &, row_type row,
-				  std::vector<unsigned int> const &) const;
-	///
-	void plaintextPrintCell(odocstringstream &,
-				OutputParams const &,
-				idx_type cell, row_type row, col_type column,
-				std::vector<unsigned int> const &,
-				bool onlydata, size_t max_length) const;
 
 	/// change associated Buffer
 	void setBuffer(Buffer & buffer);
@@ -964,6 +934,9 @@ public:
 private:
 	Buffer * buffer_;
 
+	///
+	// helper function for DocBook
+	///
 	/// Determines whether the tabular item should be generated as DocBook or XHTML.
 	enum class XmlOutputFormat : bool {
 		XHTML = true,
@@ -985,6 +958,38 @@ private:
 	XmlRowWiseBorders computeXmlBorders(row_type row) const;
 	std::vector<std::string> computeCssStylePerCell(row_type row, col_type col, idx_type cell) const;
 
+	///
+	// helper functions for plain text
+	///
+	bool plaintextTopHLine(odocstringstream &, row_type row,
+	                       std::vector<unsigned int> const &) const;
+	///
+	bool plaintextBottomHLine(odocstringstream &, row_type row,
+	                          std::vector<unsigned int> const &) const;
+	///
+	void plaintextPrintCell(odocstringstream &,
+	                        OutputParams const &,
+	                        idx_type cell, row_type row, col_type column,
+	                        std::vector<unsigned int> const &,
+	                        bool onlydata, size_t max_length) const;
+
+	///
+	// helper function for LaTeX
+	///
+	void TeXTopHLine(otexstream &, row_type row, std::list<col_type>,
+	                 std::list<col_type>) const;
+	///
+	void TeXBottomHLine(otexstream &, row_type row, std::list<col_type>,
+	                    std::list<col_type>) const;
+	///
+	void TeXCellPreamble(otexstream &, idx_type cell, bool & ismulticol, bool & ismultirow,
+	                     bool const bidi) const;
+	///
+	void TeXCellPostamble(otexstream &, idx_type cell, bool ismulticol, bool ismultirow) const;
+	///
+	void TeXLongtableHeaderFooter(otexstream &, OutputParams const &, std::list<col_type>,
+	                              std::list<col_type>) const;
+
 }; // Tabular