Skip to content

Identifier name fixes#596

Merged
tomblind merged 10 commits intomasterfrom
identifier-name-fixes
May 26, 2019
Merged

Identifier name fixes#596
tomblind merged 10 commits intomasterfrom
identifier-name-fixes

Conversation

@tomblind
Copy link
Copy Markdown
Collaborator

@tomblind tomblind commented May 25, 2019

fixes #594

@tomblind tomblind requested a review from Perryvw May 26, 2019 14:13
@ark120202
Copy link
Copy Markdown
Contributor

@decorator
export class $ {}
local ____exports = {}
____exports["$"] = {}
local _____24 = ____exports["$"]
_____24.name = "$"
_____24.__index = _____24
_____24.prototype = {}
_____24.prototype.__index = _____24.prototype
_____24.prototype.constructor = _____24
function _____24.new(...)
    local self = setmetatable({}, _____24.prototype)
    self:____constructor(...)
    return self
end
function _____24.prototype.____constructor(self)
end
$ = __TS__Decorate({decorator}, $)
return ____exports

On master it is _____24 = __TS__Decorate({decorator}, _____24)

} else {
identifier = this.transformIdentifierExpression(element.name);
}
identifier = this.createShorthandIdentifier(valueSymbol, element.name);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-let identifier: tstl.Expression | undefined;
-identifier = this.createShorthandIdentifier(valueSymbol, element.name);
+let identifier = this.createShorthandIdentifier(valueSymbol, element.name);

@tomblind tomblind merged commit a2046bd into master May 26, 2019
@tomblind tomblind deleted the identifier-name-fixes branch May 26, 2019 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

More Identifier Issues

3 participants