Re: [dev-serveez] Patch to make Serveez compile with Guile 1.7 (CVS HEAD)

[stefan]

On 5 Feb 2003, Andreas Rottmann wrote:

> > > > 1. Is it a good idea to SCM_SETVELTS(x,v) whereas 'v' is a local (stack)
> > > >    variable?  I think the previous version was correct.  Does it compile
> > > >    with Guile 1.7.0 without that part?
> > > >
> > > Hmm, after looking at the SETVELTS macro this doesn't seem a good idea
> > > indeed :-) In Guile 1.7, SCM_VELTS returns a const SCM *, so it would
> > > be probably best to do something like this:
> > >
> > > #ifndef SCM_GC_WRITABLE_VELTS
> > > #define SCM_GC_WRITABLE_VELTS SCM_VELTS
> > > #endif
> >
> > Now the question is: What's the difference in SCM_GC_WRITABLE_VELTS and
> > SCM_WRITABLE_VELTS?  The macros are identical and used in different
> > places.  I would prefer SCM_WRITABLE_VELTS.
> >
> Have overlooked SCM_WRITABLE_VELTS. Sounds better to me, too - is the
> far more common used one, it seems.

Hello!

I just applied all pending changes.  Can you please verify whether it
works now for Guile 1.7.x for you without anymore warnings (gcc 3.2)?

Thanks in advance,
        address@hidden