[16953] in Kerberos_V5_Development
Re: RFC: libverto nearing release
daemon@ATHENA.MIT.EDU (Greg Hudson)
Thu Jun 30 15:25:39 2011
From: Greg Hudson <ghudson@mit.edu>
To: Nathaniel McCallum <npmccallum@redhat.com>
In-Reply-To: <1309447877.26530.14.camel@localhost>
Date: Thu, 30 Jun 2011 15:25:34 -0400
Message-ID: <1309461934.19355.179.camel@t410>
Mime-Version: 1.0
Cc: "krbdev@mit.edu" <krbdev@mit.edu>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: krbdev-bounces@mit.edu
On Thu, 2011-06-30 at 11:31 -0400, Nathaniel McCallum wrote:
> The purpose of this email is to let everyone know that libverto is
> nearing its first release (hopefully today) and I would like to request
> feedback. The project is hosted here: https://fedorahosted.org/libverto/
Here are a few things I flagged, mostly cosmetic.
1. I couldn't find the git repository from the web site.
2. We would prefer not to see camel-case in the type names, as some of
them will appear in our function signatures.
3. We would prefer typedefs for the types in order to reduce the amount
of horizontal space it takes to add an event context to a function
signature. I have a personal preference for typedeffing the pointer
type for abstract types, so that to the caller it doesn't look like a
dereferenceable object.
4. The documentation of verto_new(NULL) exposes more of the
implementation than I think is really wise; in particular, promising to
try to figure out what event libraries you're linked to seems
over-generous.
> There are a few known issues. The most important is that libverto has
> only been tested on Fedora and there will likely be some (hopefully only
> minor) issues on other platforms.
When (1) is addressed, I will test the build on Solaris 10.
_______________________________________________
krbdev mailing list krbdev@mit.edu
https://mailman.mit.edu/mailman/listinfo/krbdev
