[lmi-commits] [5827] Remove temporary testing support

[Greg Chicares]

Revision: 5827
          http://svn.sv.gnu.org/viewvc/?view=rev&root=lmi&revision=5827
Author:   chicares
Date:     2013-11-13 15:02:13 +0000 (Wed, 13 Nov 2013)
Log Message:
-----------
Remove temporary testing support

Modified Paths:
--------------
    lmi/trunk/ChangeLog
    lmi/trunk/group_values.cpp

Modified: lmi/trunk/ChangeLog
===================================================================
--- lmi/trunk/ChangeLog 2013-11-10 23:02:13 UTC (rev 5826)
+++ lmi/trunk/ChangeLog 2013-11-13 15:02:13 UTC (rev 5827)
@@ -32892,3 +32892,8 @@
   progress_meter_wx.cpp
 When Cancel pressed, eftsoons stop dawdling.
 
+20131113T1502Z <address@hidden> [542]
+
+  group_values.cpp
+Remove temporary testing support.
+

Modified: lmi/trunk/group_values.cpp
===================================================================
--- lmi/trunk/group_values.cpp  2013-11-10 23:02:13 UTC (rev 5826)
+++ lmi/trunk/group_values.cpp  2013-11-13 15:02:13 UTC (rev 5827)
@@ -34,7 +34,6 @@
 #include "contains.hpp"
 #include "emit_ledger.hpp"
 #include "fenv_guard.hpp"
-#include "global_settings.hpp"
 #include "illustrator.hpp"              // assert_consistency()
 #include "input.hpp"
 #include "ledger.hpp"
@@ -62,25 +61,17 @@
 
 /// Number of seconds to pause between printouts.
 ///
-/// Rationale: lmi sends illustrations to a printer in census order,
+/// Motivation: lmi sends illustrations to a printer in census order,
 /// but end users have complained that they are printed in a different
 /// order. Pausing briefly between printouts seems to forestall that
-/// problem.
-///
-/// This experimental implementation inserts a delay when sending PDF
-/// output not only to a printer, but also to a file, so that testing
-/// can be performed without clogging a printer. For experimentation,
-/// it has no effect without the innermost password; the header
-/// "global_settings.hpp" is included above only for this purpose,
-/// and should be removed after testing.
+/// problem. Cf.:
+///   http://www.traction-software.co.uk/batchprint/kb/KB0027.html
+/// Heuristic testing suggests that ten seconds is enough, but two
+/// seconds is not.
 
 int intermission_between_printouts(mcenum_emission emission)
 {
-    return
-          (emission & mce_emit_pdf_to_printer) ? 10
-        : ((emission & mce_emit_pdf_file) && 
global_settings::instance().ash_nazg()) ? 5
-        : 0
-        ;
+    return (emission & mce_emit_pdf_to_printer) ? 10 : 0;
 }
 
 progress_meter::enum_display_mode progress_meter_mode(mcenum_emission emission)