Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: nodejs/node-v0.x-archive
base: 7c0419730b23
Choose a base ref
...
head repository: nodejs/node-v0.x-archive
compare: 25702abc5638
Choose a head ref
  • 1 commit
  • 1 file changed
  • 1 contributor

Commits on Oct 1, 2014

  1. net: remove use of arguments in Server constructor

    The current implementation uses the arguments object in the Server()
    constructor. Since both arguments to Server() are optional, there was a
    high likelihood of accessing a non-existent element in arguments, which
    carries a performance overhead. This commit replaces the arguments
    object with named arguments.
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    cjihrig authored and trevnorris committed Oct 1, 2014
    Copy the full SHA
    25702ab View commit details
    Browse the repository at this point in the history