X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FPrinterParams.h;h=413201733818616b40aea6514544a15ca0d97d33;hb=cb52251ccff29d6cbfa13ab90a96be566e3f41ac;hp=2c19c0af7418c4305617ed0b68a5b1facfcc6743;hpb=946895278050e4a464c9591e99232a72f5f4dec9;p=lyx.git

diff --git a/src/PrinterParams.h b/src/PrinterParams.h
index 2c19c0af74..4132017338 100644
--- a/src/PrinterParams.h
+++ b/src/PrinterParams.h
@@ -5,9 +5,9 @@
  *	    LyX, The Document Processor
  * 	 
  *	    Copyright 1995 Matthias Ettrich
- *	    Copyright 1995-2000 The LyX Team
+ *	    Copyright 1995-2001 The LyX Team
  *
- *	    This file Copyright 1999-2000
+ *	    This file Copyright 1999-2001
  *	    Allan Rae
  *======================================================
  */
@@ -21,8 +21,10 @@
 
 #ifdef ENABLE_ASSERTIONS
 #include "support/LAssert.h"
+extern bool containsOnly(string const &, char const *);
 #endif
 
+
 /**
   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
@@ -33,15 +35,27 @@
 */
 struct PrinterParams {
 	///
-	enum Target{PRINTER, FILE};
+	enum Target {
+		///
+		PRINTER,
+		///
+		FILE
+	};
 	///
 	Target target;
 	///
 	string printer_name;
 	///
 	string file_name;
-	///We allow printing of even pages in a range and so on.
-	enum WhichPages{ALL, ODD, EVEN};
+	/// We allow printing of even pages in a range and so on.
+	enum WhichPages{
+		///
+		ALL,
+		///
+		ODD,
+		///
+		EVEN
+	};
 	///
 	WhichPages which_pages;
 	/** Print a page range. Both from_page and to_page used to be strings
@@ -63,15 +77,54 @@ struct PrinterParams {
 	// 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;
+        // Override the documents orientation
+	// bool orientation;
+	// Print n pages per physical sheet
+	// unsigned int nup;
+	// Override document settings for duplex.
+	// bool duplex;
+
+	/** 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
+			if (!from_page.empty()) {
+				// Assert(from_page == number or empty)
+				lyx::Assert(containsOnly(from_page,
+							 "1234567890"));
+			}
+			if (to_page) {
+				// Assert(to_page == empty
+				//        or number iff from_page set)
+				lyx::Assert(!from_page.empty());
+			}
+			switch (target) {
+			case PRINTER:
+//  				Assert(!printer_name.empty());
+				break;
+			case FILE:
+				lyx::Assert(!file_name.empty());
+				break;
+			default:
+				lyx::Assert(false);
+				break;
+			}
+			switch (which_pages) {
+			case ALL:
+			case ODD:
+			case EVEN:
+				break;
+			default:
+				lyx::Assert(false);
+				break;
+			}
+#endif
+		}
 
-	//@name Constructors and Deconstructors
-	//@{
 	///
 	PrinterParams(Target const & t = PRINTER,
 		      string const & pname = lyxrc.printer,
@@ -108,59 +161,6 @@ struct PrinterParams {
 		{
 			testInvariant();
 		}
-	//@}
-
-
-// do we need these?
-//	friend bool operator==(PrinterParams const &, PrinterParams const &);
-//	friend bool operator<(PrinterParams const &, PrinterParams const &);
-
-	//@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
-
-
-