[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Fwd: Remove kludge in Module Code to support Guile V1.6 (MODULE_GC_KLUDG
From: |
Ian Hulin |
Subject: |
Fwd: Remove kludge in Module Code to support Guile V1.6 (MODULE_GC_KLUDGE conditional compilation) |
Date: |
Mon, 17 May 2010 18:43:52 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.9) Gecko/20100317 Lightning/1.0b1 Thunderbird/3.0.4 |
Sorry all,
Sent the original to the development list by mistake.
Cheers,
Ian Hulin
-------- Original Message --------
Subject: Remove kludge in Module Code to support Guile V1.6
(MODULE_GC_KLUDGE conditional compilation)
Date: Mon, 17 May 2010 18:32:32 +0100
From: Ian Hulin <address@hidden>
To: address@hidden
Newsgroups: gmane.comp.gnu.lilypond.devel
We have these blocks in our C++ code base:
$ git grep -A 5 MODULE_GC_KLUDGE
lily/include/ly-module.hh:#define MODULE_GC_KLUDGE
lily/include/ly-module.hh-
lily/include/ly-module.hh-#endif /* LY_MODULE_HH */
lily/include/ly-module.hh-
--
lily/ly-module.cc:#ifdef MODULE_GC_KLUDGE
lily/ly-module.cc-Protected_scm anonymous_modules = SCM_EOL;
lily/ly-module.cc-bool perform_gc_kludge;
lily/ly-module.cc-#endif
lily/ly-module.cc-
lily/ly-module.cc-void
--
lily/ly-module.cc:#ifdef MODULE_GC_KLUDGE
lily/ly-module.cc- for (SCM s = anonymous_modules;
lily/ly-module.cc- scm_is_pair (s);
lily/ly-module.cc- s = scm_cdr (s))
lily/ly-module.cc- {
lily/ly-module.cc- SCM module = scm_car (s);
--
lily/module-scheme.cc:#ifdef MODULE_GC_KLUDGE
lily/module-scheme.cc- clear_anonymous_modules ();
lily/module-scheme.cc-#endif
lily/module-scheme.cc-
lily/module-scheme.cc- return SCM_UNSPECIFIED;
lily/module-scheme.cc-}
$
As I understand it, these are to get round problems with Guile pre V1.8,
which we no longer support.
Please open a tracker so we can delete this code and I will begin work
on a patch.
Cheers,
Ian Hulin
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Fwd: Remove kludge in Module Code to support Guile V1.6 (MODULE_GC_KLUDGE conditional compilation),
Ian Hulin <=