google/closure-compiler

need flag to enable `CompilerOptions.setAssumeStaticInheritanceRequired(true)`

Open

#3,559 opened on Feb 25, 2020

View on GitHub
 (3 comments) (0 reactions) (0 assignees)Java (7,176 stars) (1,197 forks)batch import
P3good first issue

Description

To avoid compilation breaking uses of this and super in static methods of ES6 classes, you have to get CommandLineRunner to do CompilerOptions.setAssumeStaticInheritanceRequired(true) when creating the Compiler object.

Unfortunately, there is no flag for this yet, and we can't make it the default behavior because of increase in output code size it causes for Google projects.

This would be a good community-volunteer thing to do.

Contributor guide

Tech stack
javajavascript
Domain
developer experience
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Javacommand line parsing
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60
Research direction
Examine the CommandLineRunner.java file, specifically the section where CompilerOptions are configured. Look for existing similar flags (e.g., useTypesForOptimization). Add a new command line option that calls setAssumeStaticInheritanceRequired(true) when parsed. Ensure the option is documented in the help text.