Turn music functions and their signatures into C++ smobs

[lilypond.git] / lily / music-function.cc

diff --git a/lily/music-function.cc b/lily/music-function.cc
index 7d6675b3b759fd756db7723c83a66e99bfe0930b..85b21575761961e209a76811fae4506297940463 100644 (file)
--- a/lily/music-function.cc
+++ b/lily/music-function.cc
@@ -20,17 +20,28 @@
 #include "music-function.hh"
 
 #include "music.hh"
-
-static scm_t_bits music_function_tag;
+#include "ly-smobs.icc"
+
+class Musicfunction {
+  DECLARE_SIMPLE_SMOBS (Musicfunction);
+  SCM signature_;
+  SCM function_;
+public:
+  Musicfunction (SCM signature, SCM function):
+    signature_(signature), function_(function) { }
+  SCM get_function () { return function_; }
+  SCM get_signature () { return signature_; }
+};
+
+IMPLEMENT_SIMPLE_SMOBS (Musicfunction);
+IMPLEMENT_DEFAULT_EQUAL_P (Musicfunction);
 
 /* Print a textual represenation of the smob to a given port.  */
-static int
-print_music_function (SCM b, SCM port, scm_print_state *)
+int
+Musicfunction::print_smob (SCM b, SCM port, scm_print_state *)
 {
-  SCM value = SCM_CELL_OBJECT_1 (b);
-
   scm_puts ("#<Music function ", port);
-  scm_write (value, port);
+  scm_write (Musicfunction::unsmob (b)->get_function (), port);
   scm_puts (">", port);
 
   /* Non-zero means success.  */
@@ -40,7 +51,7 @@ print_music_function (SCM b, SCM port, scm_print_state *)
 bool
 is_music_function (SCM music_function)
 {
-  return (SCM_NIMP (music_function) && SCM_CELL_TYPE (music_function) == music_function_tag);
+  return Musicfunction::unsmob (music_function);
 }
 
 SCM
@@ -49,25 +60,29 @@ get_music_function_transform (SCM music_function)
   if (!is_music_function (music_function))
     return SCM_UNDEFINED;
 
-  return SCM_CELL_OBJECT_1 (music_function);
+  return Musicfunction::unsmob (music_function)->get_function ();
 }
 
-static void
-init_music_function (void)
+SCM
+make_music_function (SCM signature, SCM func)
 {
-  music_function_tag = scm_make_smob_type ("music-function", 0);
-  scm_set_smob_mark (music_function_tag, scm_markcdr);
-  scm_set_smob_print (music_function_tag, print_music_function);
+  return Musicfunction (signature, func).smobbed_copy ();
 }
 
 SCM
-make_music_function (SCM signature, SCM func)
+get_music_function_signature (SCM music_function)
 {
-  scm_set_object_property_x (func, ly_symbol2scm ("music-function-signature"),
-                             signature);
+  if (!is_music_function (music_function))
+    return SCM_UNDEFINED;
 
-  SCM_RETURN_NEWSMOB (music_function_tag, func);
+  return Musicfunction::unsmob (music_function)->get_signature ();
 }
 
-ADD_SCM_INIT_FUNC (music_function_tag, init_music_function);
-
+SCM
+Musicfunction::mark_smob (SCM s)
+{
+  Musicfunction *p = Musicfunction::unsmob (s);
+  scm_gc_mark (p->signature_);
+  ASSERT_LIVE_IS_ALLOWED ();
+  return p->function_;
+}