Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to EISOP 3.42-eisop4 from 3.42-eisop3 #451

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
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
Loading