public inbox for gsl-discuss@sourceware.org
 help / color / mirror / Atom feed
From: Gerard Jungman <jungman@lanl.gov>
To: gsl-discuss mailing list <gsl-discuss@sourceware.org>
Subject: Re: Feedback from GSL folks on libflame 4.0
Date: Fri, 19 Feb 2010 00:20:00 -0000	[thread overview]
Message-ID: <1266538880.27033.100.camel@manticore.lanl.gov> (raw)
In-Reply-To: <4a00655d1002181150j3975f69bk29084d5b50eeb974@mail.gmail.com>

On Thu, 2010-02-18 at 13:50 -0600, Rhys Ulerich wrote:

> I'm willing to take a stab at providing the necessary error handling
> underneath flame to make it play nice with GSL's error handling model.
> The obvious choice is to adopt GSL's routines.  However, flame is
> LGPL and so adopting GSL's GPLed error handling directly is not an
> option.
> 
> Do you know of an LGPL project that you believe does a good job in this regard?

I don't know of another project, but I haven't looked very hard
(and certainly not in a long time). Anyway, I think that copying
the GSL stuff (if you really want to) only requires asking us.
Brian probably has the copyright for the error-handling stuff
and can just re-license and gift that part of the code (unless
he has some objection or I am missing something).

In my follow-on comments to Field, I suggested the two-prototype model
used by (at least parts of) GSL, where each function has both a
"natural" abort-on-trouble version and a "return-code" version.
The "natural" version defers to the "return-code" version in
the obvious way. I am happy enough with this part of GSL, anyway.
Anything more complicated should probably be discussed critically.
I don't think I'm totally happy with every aspect of GSL error-handling,
mainly because I don't think it was implemented consistently across
the whole library.

The work required is obviously very easy, just a
bit tedious. Eventually the "natural" versions could be
automatically generated, if that seems worth doing. A
catalog of return codes and associated static error
messages is needed. That might require eye-balling
the libflame source, possibly doing some classification
work, depending on how organized they were about their
error conditions.


If you can think of something better, that would be great too.
Any ideas for improvement are welcome.

--
G. Jungman


  reply	other threads:[~2010-02-19  0:20 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <4a00655d1002171047t4e87fb85w88b609245e3f9a8e@mail.gmail.com>
2010-02-19  8:48 ` Field G. Van Zee
2010-02-18 19:51   ` Rhys Ulerich
2010-02-19  0:20     ` Gerard Jungman [this message]
2010-02-19 20:22     ` Brian Gough
2010-02-19  0:00   ` Gerard Jungman
2010-02-20  4:30     ` Field G. Van Zee
2010-02-23 13:13       ` Brian Gough
2010-02-24  1:42       ` Gerard Jungman
2010-02-19 18:19   ` Brian Gough
2010-02-19 18:41     ` Robert G. Brown
2010-02-19 20:57       ` Gerard Jungman
2010-02-19 21:46         ` Robert G. Brown
2010-02-19 22:34           ` Gerard Jungman
2010-02-22 23:09         ` Brian Gough
2010-02-23 10:50           ` Theodore Papadopoulo
2010-02-23 14:55             ` Robert G. Brown
2010-02-24  1:50               ` Gerard Jungman
2010-02-24 11:31                 ` Robert G. Brown
2010-02-23 17:39           ` James Amundson
2010-02-19 20:07   ` Brian Gough
2010-02-19 20:54     ` Gerard Jungman
2010-02-20  4:31     ` Field G. Van Zee

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1266538880.27033.100.camel@manticore.lanl.gov \
    --to=jungman@lanl.gov \
    --cc=gsl-discuss@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).