liquid-java / liquid-java/liquidjava
Separating logging information from other function arguments
Open
@FrogOfJuly is already working on this.
Since Oct 13, 2022.
- Dominant language
- Java
- Stars
- 67
- Forks
- 36
- Avg merge
- 10d 18h
- Merged PRs (30d)
- 3
Description
There are multiple places where `CtElement` is passed only for logging information. I think it is better to create separate type and pass it around instead.
Examples:
`liquidjava.processor.context.Contex.addVarToContext`
`liquidjava.processor.refinement_checker.object_checkers.AuxStateHandler.createStatePredicate`
`liquidjava.processor.refinement_checker.TypeChecker.checksStateSMT` (changed to other other type already)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.