You are viewing a plain text version of this content. The canonical link for it is here.
Posted to general@commons.apache.org by Garrett Rooney <ro...@electricjellyfish.net> on 2003/11/05 02:15:12 UTC

[PATCH] serf compile fixes

So with the amount of change happening in serf, I imagine these 
problems have already been fixed in gstein or jerenkrantz's local 
tree's, but in case they haven't, here's a few compile fixes.

-garrett

* serf.h
   (serf_request_get_alloc): return a pointer to a serf_bucket_alloc_t, 
as
    you can't return the structure since it's only forward declared, and 
we
    only have a pointer anyway.

* context.c
   (serf_request_deliver): remove return, this function returns void.
   (serf_request_get_alloc): adjust for change to prototype.


Re: [PATCH] serf compile fixes

Posted by Garrett Rooney <ro...@electricjellyfish.net>.
On Nov 4, 2003, at 10:12 PM, Greg Stein wrote:

> On Tue, Nov 04, 2003 at 08:15:12PM -0500, Garrett Rooney wrote:
>> So with the amount of change happening in serf, I imagine these
>> problems have already been fixed in gstein or jerenkrantz's local
>> tree's, but in case they haven't, here's a few compile fixes.
>
> Nope. I hadn't run a compile after my last round of changes. There 
> isn't
> much need to do that, except as a syntax check :-) (eventually, we'll 
> have
> enough functionality to really attack serf_get and make it work; then
> we'll be compiling all the time...)  So in that sense, your changes are
> appreciated.
>
> [ well, the suggestions of changes; no patch was attached :-) ]

Odd, in my 'Sent' folder, there is a patch attached to the mail...  
Does this mailing list strip attachments for some reason?  If so I 
suppose I'll have to send it inline with the mail next time...

> The changes were easily made, and I've committed them. Thanks!

No problem ;-)

-garrett


Re: [PATCH] serf compile fixes

Posted by Greg Stein <gs...@lyra.org>.
On Tue, Nov 04, 2003 at 08:15:12PM -0500, Garrett Rooney wrote:
> So with the amount of change happening in serf, I imagine these 
> problems have already been fixed in gstein or jerenkrantz's local 
> tree's, but in case they haven't, here's a few compile fixes.

Nope. I hadn't run a compile after my last round of changes. There isn't
much need to do that, except as a syntax check :-) (eventually, we'll have
enough functionality to really attack serf_get and make it work; then
we'll be compiling all the time...)  So in that sense, your changes are
appreciated.

[ well, the suggestions of changes; no patch was attached :-) ]

The changes were easily made, and I've committed them. Thanks!

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/