TypeScript: Modifiers not included in range of TSParameterProperty

[James Henry]

Input Code

Example of existing fixture in babylon tests:

class C {
    constructor(
        readonly r,
        public pu: number,
        protected po?,
        private pi?: number,
        public readonly pur,
        // Also works on AssignmentPattern
        readonly x = 0,
        public y?: number = 0) {}
}

The TypeScript plugin for babylon currently does not include the modifiers (readonly, public etc) within the location data for the relevant TSParameterProperty node.

This is not the case in the TypeScript compiler for the equivalent Parameter node, and also not in typescript-eslint-parser.

@andy-ms If you agree that this should be changed, I can work on this.

[Babel Bot]

This issue has been moved to babel/babel#6679.