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: c171c490f2d8
Choose a base ref
...
head repository: nodejs/node-v0.x-archive
compare: 6ed861dd7fee
Choose a head ref
  • 1 commit
  • 1 file changed
  • 1 contributor

Commits on Aug 20, 2013

  1. events: have events module exports EventEmitter 

    This change is 100% backwards compatible.

This change will make using `EventEmitter` slightly simpler / nicer and
adheres to the best practice set forth by substack.

```js
var EventEmitter = require("events")

var emitter = new EventEmitter()
```

The only difference is that we now have to set `EventEmitter` as a
property of `EventEmitter` for backwards compatibility like we do with
[`Stream`][1]

We have also set the `usingDomains` property on the `EventEmitter`
constructor itself because that aligns with it's current usage of
`require("events").usingDomains = true`

There are other internals that would benefit from this change as well
like `StringDecoder`
    @Raynos @isaacs
    Raynos authored and isaacs committed Aug 20, 2013
    3
    Copy the full SHA
    6ed861d View commit details
    Browse the repository at this point in the history