Re: MODULE_GC_KLUDGE conditional compilation blocks

2010-05-16 Thread Ian Hulin
On 15/05/10 22:42, Patrick McCarty wrote: On Sat, May 15, 2010 at 2:22 PM, Graham Percival wrote: On Sat, May 15, 2010 at 10:02 PM, Ian Hulin wrote: As I understand it, these are to get round problems with Guile pre V1.8,7, which we no longer support. Is it now safe to delete this

Re: MODULE_GC_KLUDGE conditional compilation blocks

2010-05-15 Thread James Lowe
Hello, On 15 May 2010, at 22:22, Graham Percival wrote: On Sat, May 15, 2010 at 10:02 PM, Ian Hulin wrote: As I understand it, these are to get round problems with Guile pre V1.8,7, which we no longer support. Is it now safe to delete this code? According to configure.in, we only require

Re: MODULE_GC_KLUDGE conditional compilation blocks

2010-05-15 Thread Patrick McCarty
On Sat, May 15, 2010 at 2:22 PM, Graham Percival wrote: > On Sat, May 15, 2010 at 10:02 PM, Ian Hulin wrote: >> As I understand it, these are to get round problems with Guile pre V1.8,7, >> which we no longer support.  Is it now safe to delete this code? > > According to configure.in, we only req

Re: MODULE_GC_KLUDGE conditional compilation blocks

2010-05-15 Thread Graham Percival
On Sat, May 15, 2010 at 10:02 PM, Ian Hulin wrote: > As I understand it, these are to get round problems with Guile pre V1.8,7, > which we no longer support.  Is it now safe to delete this code? According to configure.in, we only require guile 1.8.2 or higher. Should that be bumped? My main syst

MODULE_GC_KLUDGE conditional compilation blocks

2010-05-15 Thread Ian Hulin
Hi all, 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-modu