[m-rev.] for post-commit review: explicit streams in analysis*.m
Julien Fischer
jfischer at opturion.com
Fri Apr 7 14:59:37 AEST 2023
On Fri, 7 Apr 2023, Zoltan Somogyi wrote:
> Use explicit streams in analysis*.m.
>
> compiler/analysis.m:
> compiler/analysis.file.m:
> Switch to using explicit streams.
>
> Use io.format when relevant.
>
> Give some fields, predicates and variables more meaningful names.
>
> Replace some rafe-isms with code in our usual coding style.
I don't recall Ralph ever working on the analysis framework, I suspect
they are actually Simon-isms.
> Add XXXs for possible future improvements.
>
> Replace the flag that says whether debug is enabled with a maybe
> wrapped around the stream where debug output should go.
>
> compiler/handle_options.m:
> Initialize the maybe debug stream.
>
> compiler/Mercury.options:
> Stop disabling the warning for the use of implicit streams
> for analysis.m and analysis.file.m.
That's fine.
Julien.
More information about the reviews
mailing list