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: 361a2e68fd9c
Choose a base ref
...
head repository: nodejs/node-v0.x-archive
compare: dbd2b4fbb1ae
Choose a head ref
  • 11 commits
  • 17 files changed
  • 8 contributors

Commits on Aug 12, 2014

  1. lib: don't use emitter.listeners(type).length

    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    JacksonTian authored and trevnorris committed Aug 12, 2014
    Copy the full SHA
    931cbc1 View commit details
    Browse the repository at this point in the history

Commits on Aug 13, 2014

  1. querystring: remove prepended ? from query field

    Fixes an issue that caused the first querystring to be parsed prepending
    a "?" in the first variable name on relative urls with no #fragment
    
    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    warseph authored and trevnorris committed Aug 13, 2014
    Copy the full SHA
    678ead2 View commit details
    Browse the repository at this point in the history
  2. doc: typo fixes on stream, tls and http

    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    shama authored and trevnorris committed Aug 13, 2014
    Copy the full SHA
    0000416 View commit details
    Browse the repository at this point in the history
  3. dgram: remove new keyword from errnoException

    Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    JacksonTian authored and trevnorris committed Aug 13, 2014
    Copy the full SHA
    a8845eb View commit details
    Browse the repository at this point in the history
  4. src: require EventEmitter via NativeModule

    Fixes a recent change causing test-process-kill-pid.js to fail.
    
    Fixes: 931cbc1 "lib: don't use emitter.listeners(type).length"
    Signed-off-by: Trevor Norris <trev.norris@gmail.com>
    trevnorris committed Aug 13, 2014
    Copy the full SHA
    807acf7 View commit details
    Browse the repository at this point in the history
  5. src: change kIsolateSlot to 3

    The slot 0 and 1 had already been taken by "gin" and "blink" in Chrome,
    and the size of isolate's slots is 4 by default, so using 3 should hopefully
    make node work independently when embedded into other application.
    
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    zcbenz authored and indutny committed Aug 13, 2014
    Copy the full SHA
    44743ea View commit details
    Browse the repository at this point in the history
  6. tests: fix invalid hints flags dns test.

    1 is actually a valid flag on SmartOS. More generally, hints flags'
    values are defined by the underlying native flags, and these can have
    different values on different systems.
    
    Using (ADDRCONFIG | V4MAPPED) + 1 ensure that the flag will be invalid,
    since it will always be different from ADDRCONFIG, V4MAPPED, ADDRCONFIG
    | V4MAPPED,  0 and any other combination of even flags.
    Julien Gilli authored and tjfontaine committed Aug 13, 2014
    Copy the full SHA
    08f2b6c View commit details
    Browse the repository at this point in the history
  7. tests: do not hardcode service name in test-dns.

    Instead of hard-coding http service name in test-dns, retrieve it from
    /etc/services. This is not ideal, but it's still better than hard-coding
    it.
    
    Fixes #8047.
    
    Signed-off-by: Timothy J Fontaine <tjfontaine@gmail.com>
    Julien Gilli authored and tjfontaine committed Aug 13, 2014
    Copy the full SHA
    480348f View commit details
    Browse the repository at this point in the history
  8. test: check ipv6 support before testing it

    fix #7983
    fix #8049
    
    Conflicts:
    	test/common.js
    indutny authored and tjfontaine committed Aug 13, 2014
    Copy the full SHA
    dbb9834 View commit details
    Browse the repository at this point in the history
  9. tests: don't assume IPv4 only in remote addr tests

    Tests in test-net-remote-address-port.js assume that client and server
    sockets always use IPv4. However, depending on the OS and the network
    interfaces setup, this is not true. This change makes the test consider
    that both IPv4 or IPv6 sockets are valid
    
    Fixes #8096.
    Julien Gilli authored and tjfontaine committed Aug 13, 2014
    Copy the full SHA
    d46810c View commit details
    Browse the repository at this point in the history
  10. test: fix dns test

    Fix a few issues in test/internet/test-dns.js:
    - 'hint' should be 'hints'
    - reverse name lookup is not guaranteed to return 'localhost'
    - V4MAPPED hint requires IPV6 address family
    
    Signed-off-by: Timothy J Fontaine <tjfontaine@gmail.com>
    orangemocha authored and tjfontaine committed Aug 13, 2014
    Copy the full SHA
    dbd2b4f View commit details
    Browse the repository at this point in the history