[PATCH] more data structures public in mongoose.h

Ben Gras beng at shrike-systems.com
Wed Nov 19 19:30:19 UTC 2014


That is a good point. Shall I change the patch to use mg_* prefixes for all
globally visible type names?


On Wed, Nov 19, 2014 at 8:24 PM, Gedare Bloom <gedare at rtems.org> wrote:

> Hi Ben,
>
> Due to the license change, we can consider our version of Mongoose as
> "RTEMS Project" maintained for now. I don't know if anyone is
> maintaining a fork with an appropriate license.
>
> Moving the struct definitions to the header file is sensible, although
> the usual caveat is that users might have namespace conflicts.
>
> Gedare
>
>
>
> On Wed, Nov 19, 2014 at 1:16 PM, Ben Gras <beng at shrike-systems.com> wrote:
> > The attached patch is a proposed solution to a problem I am having with
> > interfacing with the mongoose httpd for which I've found no canonical (or
> > any other) clean solution.
> >
> > The problem is that some of the data structures needed for some
> operations,
> > specifically writing callbacks that get invoked for some HTTP requests,
> are
> > declared in mongoose.c and therefore are not available to external
> programs,
> > i.e. clients of mongoose. This is the mg_connection struct & its
> > dependencies. It is passed as an argument to callbacks, and even without
> > referencing the object the code won't compile if the struct isn't fully
> > known.
> >
> > The change is moving some declarations from mongoose.c to mongoose.h.
> >
> > Unfortunately this diverges from the original code, then again mongoose
> > isn't tracked any more since the license change I take it, so perhaps
> that's
> > not so bad. I've made the changes __rtems__-conditional so it's clear
> where
> > the divergence is.
> >
> >
> > _______________________________________________
> > devel mailing list
> > devel at rtems.org
> > http://lists.rtems.org/mailman/listinfo/devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20141119/f4703447/attachment-0001.html>


More information about the devel mailing list