Reading a field of a class declared later in the file loses its refinement
Les mainteneurs répondent en général sous 2 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 62/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- java
- Domaine
- compilers, testing-qa
Piste de recherche
Start with RefinementTypeChecker.visitCtField and visitCtFieldRead, which the issue identifies as the source-order context handling path. Use the supplied ReadBefore reproducer to confirm the false “not enough information” error, then compare it with the version where Job is declared earlier. Done means the reproducer reports “Correct! Passed Verification.” and reads still work when the class is declared later.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Description
Reading a field of an object whose class is declared later in the file does not pick up the field's refinement, so a correct program fails with a false "not enough information" error. Writes to such fields are handled by #314 (fixing #308); reads are not.
Minimal reproducer
import liquidjava.specification.Refinement;
public class ReadBefore {
private final Job job = new Job();
@Refinement("_ >= 0")
public int get() { return job.port; }
static class Job { @Refinement("_ >= 0") int port; }
}
Expected
Correct! Passed Verification.
Actual
Refinement Error: true is not a subtype of #ret² >= 0
5 | public int get() { return job.port; }
| ^^^^^^^^^^^^^^^^
--> Not enough information to prove the expected refinement. Add a refinement or condition to constrain it.
Moving Job above get makes it pass.
Cause
Fields enter the context only in the second pass (RefinementTypeChecker.visitCtField), in source order. When job.port is read, this#port is not in context yet, so visitCtFieldRead falls through to new Predicate() (no information).
Reproduced on main at fbfb4e23 (liquidjava-verifier 0.0.35), and on the #314 branch.
- Langage dominant
- Java
- Étoiles
- 67
- Forks
- 36
- Merge moyen
- 4 j 17 h
- PR mergées (30 j)
- 7
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de liquid-java/liquidjava
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
liquid-java/liquidjava#321 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 4/5 3-5 jours Accessibilité débutants 68/100
liquid-java/liquidjava#323 ·
Les mainteneurs répondent en général sous 2 jours
-
Soundness: a field of another class or object is read as this class's field with the same nameOuvertebug
Difficulté 3/5 1-2 jours Accessibilité débutants 76/100
liquid-java/liquidjava#322 ·
Les mainteneurs répondent en général sous 2 jours
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 52/100
liquid-java/liquidjava#318 ·
Les mainteneurs répondent en général sous 2 jours
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
liquid-java/liquidjava#317 ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de liquid-java/liquidjava
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
aoqia194/leaf-loader#19 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
apache/streampark#4521 ·
-
Update license yearOuverte0 - Backlog 1 - Ready documentation good first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
cbor
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
FasterXML/jackson-dataformats-binary#844 ·
Les mainteneurs répondent en général sous 1 jour
-
Issue: Bug
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
OpenAPITools/openapi-generator#25107 ·
Les mainteneurs répondent en général sous 1 jour