X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FPrinterParams.h;h=73b10c368ccb50ff178a7120f8148f3b83af7592;hb=68ee68103a77300c3aa01db9e63bd01b8243a0eb;hp=6f738a09143e5bfffbdb3b8a8bf01f9752c8ff5d;hpb=6bba977f42b0cde753ac2ffd26f3f9c6b32ba0b6;p=lyx.git

diff --git a/src/PrinterParams.h b/src/PrinterParams.h
index 6f738a0914..73b10c368c 100644
--- a/src/PrinterParams.h
+++ b/src/PrinterParams.h
@@ -1,49 +1,59 @@
 // -*- C++ -*-
-/* This file is part of
- * ======================================================
- * 
- *	    LyX, The Document Processor
- * 	 
- *	    Copyright 1995 Matthias Ettrich
- *	    Copyright 1995-2000 The LyX Team
+/**
+ * \file PrinterParams.h
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
+ *
+ * \author Allan Rae
  *
- *	    This file Copyright 1999-2000
- *	    Allan Rae
- *======================================================
+ * Full author contact details are available in file CREDITS.
  */
 
 #ifndef PRINTERPARAMS_H
 #define PRINTERPARAMS_H
 
-#ifndef LYXRC_H
-#error You must include lyxrc.h before PrinterParams.h
-#endif
+#include <string>
 
-#ifdef ENABLE_ASSERTIONS
-#include "support/LAssert.h"
-#endif
+
+namespace lyx {
 
 /**
-  This struct contains (or should contain) all the parameters required for
-  printing a buffer.  Some work still needs to be done on this struct and
+  This class contains (or should contain) all the parameters required for
+  printing a buffer.  Some work still needs to be done on this class and
   printing handling in general to make it nice and full-featured.
   The main things I'd like to add now is the ability to print a read-only
   document with different orientation, papersize or single/duplex state
   than the document's settings. ARRae 20000423
 */
-struct PrinterParams {
+class PrinterParams
+{
+public:
+	///
+	PrinterParams();
+
 	///
-	enum Target{PRINTER, FILE};
+	enum Target {
+		///
+		PRINTER,
+		///
+		FILE
+	};
+
+	/** Test that all the fields contain valid entries.  It's unlikely
+	    that the internal code will get this wrong however new ports
+	    and external scripts might drive the wrong values in.
+	 */
+	void testInvariant() const;
+
+public:
 	///
 	Target target;
 	///
-	string printer_name;
+	std::string printer_name;
 	///
-	string file_name;
-	///We allow printing of even pages in a range and so on.
-	enum WhichPages{ALL, ODD, EVEN};
+	std::string file_name;
 	///
-	WhichPages which_pages;
+	bool all_pages;
 	/** Print a page range. Both from_page and to_page used to be strings
 	    because they're actually easier to work with that way.  I've
 	    switched to_page to be an int.  However, from_page will remain a
@@ -51,118 +61,32 @@ struct PrinterParams {
 	    a page range "1,3-5" and so on.
 	    I've modified the invariant test to match. ARRae 20000518
 	 */
-	string from_page;
+	unsigned int from_page;
 	///
-	int to_page;
+	unsigned int to_page;
 	///
-	bool reverse_order;
+	bool odd_pages;
+	///
+	bool even_pages;
+	///
+	unsigned int count_copies;
 	///
-	bool unsorted_copies;
+	bool sorted_copies;
 	///
-  	int count_copies;
+	bool reverse_order;
 	// The settings below should allow us to print any read-only doc in
 	// whatever size/orientation we want it -- overriding the documents
 	// settings.
-        /// Override the documents orientation
-	//bool orientation;
-	/// Print n pages per physical sheet
-	//unsigned int nup;
-	/// Override document settings for duplex.
-	//bool duplex;
-
-	//@name Constructors and Deconstructors
-	//@{
-	///
-	PrinterParams(Target const & t = PRINTER,
-		      string const & pname = lyxrc.printer,
-		      string const & fname = string(),
-		      WhichPages const wp = ALL,
-		      string const & from = string(),
-		      int const & to = 0,
-		      bool const reversed = false,
-		      bool const unsorted = false,
-		      int const & num_copies = 1)
-		: target(t),
-		  printer_name(pname),
-		  file_name(fname),
-		  which_pages(wp),
-		  from_page(from),
-		  to_page(to),
-		  reverse_order(reversed),
-		  unsorted_copies(unsorted),
-		  count_copies(num_copies)
-		{
-			testInvariant();
-		}
-	///
-	PrinterParams(PrinterParams const & pp)
-		: target(pp.target),
-		  printer_name(pp.printer_name),
-		  file_name(pp.file_name),
-		  which_pages(pp.which_pages),
-		  from_page(pp.from_page),
-		  to_page(pp.to_page),
-		  reverse_order(pp.reverse_order),
-		  unsorted_copies(pp.unsorted_copies),
-		  count_copies(pp.count_copies)
-		{
-			testInvariant();
-		}
-	///
-	~PrinterParams() {}
-	//@}
-
-
-// do we need these?
-//	friend bool operator==(PrinterParams const &, PrinterParams const &);
-//	friend bool operator<(PrinterParams const &, PrinterParams const &);
+	// Override the documents orientation
+	// bool orientation;
+	// Print n pages per physical sheet
+	// unsigned int nup;
+	// Override document settings for duplex.
+	// bool duplex;
 
-	//@name Invariant Test Method
-	//@{
-	/** Test that all the fields contain valid entries.  It's unlikely
-	    that the internal code will get this wrong (at least for the
-	    xforms code anyway) however new ports and external scripts
-	    might drive the wrong values in.
-	 */
-	void testInvariant() const
-		{
-#ifdef ENABLE_ASSERTIONS
-			extern bool containsOnly(string const &, char const *);
-			if (!from_page.empty()) {
-				// Assert(from_page == number or empty)
-				Assert(containsOnly(from_page, "1234567890"));
-			}
-			if (to_page) {
-				// Assert(to_page == empty
-				//        or number iff from_page set)
-				Assert(!from_page.empty());
-			}
-			switch (target) {
-			case PRINTER:
-//  				Assert(!printer_name.empty());
-				break;
-			case FILE:
-				Assert(!file_name.empty());
-				break;
-			default:
-				Assert(false);
-				break;
-			}
-			switch (which_pages) {
-			case ALL:
-			case ODD:
-			case EVEN:
-				break;
-			default:
-				Assert(false);
-				break;
-			}
-#endif
-		}
-	//@}
 };
 
-#endif
-
 
+} // namespace lyx
 
+#endif