Re: [libmicrohttpd] socket_context not set (libmicrohttpd-0.9.50)

2017-06-05 Thread Miguel Sancho
Christian, > Eh, what is that "*" doing in the "if" line? reason is that in definition of the MHD_ConnectionInfo union, I see socket_context defined as "void**", maybe it should be just "void*" > Also, what code do you use in the connection notification callback to set the socket_contest? MyCon

Re: [libmicrohttpd] socket_context not set (libmicrohttpd-0.9.50)

2017-06-05 Thread Christian Grothoff
On 06/05/2017 06:01 PM, Miguel Sancho wrote: > Christian, > >> Eh, what is that "*" doing in the "if" line? > > reason is that in definition of the MHD_ConnectionInfo union, I see > socket_context defined as "void**", maybe it should be just "void*" It is indeed "void *" in more recent versions

Re: [libmicrohttpd] socket_context not set (libmicrohttpd-0.9.50)

2017-06-05 Thread Miguel Sancho
ok, I understand now, thanks!

[libmicrohttpd] [GSoC Update] Week 1

2017-06-05 Thread Didik Setiawan
= SUMMARY == My public fork of Wget2 project is available here [1]. I will continuously keep pushing my work so anyone interested can track me there. Feel free to participate in the discussions going on merge request with my mentor

Re: [libmicrohttpd] [GSoC Update] Week 1

2017-06-05 Thread Christian Grothoff
Maybe I missed something, but why should the MHD dependency become _mandatory_? Why not conditionally-compile and run the tests only if MHD is present? MHD does the same for libcurl. Making the dependency optional also avoids the obvious possibility of circular dependencies if we ever were to add

Re: [libmicrohttpd] [GSoC Update] Week 1

2017-06-05 Thread Didik Setiawan
On Mon, 05 Jun 2017 23:51:00 +0700 Christian Grothoff wrote: > Maybe I missed something, but why should the MHD dependency become > _mandatory_? Why not conditionally-compile and run the tests only if MHD > is present? Maybe I just misinterpreting this [1]? What Darshit said about removing old

Re: [libmicrohttpd] [GSoC Update] Week 1

2017-06-05 Thread Tim Rühsen
Hi Didik, thanks for the summary of week #1 ! On Dienstag, 6. Juni 2017 00:40:56 CEST Didik Setiawan wrote: > On Mon, 05 Jun 2017 23:51:00 +0700 Christian Grothoff wrote: > > Maybe I missed something, but why should the MHD dependency become > > _mandatory_? Why not conditionally-compile and ru

Re: [libmicrohttpd] [GSoC Update] Week 1

2017-06-05 Thread Didik Setiawan
On Tue, 06 Jun 2017 02:37:54 +0700 Tim Rühsen wrote: > I guess Christian meant to skip the appropriate tests if MHD is not > installed/ > available. We should definitely get rid of our server code in libwget and > focus on client functionality. Kick it out once MHD code works :-) Understood.