Repeated code with exporting

[falsandtru]

This repeated code is added in a few weeks.

TypeScript Version: master

Code

export class C {}
export namespace C {
  var c;
}

Expected behavior:

"use strict";
var C = (function () {
    function C() {
    }
    return C;
}());
exports.C = C;
(function (C) {
    var c;
})(C = exports.C || (exports.C = {}));

Actual behavior:

"use strict";
var C = (function () {
    function C() {
    }
    return C;
}());
exports.C = C;
(function (C) {
    var c;
})(C = exports.C || (exports.C = {}));
exports.C = C; // <- here

cf:

export namespace C {
  var c;
}

"use strict";
var C;
(function (C) {
    var c;
})(C = exports.C || (exports.C = {}));

[falsandtru]

Oh... seems like I hacked GitHub while I was sleeping. I reported it to GitHub.

@falsandtru falsandtru added the Bug label 3 hours ago
@rbuckton rbuckton was assigned by falsandtru 3 hours ago
@falsandtru falsandtru added this to the Future milestone 3 hours ago

[mhegazy]

:D I did that on your behalf.

looks like github has been having issues today, and all my labels have been attributed to issue author.

[falsandtru]

I got a response from GitHub. They said Our team just pushed a fix for this. So it will be fixed soon.

[jakebailey]

Just checking out old bugs; this one was actually "fixed" very recently in 5.1 by #53901.

Though, I will say that this may not have really been a "bug" at all due to this sort of thing being required to emulate the fact that one can observe the exported value indirectly while it's partially evaluated (e.g. before the second declaration has had time to execute), as discussed in the PR.

But, the "unwanted" line noted in the issue is now gone so I think this can be closed.