Revision c021b2859740f76e6da61670d7cf9011989bfc36 authored by TypeScript Bot on 23 November 2019, 00:26:51 UTC, committed by Nathan Shively-Sanders on 23 November 2019, 00:26:51 UTC
Component commits:
5810765259 Emit defineProperty calls before param prop assignments
Note that I restricted this to --useDefineForClassFields is true.
Nothing changes when it's off. I think this is the correct fix for a
patch release.
However, in principal there's nothing wrong with moving parameter
property initialisation after property declaration initialisation. It
would be Extremely Bad and Wrong to rely on this working:
```ts
class C {
p = this.q // what is q?
constructor(public q: number) { }
}
```
But today it does, and probably somebody relies on it without knowing.
ec7959091a Put parameter property initialiser into defineProperty's value
be863550b7 Merge branch 'master' into fix-defineProperty-parameter-property-emit
8ff59b98b8 Combine ES5/ESNext into one test 1 parent d5bcb6f
.npmignore
built
doc
Gulpfile.js
internal
jenkins.sh
lib/README.md
lib/enu
netci.groovy
scripts
src
tests
Jakefile.js
.eslintrc
.eslintignore
.editorconfig
.failed-tests
.git
.git/
.gitattributes
.github/
.gitmodules
.settings/
.travis.yml
.circleci
.vscode/
.parallelperf.json
.mailmap
test.config
package-lock.json
yarn.lock
CONTRIBUTING.md
TEST-results.xml
.dockerignore
Dockerfile
.DS_Store
.eslintrc.json
.yarnrc
tmp
Computing file changes ...
