echo-nginx-module

info · 2013-12-06T02:47:56+00:00

Hello! On Wed, Dec 4, 2013 at 4:12 PM, leaf corcoran wrote: > If you're using require to load packages then you can already implement this > in pure L...

echo-nginx-module

info

Heya agentzh,

Can you fix these pse;

/echo-nginx-module/src/ngx_http_echo_subrequest.c(346) : warning C4244: '=' : conversion from 'off_t' to 'ssize_t', possible loss of data
/echo-nginx-module/src/ngx_http_echo_subrequest.c(744) : warning C4307: '*' : integral constant overflow
\echo-nginx-module\src\ngx_http_echo_subrequest.c(553) : warning C4702: unreachable code

agentzh

Hello!

On Thu, Dec 5, 2013 at 10:47 AM,  <i...@ecsystems.nl> wrote:
> Can you fix these pse;
>
> /echo-nginx-module/src/ngx_http_echo_subrequest.c(346) : warning C4244: '='
> : conversion from 'off_t' to 'ssize_t', possible loss of data
> /echo-nginx-module/src/ngx_http_echo_subrequest.c(744) : warning C4307: '*'
> : integral constant overflow
> \echo-nginx-module\src\ngx_http_echo_subrequest.c(553) : warning C4702:
> unreachable code
>

Thank you for the report! Just committed a fix for these warnings from
the Microsoft compiler:

  https://github.com/agentzh/echo-nginx-module/commit/58af452

Please try out the latest git master.

Thanks!
-agentzh

info

On Thursday, December 5, 2013 9:55:28 PM UTC+1, agentzh wrote:

Please try out the latest git master.

Thanks!
-agentzh

We have a winner ! tnx :)