(Score): oops. Copy error_found_ too.

[lilypond.git] / lily / book.cc

diff --git a/lily/book.cc b/lily/book.cc
index 50fee8f48e038ac054105ec147c97bd036940314..6a873251f2ba482abf914c870a150dcaa268fa5d 100644 (file)
--- a/lily/book.cc
+++ b/lily/book.cc
@@ -8,23 +8,21 @@
 
 #include <stdio.h>
 
-
-#include "ly-smobs.icc"
-#include "stencil.hh"
 #include "book.hh"
 #include "global-context.hh"
 #include "ly-module.hh"
 #include "main.hh"
 #include "music-iterator.hh"
-#include "output-def.hh"
 #include "music-output.hh"
 #include "music.hh"
-#include "page.hh"
-#include "paper-book.hh"
 #include "output-def.hh"
+#include "paper-book.hh"
 #include "score.hh"
+#include "stencil.hh"
 #include "warn.hh"
 
+#include "ly-smobs.icc"
+
 Book::Book ()
   : Input ()
 {
@@ -61,78 +59,73 @@ Book::print_smob (SCM, SCM p, scm_print_state*)
   return 1;
 }
 
-void
-Book::process (String outname, Output_def *default_def, SCM header)
+/* This function does not dump the output; outname is required eg. for
+   dumping header fields.  */
+Paper_book *
+Book::process (String outname, Output_def *default_def)
 {
-  Paper_book *paper_book = new Paper_book ();
+  bool error = false;
+  for (int i = 0; i < scores_.size(); i++)
+    error = error || scores_[i]->error_found_;
 
-        
-  Real scale = ly_scm2double (bookpaper_->c_variable ("outputscale"));
+  if (error)
+    return 0;
+    
+  Paper_book *paper_book = new Paper_book ();
+  Real scale = scm_to_double (bookpaper_->c_variable ("outputscale"));
   
   Output_def * scaled_bookdef = scale_output_def (bookpaper_, scale);
 
   paper_book->bookpaper_ = scaled_bookdef;
+  scm_gc_unprotect_object (scaled_bookdef->self_scm());
+  
+  paper_book->header_ = header_;
+  
   int score_count = scores_.size ();
   for (int i = 0; i < score_count; i++)
     {
       SCM systems = scores_[i]->book_rendering (outname,
                                                paper_book->bookpaper_,
                                                default_def);
-      if (systems != SCM_UNDEFINED)
+      
+      /* If the score is empty, generate no output.  Should we
+        do titling?  */
+      if (SCM_NFALSEP(scm_vector_p (systems)))
        {
-         Score_lines sc;
-         sc.lines_ = systems;
-         sc.header_ = header;
-
-         paper_book->score_lines_.push (sc);
+         Score_systems sc;
+         sc.systems_ = systems;
+         sc.header_ = scores_[i]->header_;
+         paper_book->score_systems_.push (sc);
        }
     }
 
-  paper_book->output (outname);
-  
-  scm_gc_unprotect_object (paper_book->bookpaper_->self_scm ());
-  scm_gc_unprotect_object (paper_book->self_scm ());
+  return paper_book;
 }
 
-/* FIXME: WIP, this is a hack.  Return first page as stencil.  */
-SCM
-Book::to_stencil (Output_def *default_def, SCM header)
+LY_DEFINE(ly_make_book, "ly:make-book",
+         2, 0, 1, (SCM bookpaper, SCM header, SCM scores),
+         "Make a \\book of @var{bookpaper} and @var{header} (which may be #f as well)  "
+         "containing @code{\\scores}.")
 {
-  Paper_book *paper_book = new Paper_book ();
-  Real scale = ly_scm2double (bookpaper_->c_variable ("outputscale"));
-  
-  Output_def * scaled_bookdef = scale_output_def (bookpaper_, scale);
+  Output_def * odef = unsmob_output_def (bookpaper);
+  SCM_ASSERT_TYPE (odef, bookpaper,
+                  SCM_ARG1, __FUNCTION__, "Output_def");
 
-  paper_book->bookpaper_ = scaled_bookdef;
+  Book *book = new Book;
+  book->bookpaper_ = odef;
 
-  int score_count = scores_.size ();
-  for (int i = 0; i < score_count; i++)
-    {
-      SCM systems = scores_[i]->book_rendering ("<markup>",
-                                               bookpaper_,
-                                               default_def);
-      
-      if (systems != SCM_UNDEFINED)
-       {
-         Score_lines sc;
-         sc.lines_ = systems;
-         sc.header_ =header;
-
-         paper_book->score_lines_.push (sc);
-
-         // wtf: code dup.
-       }
-    }
-
-  SCM pages = paper_book->pages ();
-  paper_book = 0;
-  if (pages != SCM_EOL)
+  if (ly_c_module_p (header))
+    book->header_ = header;
+  
+  for (SCM s = scores; scm_is_pair (s); s = ly_cdr (s))
     {
-      progress_indication (_f ("paper output to `%s'...", "<markup>"));
-      return (unsmob_page (ly_car (pages)))->to_stencil ().smobbed_copy ();
+      Score *score = unsmob_score (ly_car (s));
+      if (score)
+       book->scores_.push (score);
     }
-
-  scm_gc_unprotect_object (paper_book->bookpaper_->self_scm ());
   
-  return SCM_EOL;
+  SCM x = book->self_scm ();
+  scm_gc_unprotect_object (x);
+  return x;
 }
+