Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ jobs:
- name: Install Python dependencies
run: |
python3 -m pip install --upgrade pip
pip install --upgrade setuptools wheel
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
- name: Spotless Check
if: matrix.jdk == 11 || matrix.jdk == 17
Expand Down
7 changes: 3 additions & 4 deletions src/checkers/inference/InferenceAnnotatedTypeFactory.java
Original file line number Diff line number Diff line change
Expand Up @@ -512,9 +512,8 @@ protected void applyInferredAnnotations(
* This is the same as the super method, but we do not want to use defaults or typeAnnotator.
*/
@Override
protected void addComputedTypeAnnotations(
Tree tree, AnnotatedTypeMirror type, boolean iUseFlow) {
assert root != null
protected void addComputedTypeAnnotations(Tree tree, AnnotatedTypeMirror type) {
assert getRoot() != null
: "GenericAnnotatedTypeFactory.annotateImplicit: "
+ " root needs to be set when used on trees; factory: "
+ this.getClass();
Expand All @@ -527,7 +526,7 @@ protected void addComputedTypeAnnotations(
// typeAnnotator.visit(type, null);
defaults.annotate(tree, type);

if (iUseFlow) {
if (getUseFlow()) {
CFValue as = getInferredValueFor(tree);
if (as != null) {
applyInferredAnnotations(type, as);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -74,9 +74,8 @@ public void applyDefault(Default def) {
}

@Override
protected boolean shouldBeAnnotated(AnnotatedTypeMirror type, boolean applyToTypeVar) {
return super.shouldBeAnnotated(type, applyToTypeVar)
&& !type.hasAnnotation(VarAnnot.class);
protected boolean shouldBeAnnotated(AnnotatedTypeMirror type) {
return super.shouldBeAnnotated(type) && !type.hasAnnotation(VarAnnot.class);
}
}
}
5 changes: 2 additions & 3 deletions src/sparta/checkers/SimpleFlowAnnotatedTypeFactory.java
Original file line number Diff line number Diff line change
Expand Up @@ -233,11 +233,10 @@ protected void addCheckedCodeDefaults(QualifierDefaults defaults) {
}

@Override
protected void addComputedTypeAnnotations(
Tree tree, AnnotatedTypeMirror type, boolean useFlow) {
protected void addComputedTypeAnnotations(Tree tree, AnnotatedTypeMirror type) {
Element element = TreeUtils.elementFromTree(tree);
handleDefaulting(element, type);
super.addComputedTypeAnnotations(tree, type, useFlow);
super.addComputedTypeAnnotations(tree, type);
}

@Override
Expand Down