Welcome! Log In Create A New Profile

Advanced

Re: Dead code in accept

August 05, 2013 05:06AM
I agree, I view dead code at run-time as a liability. Something that can go
wrong. At compile time, you know everything about the sytem, the config
script makes assumptions and the binaries shouldnt work if moved to a
different OS etc anyway.

I am not a nginx developer just a C++ developer.


On Mon, Aug 5, 2013 at 4:28 PM, Serguei I. Ivantsov <
manowar@gsc-game.kiev.ua> wrote:

> I think it is not a matter for trade - "dead code" - is bad programming
> technique. And it should be eliminated in a fast web server.
> Of course, general impact on overall performance is not significant, but
> on function level it will be much noticeable. I can make a perf profiling
> test to get exact counters.
>
> > Hello!
> >
> > On Mon, Aug 05, 2013 at 12:15:04AM +0300, Serguei I. Ivantsov wrote:
> >
> > [...]
> >
> >> A little test with high volume of simple requests shows 0.5% overall
> >> speed
> >> improvement.
> >
> > Are you sure the numbers are significant? Doing a ministat(1)
> > analysis or similar is a good idea.
> >
> > http://www.freebsd.org/cgi/man.cgi?query=ministat&amp;sektion=1
> >
> > --
> > Maxim Dounin
> > http://nginx.org/en/donation.html
> >
>
>
> _______________________________________________
> nginx-devel mailing list
> nginx-devel@nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel
>
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

Dead code in accept

Serguei I. Ivantsov 679 August 04, 2013 05:16PM

Re: Dead code in accept

Maxim Dounin 305 August 04, 2013 06:20PM

Re: Dead code in accept

Tom van der Woerdt 297 August 05, 2013 03:16AM

Re: Dead code in accept

splitice 558 August 05, 2013 05:06AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 95
Record Number of Users: 6 on February 13, 2018
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready