patch: remove unused futures code

View: New views
3 Messages — Rating Filter:   Alert me  

patch: remove unused futures code

by Ken Raeburn-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


     Since support for "futures" in C has been completely disabled for  
some
     time, and should be easily implementable in Scheme with the current
     thread support, delete the C code.

     * libguile/futures.c, libguile/futures.h: Delete.
     * libguile/Makefile.am (libguile_la_SOURCES, DOT_X_FILES,
       DOT_DOC_FILES, modinclude_HEADERS): Delete references to  
futures.*
       files.

     * libguile.h: Don't include futures.h.
     * libguile/eval.c: Don't include futures.h.
       (isymnames): Delete "#@future" entry.
       (scm_m_future, s_future, scm_sym_future, unmemoize_future,
       unmemoize_builtin_macro): Delete disabled futures code.
     * libguile/eval.i.c (CEVAL): Delete disabled futures code.
     * libguile/init.c: Don't include futures.h.
       (scm_i_init_guile): Delete disabled futures initialization call.
     * libguile/tags.h (SCM_IM_FUTURE): Delete.
       (SCM_IM_CALL_WITH_VALUES, SCM_IM_ELSE, SCM_IM_ARROW,
       SCM_IM_NIL_COND, SCM_IM_BIND): Renumber.

     * doc/ref/api-scheduling.texi: Delete commented-out node on  
Futures.
     * doc/maint/guile.texi: Delete make-future and future-ref mentions.

The attached patch doesn't include the diffs for the files going away.



no-future.diff (12K) Download Attachment

Re: patch: remove unused futures code

by Neil Jerram :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Ken Raeburn <raeburn@...> writes:

>     Since support for "futures" in C has been completely disabled for
> some
>     time, and should be easily implementable in Scheme with the current
>     thread support, delete the C code.

Fine with me.

      Neil



Re: patch: remove unused futures code

by Ludovic Courtès-3 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Fine with me too.  Please apply!

Thanks,
Ludo'.