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: 5ce4eed54dce
Choose a base ref
...
head repository: nodejs/node-v0.x-archive
compare: 001f9b46e72a
Choose a head ref
  • 6 commits
  • 5 files changed
  • 5 contributors

Commits on Nov 27, 2013

  1. stream_wrap: don't call Number::New() 

    Replace call to Number::New() with a call to Integer::NewFromUnsigned().

Profiling a Real World(TM) application with perf(1) suggests that the
conversion of its argument from integer to double is disproportionally
costly: over 60% of CPU cycles accountable to WriteStringImpl() are
attributable to the conversion.

After changing it to Integer::NewFromUnsigned(), WriteStringImpl()
has dropped from the 'most costly functions' top ten altogether.
    @bnoordhuis
    bnoordhuis committed Nov 27, 2013
    Copy the full SHA
    658aeb2 View commit details
    Browse the repository at this point in the history

Commits on Nov 28, 2013

  1. doc: http: properly document callback argument

    @seishun @bnoordhuis
    seishun authored and bnoordhuis committed Nov 28, 2013
    Copy the full SHA
    207a3e1 View commit details
    Browse the repository at this point in the history

  2. doc: http: document ServerResponse 'finish' event

    @mridgway @bnoordhuis
    mridgway authored and bnoordhuis committed Nov 28, 2013
    Copy the full SHA
    a32b878 View commit details
    Browse the repository at this point in the history

Commits on Nov 30, 2013

  1. doc: net: fix typo in example code

    @yorkie @bnoordhuis
    yorkie authored and bnoordhuis committed Nov 30, 2013
    Copy the full SHA
    fcfaa39 View commit details
    Browse the repository at this point in the history

Commits on Dec 2, 2013

  1. build: include postmortem symbols on linux 

    Previously we were building the symbols, but the linker was garbage
collecting the symbols because they weren't used. Inform the linker
that we want to keep all symbols from v8 around.
    @tjfontaine
    tjfontaine committed Dec 2, 2013
    2
    Copy the full SHA
    6877e64 View commit details
    Browse the repository at this point in the history

  2. Merge remote-tracking branch 'upstream/v0.10' 

    Conflicts:
	src/stream_wrap.cc
    @tjfontaine
    tjfontaine committed Dec 2, 2013
    Copy the full SHA
    001f9b4 View commit details
    Browse the repository at this point in the history