unofficial mirror of guile-devel@gnu.org 
 help / color / mirror / Atom feed
* `scm_list ()'
@ 2006-02-02  9:38 Ludovic Courtès
  2006-02-02 18:58 ` Neil Jerram
  2006-02-02 21:25 ` Kevin Ryde
  0 siblings, 2 replies; 5+ messages in thread
From: Ludovic Courtès @ 2006-02-02  9:38 UTC (permalink / raw)


Hi,

`scm_list ()' is no longer available in 1.7, not even as a deprecated
function.  In 1.6, it is documented as follows:

  -- Scheme Procedure: list . objs
  -- C Function: scm_list (objs)
      Return a list containing OBJS, the arguments to `list'.

The funny thing is that to get a list, you had to have a list already
(OBJS is a list here).  I don't really get the point of this function in
fact.  ;-)

Should the following be enough to ensure compatibility?

  SCM
  scm_list (SCM obj)
  {
    scm_c_issue_deprecation_warning ("`scm_list' is deprecated.  Use  "
                                     "`scm_list_n' instead.");

    SCM_VALIDATE_LIST (1, obj);

    return obj;
  }

Thanks,
Ludovic.


_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/guile-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: `scm_list ()'
  2006-02-02  9:38 `scm_list ()' Ludovic Courtès
@ 2006-02-02 18:58 ` Neil Jerram
  2006-02-02 21:25 ` Kevin Ryde
  1 sibling, 0 replies; 5+ messages in thread
From: Neil Jerram @ 2006-02-02 18:58 UTC (permalink / raw)


ludovic.courtes@laas.fr (Ludovic Courtès) writes:

> The funny thing is that to get a list, you had to have a list already
> (OBJS is a list here).  I don't really get the point of this function in
> fact.  ;-)

Presumably that's why it has been removed!

> Should the following be enough to ensure compatibility?
>
>   SCM
>   scm_list (SCM obj)
>   {
>     scm_c_issue_deprecation_warning ("`scm_list' is deprecated.  Use  "
>                                      "`scm_list_n' instead.");

Shouldn't that be something like "`scm_list' is deprecated.  Wherever
you have `scm_list (X)', you can replace it by just `X'."

    Neil



_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/guile-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: `scm_list ()'
  2006-02-02  9:38 `scm_list ()' Ludovic Courtès
  2006-02-02 18:58 ` Neil Jerram
@ 2006-02-02 21:25 ` Kevin Ryde
  2006-02-06  9:15   ` Ludovic Courtès
  1 sibling, 1 reply; 5+ messages in thread
From: Kevin Ryde @ 2006-02-02 21:25 UTC (permalink / raw)


ludovic.courtes@laas.fr (Ludovic Courtès) writes:
>
> The funny thing is that to get a list, you had to have a list already
> (OBJS is a list here).  I don't really get the point of this function in
> fact.  ;-)

That's why I dropped it :-).  But yes, I'll put it back just in case.

>     scm_c_issue_deprecation_warning ("`scm_list' is deprecated.  Use  "
>                                      "`scm_list_n' instead.");
>     SCM_VALIDATE_LIST (1, obj);

No thanks, I'll put the original no-op.


_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/guile-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: `scm_list ()'
  2006-02-02 21:25 ` Kevin Ryde
@ 2006-02-06  9:15   ` Ludovic Courtès
  2006-02-06 20:26     ` Kevin Ryde
  0 siblings, 1 reply; 5+ messages in thread
From: Ludovic Courtès @ 2006-02-06  9:15 UTC (permalink / raw)


Hi,

Kevin Ryde <user42@zip.com.au> writes:

> No thanks, I'll put the original no-op.

Why not marking it as deprecated as well?

Thanks,
Ludovic.


_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/guile-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: `scm_list ()'
  2006-02-06  9:15   ` Ludovic Courtès
@ 2006-02-06 20:26     ` Kevin Ryde
  0 siblings, 0 replies; 5+ messages in thread
From: Kevin Ryde @ 2006-02-06 20:26 UTC (permalink / raw)


ludovic.courtes@laas.fr (Ludovic Courtès) writes:
>
> Why not marking it as deprecated as well?

Because there's nothing wrong with it! :)  Deprecated should mean
something that has a fatal flaw, something so bad that applications
must be forced away from it.


_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/guile-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2006-02-06 20:26 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-02-02  9:38 `scm_list ()' Ludovic Courtès
2006-02-02 18:58 ` Neil Jerram
2006-02-02 21:25 ` Kevin Ryde
2006-02-06  9:15   ` Ludovic Courtès
2006-02-06 20:26     ` Kevin Ryde

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).